Be explicit about namespace testing to not mix it with blink::testing

Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2970833002
Cr-Commit-Position: refs/heads/master@{#484087}
Committed: https://chromium.googlesource.com/chromium/src/+/51287d5e74cd270ce91e432d072e856718b677e3

[Daniel Bratell, 2017-07-04 09:55:31]

Description was changed from

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
==========

to

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Daniel Bratell, 2017-07-04 09:55:32]

Description was changed from

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Daniel Bratell, 2017-07-04 09:57:01]

The CQ bit was checked by bratell@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-07-04 09:57:13]

Dry run: CQ is trying da patch.

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

[Daniel Bratell, 2017-07-04 09:57:47]

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

[Daniel Bratell, 2017-07-04 09:57:48]

fs, PTAL.

I think that if it compiles, it's right.

[fs, 2017-07-04 10:24:35]

lgtm

[commit-bot: I haz the power, 2017-07-04 11:18:30]

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

[commit-bot: I haz the power, 2017-07-04 11:18:31]

Dry run: This issue passed the CQ dry run.

[Daniel Bratell, 2017-07-04 11:29:49]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2017-07-04 11:30:01]

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-07-04 11:36:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499167789169220, "parent_rev":
"4ff2d41025006a482ea0d5fb40fb4aa3a585e3d3", "commit_rev":
"51287d5e74cd270ce91e432d072e856718b677e3"}

[commit-bot: I haz the power, 2017-07-04 11:36:20]

Description was changed from

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Be explicit about namespace testing to not mix it with blink::testing

There are two namespaces named testing used in Blink unit tests, one
from gtest and one internal helper namespace. If both are visible then
the gtest testing namespace won't be used unless prepended with ::, as
is already done in large parts of the code.

BUG=739121
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2970833002
Cr-Commit-Position: refs/heads/master@{#484087}
Committed:
https://chromium.googlesource.com/chromium/src/+/51287d5e74cd270ce91e432d072e...
==========

[commit-bot: I haz the power, 2017-07-04 11:36:22]

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