[counters] adding runtime call timers for GC

[counters] adding runtime call timers for GC

Adding an additional timer for the GC to make the other runtime call counters
more stable.

BUG=

Committed: https://crrev.com/9ec649638d553a891b0356155f4f15f3a260c1af
Cr-Commit-Position: refs/heads/master@{#35033}

[Camillo Bruni, 2016-03-22 16:00:56]

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

[Camillo Bruni, 2016-03-22 16:00:56]

PTAL

[Camillo Bruni, 2016-03-23 13:54:02]

PTAL again

https://codereview.chromium.org/1825093002/diff/20001/src/heap/gc-tracer.h
File src/heap/gc-tracer.h (right):

https://codereview.chromium.org/1825093002/diff/20001/src/heap/gc-tracer.h#ne...
src/heap/gc-tracer.h:155: RuntimeCallTimer timer_;
RuntimeCallTimer is a 3 word object.

[ulan, 2016-03-23 13:59:56]

https://codereview.chromium.org/1825093002/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1825093002/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:1012: RuntimeCallTimerScope timer(
Please move this to GcTracer::Start / Stop similar to the scope and guard by the
runtime flag.

[Camillo Bruni, 2016-03-23 14:30:01]

PTAL again

[ulan, 2016-03-23 14:35:20]

lgtm

[Camillo Bruni, 2016-03-23 14:39:40]

The CQ bit was checked by cbruni@chromium.org

[commit-bot: I haz the power, 2016-03-23 14:39:43]

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

[commit-bot: I haz the power, 2016-03-23 15:04:01]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-23 15:04:29]

Description was changed from

==========
[counters] adding runtime call timers for GC

Adding an additional timer for the GC to make the other runtime call counters
more stable.

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

to

==========
[counters] adding runtime call timers for GC

Adding an additional timer for the GC to make the other runtime call counters
more stable.

BUG=

Committed: https://crrev.com/9ec649638d553a891b0356155f4f15f3a260c1af
Cr-Commit-Position: refs/heads/master@{#35033}
==========

[commit-bot: I haz the power, 2016-03-23 15:04:30]

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

[Michael Achenbach, 2016-03-23 15:37:28]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1827563003/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] ASAN bot is really badass today:
https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20ASAN/builds/10844.