Bug 250575 - Sync SVGAnimationElement.idl with the specs
Summary: Sync SVGAnimationElement.idl with the specs
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: Safari Technology Preview
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-01-13 07:38 PST by Ahmad Saleem
Modified: 2023-04-21 11:21 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-01-13 07:38:28 PST
Hi Team,

While going through Blink's commit, came across few more failing test case in Safari Technology Preview 161:

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/751a75155d16a66b3f615545671c58443a70ab82

Failing Test Case - method-argument-typechecks.html - https://jsfiddle.net/x74nutya/show
Failing Test Case - method-argument-aritychecks.html - https://jsfiddle.net/ck2z6gsw/show

^ In the last one, Chrome Canary 111 and Firefox Nightly 110 also fail two test. We can get updated copy from source.chromium.org, which might not have these.

Just wanted to raise so we can fix these failures.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-01-20 07:39:15 PST
<rdar://problem/104475517>
Comment 2 Said Abou-Hallawa 2023-01-25 16:01:45 PST
Syncing SVGFEDropShadowElement and SVGFEGaussianBlurElement.setStdDeviation() with the specs is addressed in bug 251182. 

Syncing SVGTests with the specs is addressed in bug 204125


Let's have this bug for syncing the SVGAnimationElement with the specs.
Comment 3 Ahmad Saleem 2023-01-25 16:02:33 PST
(In reply to Said Abou-Hallawa from comment #2)
> Syncing SVGFEDropShadowElement and
> SVGFEGaussianBlurElement.setStdDeviation() with the specs is addressed in
> bug 251182. 
> 
> Syncing SVGTests with the specs is addressed in bug 204125
> 
> 
> Let's have this bug for syncing the SVGAnimationElement with the specs.

Cool! I will push new commit tomorrow. Got distract with other quick wins. :-)
Comment 5 EWS 2023-04-21 11:21:17 PDT
Committed 263253@main (c7a661464e1f): <https://commits.webkit.org/263253@main>

Reviewed commits have been landed. Closing PR #12762 and removing active labels.