Intent to Remove: SVGZoomEvent

閲覧: 52 回
最初の未読メッセージにスキップ

Shanmuga

未読、
2016/07/12 6:06:132016/07/12
To: blink-dev

Primary eng (and PM) emails

da...@opera.com, shanm...@samsung.com



Link to “Intent to Deprecate” thread

https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/5PEUh2qHa7c/IUGlM4a9CQAJ

 

Summary

Remove the SVGZoomEvent interface and the corresponding onzoom attribute. 



Motivation

SVGZoomEvent doesn't work at all in Blink. It can be created and hooked up using onzoom="..." but it is never fired by the browser.

 

It has been removed from the SVG 2.0 specification:

 

https://github.com/w3c/svgwg/issues/21

https://github.com/w3c/svgwg/commit/3844475c05b865e961a0bc52cd63e627412360e2



Compatibility Risk

A httparchive search has been conducted:

https://code.google.com/p/chromium/issues/detail?id=367890#c11

indicating very low usage. From the search it looks likely that the event is possible to remove.

The interface has been around in WebKit and Blink for a long time. Originates in:

https://bugs.webkit.org/show_bug.cgi?id=10826

 

Usage information from UseCounter

https://www.chromestatus.com/metrics/feature/timeline/popularity/1232

 

OWP launch tracking bug

No OWP launch tracking bug but there is:

https://code.google.com/p/chromium/issues/detail?id=367890



Entry on the feature dashboard

https://www.chromestatus.com/feature/5760883808534528



Requesting approval to remove too?

Yes, M52 is the suggested milestone.

Philip Jägenstedt

未読、
2016/07/12 6:33:412016/07/12
To: Shanmuga,blink-dev
LGTM1

Can you update https://www.chromestatus.com/feature/5760883808534528 with the actual removal milestone, which will be M54?

(Given the extremely low usage, I don't think it's worthwhile in this case to update the deprecation message for a release cycle.)

Shanmuga

未読、
2016/07/12 6:37:532016/07/12
To: blink-dev,shanm...@samsung.com
Updated the milestone to M54

Shanmuga

未読、
2016/07/12 9:19:522016/07/12
To: blink-dev,shanm...@samsung.com

Rick Byers

未読、
2016/07/12 9:24:352016/07/12
To: Shanmuga,blink-dev

LGTM2

Philip Jägenstedt

未読、
2016/07/13 5:16:462016/07/13
To: Rick Byers,Shanmuga,blink-dev
I didn't mention waiting for 3xLGTM when I reviewed https://codereview.chromium.org/2133983003/, so it has already landed. Since this intent seems 99% likely to pass I suggested leaving it in, but a 3rd LGTM is still needed here. Oops :)

Jochen Eisinger

未読、
2016/07/13 5:22:202016/07/13
To: Philip Jägenstedt,Rick Byers,Shanmuga,blink-dev
lgtm3
全員に返信
投稿者に返信
転送
新着メール 0 件