Bug 304383
| Summary: | Remove `SVGCursorElement` | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Ahmad Saleem <ahmad.saleem792> |
| 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 | ||
Ahmad Saleem
Hi Team,
This is tracking but to remove SVGCursorElement code, which was made to be not web exposed in 2022 (https://commits.webkit.org/255702@main).
Blink Removed it in 2016 - https://chromium.googlesource.com/chromium/src.git/+/f63d67df608751f83eaa15a67df069a38714f171
Blink's old Intent for Removal - https://groups.google.com/a/chromium.org/g/blink-dev/c/r0lnh5Rw23M
Just raising so we can fix it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/167158212>
Karl Dubost
https://wpt.fyi/results/svg/historical.html
Firefox removed it too.
https://searchfox.org/firefox-main/search?q=SVGCursorElement&path=&case=false®exp=false
Karl Dubost
On the other hand it's called in many places.
https://searchfox.org/wubkat/search?q=SVGCursorElement&path=cpp&case=false®exp=false
Ahmad Saleem
(In reply to Karl Dubost from comment #3)
> On the other hand it's called in many places.
> https://searchfox.org/wubkat/
> search?q=SVGCursorElement&path=cpp&case=false®exp=false
Yes but I was able to remove it successfully in local build. Although we might need to run it via EWS to see test failures properly.
Ahmad Saleem
Pull request: https://github.com/WebKit/WebKit/pull/60649
EWS
Committed 309308@main (230771c93ca1): <https://commits.webkit.org/309308@main>
Reviewed commits have been landed. Closing PR #60649 and removing active labels.