Bug 216206 - Remove SVGZoomEvent
Summary: Remove SVGZoomEvent
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sam Weinig
URL:
Keywords: InRadar
Depends on:
Blocks: 191292
  Show dependency tree / graph
 
Reported: 2020-09-04 17:19 PDT by Sam Weinig
Modified: 2022-09-26 02:16 PDT (History)
18 users (show)

See Also:

Attachments
Patch (26.44 KB, patch)
2022-09-24 00:26 PDT, Rob Buis 		no flags Details | Formatted Diff | Diff
Patch (30.87 KB, patch)
2022-09-25 05:01 PDT, Rob Buis 		no flags Details | Formatted Diff | Diff
Patch (33.94 KB, patch)
2022-09-25 13:41 PDT, Rob Buis 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) 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 Sam Weinig 2020-09-04 17:19:29 PDT 
According to https://github.com/whatwg/dom/issues/283 and https://github.com/w3c/svgwg/commit/3844475, it looks like SVGZoomEvent has been removed from the spec and from at least Chrome and Firefox. Should we remove it too?
Comment 1 Sam Weinig 2020-09-04 17:20:00 PDT 
Said, any thoughts on this?
Comment 2 Said Abou-Hallawa 2020-09-04 18:58:41 PDT 
Yes, I think we should remove it. This page mentioned it is removed from SVG2: https://www.w3.org/TR/SVG/changes.html

Also our WPT expectation for svg has this line:

imported/w3c/web-platform-tests/svg/historical-expected.txt:FAIL SVGZoomEvent interface must be removed assert_false: expected false got true
Comment 3 Radar WebKit Bug Importer 2020-09-11 17:20:14 PDT 
<rdar://problem/68740396>
Comment 4 Rob Buis 2022-09-24 00:26:58 PDT 
Created attachment 462577 [details]
Patch
Comment 5 Rob Buis 2022-09-25 05:01:44 PDT 
Created attachment 462596 [details]
Patch
Comment 6 Rob Buis 2022-09-25 13:41:37 PDT 
Created attachment 462598 [details]
Patch
Comment 7 EWS 2022-09-26 02:16:44 PDT 
Committed 254852@main (cf0cfbaf1ad7): <https://commits.webkit.org/254852@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 462598 [details].