Intent to Remove: NodeIterator.expandEntityReferences and TreeWalker.expandEntityReferences

unread,

Nov 20, 2014, 7:37:28 PM11/20/14

to blink-dev

Primary eng (and PM) emails

Link to “Intent to Deprecate” thread

There was none. This was deprecated by Chris Dumez (no longer working in Blink) in https://crbug.com/367866 which reached stable with M36 in July.

Summary

Remove NodeIterator.expandEntityReferences and TreeWalker.expandEntityReferences.

Motivation

They always return false and have been removed from the DOM Standard:

It's already been removed in Gecko.

Usage information from UseCounter

To date, it hasn't reached the 0.0001% that shows up in these graphs. It looks like many days the counters haven't been hit at all.

None.

Compatibility Risk

Virtually zero, given the false->undefined change and ~0% usage.

unread,

Nov 20, 2014, 7:39:42 PM11/20/14

to Philip Jägenstedt, blink-dev

LGTM

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

unread,

Nov 20, 2014, 7:40:59 PM11/20/14

to Chris Harrelson, Philip Jägenstedt, blink-dev

LGTM

unread,

Nov 20, 2014, 7:48:39 PM11/20/14

to Philip Rogers, Chris Harrelson, Philip Jägenstedt, blink-dev

I can see the internal numbers behind the chromestatus.com graphs, and yeah, the suspicion of not being hit is pretty accurate.

(non-owner) LGTM

unread,

Nov 20, 2014, 8:44:46 PM11/20/14

to Mike Lawther, Philip Rogers, Chris Harrelson, Philip Jägenstedt, blink-dev

LGTM3.

--

TAMURA Kent
Software Engineer, Google

Reply all

Reply to author

Forward