Deprecate NodeIterator.expandEntityReferences / TreeWalker.expandEntityReferences

Deprecate NodeIterator.expandEntityReferences / TreeWalker.expandEntityReferences

Deprecate NodeIterator.expandEntityReferences / TreeWalker.expandEntityReferences
as those were removed from the DOM specification:
http://dom.spec.whatwg.org/#dom-traversal

Those properties were never implemented in Chromium and always return false. We already
dropped the corresponding argument to createNodeIterator() / createTreeWalker().

Firefox has already removed them since version 21:
https://developer.mozilla.org/en-US/docs/Web/API/NodeIterator.expandEntityReferences
https://developer.mozilla.org/en-US/docs/Web/API/TreeWalker.expandEntityReferences

However, since these properties were standard and are still present in IE11, I
propose to deprecate them and measure their usage to make sure it is safe to
eventually remove.

R=arv@chromium.org, tkent@chromium.org
BUG=367866
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172913

[arv (Not doing code reviews), 2014-04-28 18:45:45]

LGTM

[Inactive, 2014-04-29 13:12:55]

Could an API OWNER please review?

[eseidel, 2014-04-29 17:00:02]

lgtm

https://www.youtube.com/watch?v=-dJolYw8tnk

[commit-bot: I haz the power, 2014-04-29 17:00:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/251183002/1

[commit-bot: I haz the power, 2014-04-29 17:00:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-29 17:00:40]

Failed to apply patch for Source/core/frame/UseCounter.h:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/frame/UseCounter.h
  Hunk #1 FAILED at 364.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/frame/UseCounter.h.rej

Patch:       Source/core/frame/UseCounter.h
Index: Source/core/frame/UseCounter.h
diff --git a/Source/core/frame/UseCounter.h b/Source/core/frame/UseCounter.h
index
d91a110ecca357a2fa49471ffbba0bbc78a1c485..c43d7c8f89e42d46aaa9c7717523b9f9e5b35001
100644
--- a/Source/core/frame/UseCounter.h
+++ b/Source/core/frame/UseCounter.h
@@ -364,6 +364,8 @@ public:
         EventPath = 345,
         EventClipboardData = 346,
         NodeIteratorDetach = 347,
+        NodeIteratorExpandEntityReferences = 348,
+        TreeWalkerExpandEntityReferences = 349,
         // Add new features immediately above this line. Don't change assigned
         // numbers of any item, and don't reuse removed slots.
         NumberOfFeatures, // This enum value must be last.

[Inactive, 2014-04-29 17:05:02]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-04-29 17:06:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/251183002/20001

[commit-bot: I haz the power, 2014-04-30 04:13:47]

Change committed as 172913