Bug 226721

Summary: SVGSVGElement's currentView + useCurrentView are non-standard
Product: WebKit Reporter: Philip Jägenstedt <philip>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: ahmad.saleem792, cdumez, dino, esprehn+autocc, ews-watchlist, fmalita, gyuyoung.kim, heycam, karlcow, kondapallykalyan, pdr, rbuis, sabouhallawa, schenney, sergio, webkit-bug-importer, zimmermann
Priority: P2 Keywords: GoodFirstBug, InRadar, WPTImpact
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 191292    
Attachments:
Description Flags
Patch ews-feeder: commit-queue-

Philip Jägenstedt
Reported 2021-06-07 03:40:02 PDT
These two attributes on SVGSVGElement are gone from the spec: https://github.com/w3c/svgwg/commit/4c26fd36937a65192024208d85c144a21071b057 https://www.w3.org/Graphics/SVG/WG/track/actions/3800?changelog They were removed from Chromium here: https://bugs.chromium.org/p/chromium/issues/detail?id=629445 Gecko doesn't have currentView, but the bug for removing useCurrentView is here: https://bugzilla.mozilla.org/show_bug.cgi?id=1174097
Attachments
Patch (2.76 KB, patch)
2022-09-24 10:04 PDT, Rob Buis 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Cameron McCormack (:heycam)
Comment 1 2021-06-07 16:22:27 PDT
(In reply to Philip Jägenstedt from comment #0) > Gecko doesn't have currentView, but the bug for removing useCurrentView is > here: > https://bugzilla.mozilla.org/show_bug.cgi?id=1174097 That has a patch I wrote 6 years ago and didn't land. :-) Anyway, yes these are a straightforward removal.
Radar WebKit Bug Importer
Comment 2 2021-06-14 03:40:15 PDT
<rdar://problem/79281167>
Rob Buis
Comment 3 2022-09-24 10:04:14 PDT
Created attachment 462585 [details] Patch
Ahmad Saleem
Comment 5 2023-08-24 14:56:31 PDT
We have following as well: https://github.com/w3c/svgwg/issues/747 To revert these changes, I tried to do this PR: https://github.com/WebKit/WebKit/pull/13851 but now will look for input before proceeding. @Said - any input on above GitHub issue.
Note You need to log in before you can comment on or make changes to this bug.