Remove Node.rootNode

Remove Node.rootNode

Before shipping Shadow DOM v1, we have to remove this feature because Node.rootNode
is guarded by ShadowDOMV1 flag.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=608006.
Ship of Node.rootNode was reverted there.

BUG=531990
Committed: https://crrev.com/5401b41c540d2ace9e9d773ece97a615729b6557
Cr-Commit-Position: refs/heads/master@{#402697}

[hayato, 2016-06-28 04:22:33]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-28 04:22:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hayato, 2016-06-28 04:23:27]

Description was changed from

==========
Remove Node.rootNode

Before shipping Shadow DOM v1, we have to remove this feature because
Node.rootNode
is guarded by ShadowDOMV1 flag.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=608006.
Ship of Node.rootNode was revereted.

BUG=531990
==========

to

==========
Remove Node.rootNode

Before shipping Shadow DOM v1, we have to remove this feature because
Node.rootNode
is guarded by ShadowDOMV1 flag.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=608006.
Ship of Node.rootNode was reverted there.

BUG=531990
==========

[hayato, 2016-06-28 04:30:56]

hayato@chromium.org changed reviewers:
+ kochi@chromium.org, rkaplow@chromium.org

[hayato, 2016-06-28 04:30:57]

PTAL.

Adding rkaplow@ to a reviewer for histgrams.xml.

[kochi, 2016-06-28 04:39:47]

What's the plan for replacing Node.rootNode with
Node.getRootNode()?

https://github.com/whatwg/dom/issues/241

[hayato, 2016-06-28 04:44:27]

That should be done later.

[kochi, 2016-06-28 04:58:24]

On 2016/06/28 04:44:27, hayato wrote:
> That should be done later.

I see, because it needs another intent to ship statement?

lgtm for this CL, then.

[commit-bot: I haz the power, 2016-06-28 05:55:19]

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

[commit-bot: I haz the power, 2016-06-28 05:55:20]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[hayato, 2016-06-28 06:03:02]

Let me fix falling layout tests.

[hayato, 2016-06-28 06:08:31]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[hayato, 2016-06-28 06:08:38]

Rebase failing tests

[commit-bot: I haz the power, 2016-06-28 06:08:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 08:35:04]

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

[commit-bot: I haz the power, 2016-06-28 08:35:05]

Dry run: This issue passed the CQ dry run.

[rkaplow, 2016-06-28 14:40:33]

lgtm

https://codereview.chromium.org/2106683002/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2106683002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:75541: -  <int value="1154"
label="NodeRootNode"/>
prefer leaving the line here, and rewriting label as NodeRootNode (obsolete) -
sicne we may still get old data tracking this.

[hayato, 2016-06-29 01:35:03]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[hayato, 2016-06-29 01:35:05]

leave the line

[hayato, 2016-06-29 01:35:58]

https://codereview.chromium.org/2106683002/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2106683002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:75541: -  <int value="1154"
label="NodeRootNode"/>
Thanks! Done.

[hayato, 2016-06-29 01:36:05]

The CQ bit was unchecked by hayato@chromium.org

[hayato, 2016-06-29 01:36:08]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-06-29 01:36:08]

The patchset sent to the CQ was uploaded after l-g-t-m from kochi@chromium.org,
rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2106683002/#ps40001
(title: "leave the line")

[commit-bot: I haz the power, 2016-06-29 01:36:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 03:07:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-29 03:11:01]

Description was changed from

==========
Remove Node.rootNode

Before shipping Shadow DOM v1, we have to remove this feature because
Node.rootNode
is guarded by ShadowDOMV1 flag.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=608006.
Ship of Node.rootNode was reverted there.

BUG=531990
==========

to

==========
Remove Node.rootNode

Before shipping Shadow DOM v1, we have to remove this feature because
Node.rootNode
is guarded by ShadowDOMV1 flag.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=608006.
Ship of Node.rootNode was reverted there.

BUG=531990

Committed: https://crrev.com/5401b41c540d2ace9e9d773ece97a615729b6557
Cr-Commit-Position: refs/heads/master@{#402697}
==========

[commit-bot: I haz the power, 2016-06-29 03:11:03]

Patchset 3 (id:??) landed as
https://crrev.com/5401b41c540d2ace9e9d773ece97a615729b6557
Cr-Commit-Position: refs/heads/master@{#402697}