Bug 226721 - SVGSVGElement's currentView + useCurrentView are non-standard
Summary: SVGSVGElement's currentView + useCurrentView are non-standard
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: GoodFirstBug, InRadar, WPTImpact
Depends on:
Blocks: 191292
  Show dependency tree / graph
 
Reported: 2021-06-07 03:40 PDT by Philip Jägenstedt
Modified: 2023-11-20 15:19 PST (History)
17 users (show)

See Also:

Attachments
Patch (2.76 KB, patch)
2022-09-24 10:04 PDT, Rob Buis 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Jägenstedt 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
Comment 1 Cameron McCormack (:heycam) 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.
Comment 2 Radar WebKit Bug Importer 2021-06-14 03:40:15 PDT 
<rdar://problem/79281167>
Comment 3 Rob Buis 2022-09-24 10:04:14 PDT 
Created attachment 462585 [details]
Patch
Comment 5 Ahmad Saleem 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.