Rename Node::treeScope() to Node::treeScopeOrDocument()

Rename Node::treeScope() to Node::treeScopeOrDocument()

It is confusing that Node::treeScope() returns the Document when the node is not in a document and its root is not a TreeScope.

See the bug for details.

BUG=602556

[hayato, 2016-04-12 07:20:28]

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

[commit-bot: I haz the power, 2016-04-12 07:20:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885453002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885453002/1

[hayato, 2016-04-12 07:25:04]

Description was changed from

==========
Rename

BUG=
==========

to

==========
Rename Node::treeScope() to Node::treeScopeOrDocument()

It's confusing that Node::treeScope() returns the Document when the node is not
in a document and its root is not a TreeScope.

See the bug for details.

BUG=602556
==========

[hayato, 2016-04-12 07:25:19]

Description was changed from

==========
Rename Node::treeScope() to Node::treeScopeOrDocument()

It's confusing that Node::treeScope() returns the Document when the node is not
in a document and its root is not a TreeScope.

See the bug for details.

BUG=602556
==========

to

==========
Rename Node::treeScope() to Node::treeScopeOrDocument()

It is confusing that Node::treeScope() returns the Document when the node is not
in a document and its root is not a TreeScope.

See the bug for details.

BUG=602556
==========

[hayato, 2016-04-12 07:30:14]

hayato@chromium.org changed reviewers:
+ tkent@chromium.org

[hayato, 2016-04-12 07:30:16]

PTAL

[tkent, 2016-04-12 07:34:01]

lgtm

[commit-bot: I haz the power, 2016-04-12 07:37:28]

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

[commit-bot: I haz the power, 2016-04-12 07:37:28]

Dry run: Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[hayato, 2016-04-12 08:10:02]

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

[hayato, 2016-04-12 08:10:07]

rebased

[hayato, 2016-04-12 08:10:20]

The CQ bit was unchecked by hayato@chromium.org

[hayato, 2016-04-12 08:10:26]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-04-12 08:10:27]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1885453002/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2016-04-12 08:11:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885453002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885453002/20001

[hayato, 2016-04-12 08:48:00]

The CQ bit was unchecked by hayato@chromium.org

[hayato, 2016-04-12 10:04:36]

The discussion is on-going in the bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=602556

I will not land the patch soon.