[heap] Added fine grained timers to MC_MARK_WEAK_CLOSURE.

[heap] Added fine grained timers to MC_MARK_WEAK_CLOSURE.

BUG=

Committed: https://crrev.com/3b1578db0c96a686d9db3bd189afd299ccc3e9a3
Cr-Commit-Position: refs/heads/master@{#35107}

[Hannes Payer (out of office), 2016-03-29 12:24:52]

hpayer@chromium.org changed reviewers:
+ ulan@chromium.org

[Michael Lippautz, 2016-03-29 12:25:41]

mlippautz@chromium.org changed reviewers:
+ mlippautz@chromium.org

[Michael Lippautz, 2016-03-29 12:25:42]

dbc: Don't forget the script in tools/ :)

[ulan, 2016-03-29 12:43:58]

lgtm with a comment.

https://codereview.chromium.org/1836013004/diff/40001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1836013004/diff/40001/src/heap/mark-compact.c...
src/heap/mark-compact.cc:2183:
GCTracer::Scope::MC_MARK_WEAK_CLOSURE_EPHEMERAL_HARMONY);
Let's name it MC_MARK_WEAK_CLOSURE_HARMONY or MC_MARK_WEAK_CLOSURE_EPHEMERAL2.
Otherwise, its name suggests that the scope is nested in
MC_MARK_WEAK_CLOSURE_EPHEMERAL.

[Hannes Payer (out of office), 2016-03-29 12:52:21]

https://codereview.chromium.org/1836013004/diff/40001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1836013004/diff/40001/src/heap/mark-compact.c...
src/heap/mark-compact.cc:2183:
GCTracer::Scope::MC_MARK_WEAK_CLOSURE_EPHEMERAL_HARMONY);
On 2016/03/29 12:43:57, ulan wrote:
> Let's name it MC_MARK_WEAK_CLOSURE_HARMONY or MC_MARK_WEAK_CLOSURE_EPHEMERAL2.
> Otherwise, its name suggests that the scope is nested in
> MC_MARK_WEAK_CLOSURE_EPHEMERAL.

Done.

[Hannes Payer (out of office), 2016-03-29 12:52:37]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2016-03-29 12:52:37]

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

[commit-bot: I haz the power, 2016-03-29 12:52:39]

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

[commit-bot: I haz the power, 2016-03-29 14:51:40]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-29 14:52:29]

Description was changed from

==========
[heap] Added fine grained timers to MC_MARK_WEAK_CLOSURE.

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

to

==========
[heap] Added fine grained timers to MC_MARK_WEAK_CLOSURE.

BUG=

Committed: https://crrev.com/3b1578db0c96a686d9db3bd189afd299ccc3e9a3
Cr-Commit-Position: refs/heads/master@{#35107}
==========

[commit-bot: I haz the power, 2016-03-29 14:52:30]

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