Intent to Deprecate and Remove: SVGFEMorphologyElement.setRadius(radiusX, radiusY)

27 views
Skip to first unread message

Philip Jägenstedt

unread,
Jul 29, 2014, 4:53:53 PM7/29/14
to blink-dev

Primary eng (and PM) emails

phi...@opera.com


Summary

Remove SVGFEMorphologyElement.setRadius(radiusX, radiusY).


Motivation

It is non-standard and not supported in any non-WebKit/Blink browser:

http://jsconsole.com/?document.createElementNS('http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg'%2C%20'feMorphology').setRadius

It came with the initial implementation of feMorphology in WebKit:

Compatibility Risk

Zero usage means zero risk.


Alternative implementation suggestion for web developers

Set elm.radiusX/Y.baseVal instead.


Usage information from UseCounter

http://www.chromestatus.com/metrics/feature/timeline/popularity/408


The counter is in M36 and hasn't registered at all, which I've never seen before. I've checked the generated V8SVGFEMorphologyElement.cpp and verified that the counter is actually there.

Entry on chromestatus.com, crbug.com, or MDN

None.


Requesting approval to remove too?

Yes.

Dimitri Glazkov

unread,
Jul 29, 2014, 4:56:28 PM7/29/14
to Philip Jägenstedt, blink-dev
LGTM.

Philip Rogers

unread,
Jul 29, 2014, 5:00:26 PM7/29/14
to Dimitri Glazkov, Philip Jägenstedt, blink-dev
LGTM.

Eric Seidel

unread,
Jul 29, 2014, 7:43:42 PM7/29/14
to Philip Rogers, Dimitri Glazkov, Philip Jägenstedt, blink-dev
lgtm

TAMURA, Kent

unread,
Jul 29, 2014, 8:15:35 PM7/29/14
to Eric Seidel, Philip Rogers, Dimitri Glazkov, Philip Jägenstedt, blink-dev
LGTM3.  Approved.

--
TAMURA Kent
Software Engineer, Google


Reply all
Reply to author
Forward
0 new messages