Intent to Remove: ShadowRoot.getElementsBy*()

74 views
Skip to first unread message

Hayato Ito

unread,
Apr 6, 2015, 4:56:04 AM4/6/15
to blink-dev

Primary eng (and PM) emails

hay...@chromium.org


Link to “Intent to Deprecate” thread

https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/YHObTvjl1tU


Summary

Remove getElementsByClassName(), getElementsByTagName() and getElementsByTagNameNS() on ShadowRoot.


Note: getElementById() is not involved in any way.

(copied-and-pasted from "Intent to Deprecated" thread)

Motivation

https://www.w3.org/Bugs/Public/show_bug.cgi?id=23620 Follow the link for more context. In summary, this was agreed upon a year ago.

(copied-and-pasted from "Intent to Deprecated" thread)


Usage information from UseCounter

ShadowRootGetElementsByClassName: <=0.0001% https://www.chromestatus.com/metrics/feature/timeline/popularity/623 ShadowRootGetElementsByTagName: 0.0039% https://www.chromestatus.com/metrics/feature/timeline/popularity/624

ShadowRootGetElementsByTagNameNS: <=0.0001% https://www.chromestatus.com/metrics/feature/timeline/popularity/625


FYI.
As a basic for comparison, Element.createShadowRoot is: 2.6288%

Entry on chromestatus.com

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


Compatibility Risk

Any content using these methods will throw exceptions.



TAMURA, Kent

unread,
Apr 7, 2015, 3:19:46 AM4/7/15
to Hayato Ito, blink-dev
LGTM to remove.

--
TAMURA Kent
Software Engineer, Google


Dimitri Glazkov

unread,
Apr 7, 2015, 10:56:46 AM4/7/15
to TAMURA, Kent, Hayato Ito, blink-dev
LGTM2

Chris Harrelson

unread,
Apr 7, 2015, 11:54:56 AM4/7/15
to Dimitri Glazkov, TAMURA, Kent, Hayato Ito, blink-dev
LGTM3

LGTM2
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.

p...@google.com

unread,
Apr 7, 2015, 1:02:05 PM4/7/15
to blin...@chromium.org, dgla...@chromium.org, tk...@chromium.org, hay...@chromium.org, chri...@chromium.org
LGTM, better to remove this early before certain people start using this API.
Reply all
Reply to author
Forward
0 new messages