Bug 306387
| Summary: | Percent-decode fragments before using them for SVG references | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Karl Dubost <karlcow> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | karlcow, sabouhallawa, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://github.com/web-platform-tests/wpt/pull/58954 | ||
Ahmad Saleem
Hi Team,
While looking into SVGURL bugs, I noticed we are failing following tests:
Test Case: https://jsfiddle.net/p4e8h5da/ and https://jsfiddle.net/10zuhe7d/
Blink Commit - https://chromium.googlesource.com/chromium/src.git/+/7fed5aeb041c935425adb69333bdf610bef896ee
We might need to use `decodeURLEscapeSequences` similar to Blink in our SVG URL implementation.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/169582378>
Karl Dubost
https://jsfiddle.net/p4e8h5da/ is green in STP 240, Firefox Nightly, Chrome Canary
https://jsfiddle.net/10zuhe7d/ is red in STP 240, Green in Firefox Nightly, Chrome Canary
The test is
<!DOCTYPE html>
<svg>
<defs>
<linearGradient id="foo">
<stop stop-color="green"/>
</linearGradient>
</defs>
<rect width="100" height="100" fill="url(#%66%6f%6f) red"/>
</svg>
Karl Dubost
Pull request: https://github.com/WebKit/WebKit/pull/61899
EWS
Committed 310443@main (2dc442120874): <https://commits.webkit.org/310443@main>
Reviewed commits have been landed. Closing PR #61899 and removing active labels.
Karl Dubost
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/58954