WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
279379
SVGAnimatedLength initial value implementation needs work
https://bugs.webkit.org/show_bug.cgi?id=279379
Summary
SVGAnimatedLength initial value implementation needs work
Ahmad Saleem
Reported
2024-09-09 14:18:30 PDT
Hi Team, While looking into Blink's commit, I came across few more failing test cases: Test Case:
https://jsfiddle.net/53f12ame/
^ Chrome Canary 130 passes all, while Firefox Nightly 132 fails 8 while Safari Technology Preview 202 fails 48. Blink Commit:
https://chromium.googlesource.com/chromium/src/+/db0702d6bdda54529bef038201752b37a3eadf8e
Just raising so we can track, don't know if we can merge it or not or do quick stuff. Thanks!
Attachments
test case (should not alert)
(585 bytes, text/html)
2024-09-09 15:20 PDT
,
Said Abou-Hallawa
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Said Abou-Hallawa
Comment 1
2024-09-09 15:20:32 PDT
Created
attachment 472508
[details]
test case (should not alert)
Said Abou-Hallawa
Comment 2
2024-09-09 15:21:58 PDT
The bug happens because when removing an attribute, like the 'x' of SVGFilterElement, SVGFilterElement::attributeChanged(() gets called with newValue equal to empty string.
Radar WebKit Bug Importer
Comment 3
2024-09-16 14:19:14 PDT
<
rdar://problem/136102554
>
Ahmad Saleem
Comment 4
2024-12-29 07:02:37 PST
Pull request:
https://github.com/WebKit/WebKit/pull/38414
Robert Longson
Comment 5
2025-01-25 05:36:55 PST
https://bugzilla.mozilla.org/show_bug.cgi?id=1943740
fixes the remaining Firefox failures and adds a WPT so perhaps you won't need to have a webkit test, just mark the WPT as passing.
EWS
Comment 6
2025-11-26 11:41:06 PST
Committed
303577@main
(38b65cc65f13): <
https://commits.webkit.org/303577@main
> Reviewed commits have been landed. Closing PR #38414 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug