Merge VisitorScope to Visitor

Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior change.

BUG=None.

Committed: https://crrev.com/7d65084979b7176f0dfaf5f8da16d0a80d9a7f35
Cr-Commit-Position: refs/heads/master@{#378718}

[keishi, 2016-03-01 05:55:30]

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

[commit-bot: I haz the power, 2016-03-01 05:55:51]

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

[commit-bot: I haz the power, 2016-03-01 06:08:31]

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

[commit-bot: I haz the power, 2016-03-01 06:08:32]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[keishi, 2016-03-01 06:34:40]

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

[commit-bot: I haz the power, 2016-03-01 06:34:46]

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

[commit-bot: I haz the power, 2016-03-01 07:42:56]

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

[commit-bot: I haz the power, 2016-03-01 07:42:57]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[keishi, 2016-03-01 07:57:54]

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

[commit-bot: I haz the power, 2016-03-01 07:58:07]

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

[commit-bot: I haz the power, 2016-03-01 11:34:05]

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

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

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[keishi, 2016-03-02 07:05:17]

Description was changed from

==========
Merge VisitorScope to Visitor

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

to

==========
Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior
change.

BUG=None.
==========

[keishi, 2016-03-02 07:05:35]

keishi@chromium.org changed reviewers:
+ haraken@chromium.org, oilpan-reviews@chromium.org

[keishi, 2016-03-02 07:05:36]

PTAL

[haraken, 2016-03-02 07:13:24]

LGTM

https://codereview.chromium.org/1753623002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/heap/MarkingVisitor.h (right):

https://codereview.chromium.org/1753623002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/heap/MarkingVisitor.h:18:
MarkingVisitor(ThreadState* state)

Add explicit.

[keishi, 2016-03-02 07:27:11]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-03-02 07:27:12]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1753623002/#ps80001
(title: "added explicit")

[commit-bot: I haz the power, 2016-03-02 07:27:24]

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

[commit-bot: I haz the power, 2016-03-02 10:38:30]

Description was changed from

==========
Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior
change.

BUG=None.
==========

to

==========
Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior
change.

BUG=None.
==========

[commit-bot: I haz the power, 2016-03-02 10:38:32]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-02 10:39:44]

Description was changed from

==========
Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior
change.

BUG=None.
==========

to

==========
Merge VisitorScope to Visitor

Merging because VisitorScope and Visitor have the same lifetime. No behavior
change.

BUG=None.

Committed: https://crrev.com/7d65084979b7176f0dfaf5f8da16d0a80d9a7f35
Cr-Commit-Position: refs/heads/master@{#378718}
==========

[commit-bot: I haz the power, 2016-03-02 10:39:45]

Patchset 5 (id:??) landed as
https://crrev.com/7d65084979b7176f0dfaf5f8da16d0a80d9a7f35
Cr-Commit-Position: refs/heads/master@{#378718}