Rename duplicate symbols inside core/dom

Rename duplicate symbols inside core/dom

While experimenting with unity builds I got into problems with
code that used the same symbol name for multiple purposes. This
renames those which also will make the life for symbol debugging
slightly easier.

kSupplementName: Name used in both CSSSelectorWatch.cpp and
DocumentParserTiming.cpp.

WTF::Unicode symbols: Depending on how unity builds are created,
global "using" statements can affect more than intended and trigger
warnings and errors. Now global "using" statements are not allowed by
the Chromium coding standard so assuming it will eventually apply to
Blink as well, the using statements might as well be removed now.

previousAncestorSiblingPostOrder was a function in both
FlatTreeTraversal.cpp and NodeTraversal.cpp. Could easily be made
class local and the name collision was no more.

Review-Url: https://codereview.chromium.org/2802723002
Cr-Commit-Position: refs/heads/master@{#462123}
Committed: https://chromium.googlesource.com/chromium/src/+/805ba25019edb994b874d506cff773134d06a970

[fs, 2017-04-05 15:10:09]

fs@opera.com changed reviewers:
+ fs@opera.com

[fs, 2017-04-05 15:10:09]

lgtm

[Daniel Bratell, 2017-04-05 15:16:44]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2017-04-05 15:17:22]

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, 2017-04-05 15:31:18]

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

[commit-bot: I haz the power, 2017-04-05 15:31:19]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Daniel Bratell, 2017-04-05 16:03:49]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2017-04-05 16:04:25]

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, 2017-04-05 17:05:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491408229259430, "parent_rev":
"890cfea7950ebc5910372deb91ebcbf665362817", "commit_rev":
"805ba25019edb994b874d506cff773134d06a970"}

[commit-bot: I haz the power, 2017-04-05 17:06:12]

Description was changed from

==========
Rename duplicate symbols inside core/dom

While experimenting with unity builds I got into problems with
code that used the same symbol name for multiple purposes. This
renames those which also will make the life for symbol debugging
slightly easier.

kSupplementName: Name used in both CSSSelectorWatch.cpp and
DocumentParserTiming.cpp.

WTF::Unicode symbols: Depending on how unity builds are created,
global "using" statements can affect more than intended and trigger
warnings and errors. Now global "using" statements are not allowed by
the Chromium coding standard so assuming it will eventually apply to
Blink as well, the using statements might as well be removed now.

previousAncestorSiblingPostOrder was a function in both
FlatTreeTraversal.cpp and NodeTraversal.cpp. Could easily be made
class local and the name collision was no more.
==========

to

==========
Rename duplicate symbols inside core/dom

While experimenting with unity builds I got into problems with
code that used the same symbol name for multiple purposes. This
renames those which also will make the life for symbol debugging
slightly easier.

kSupplementName: Name used in both CSSSelectorWatch.cpp and
DocumentParserTiming.cpp.

WTF::Unicode symbols: Depending on how unity builds are created,
global "using" statements can affect more than intended and trigger
warnings and errors. Now global "using" statements are not allowed by
the Chromium coding standard so assuming it will eventually apply to
Blink as well, the using statements might as well be removed now.

previousAncestorSiblingPostOrder was a function in both
FlatTreeTraversal.cpp and NodeTraversal.cpp. Could easily be made
class local and the name collision was no more.

Review-Url: https://codereview.chromium.org/2802723002
Cr-Commit-Position: refs/heads/master@{#462123}
Committed:
https://chromium.googlesource.com/chromium/src/+/805ba25019edb994b874d506cff7...
==========

[commit-bot: I haz the power, 2017-04-05 17:06:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/805ba25019edb994b874d506cff7...