[gc] Add runtime timer for gc prologue and epilogue callbacks

[gc] Add runtime timer for gc prologue and epilogue callbacks

BUG=

Committed: https://crrev.com/c3e0ef8b003e43c9fcb1d21d01f505dcb46bc495
Cr-Commit-Position: refs/heads/master@{#41233}

[Camillo Bruni, 2016-11-23 09:16:17]

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

[Camillo Bruni, 2016-11-23 09:16:17]

PTAL

[Camillo Bruni, 2016-11-23 10:12:48]

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

[commit-bot: I haz the power, 2016-11-23 10:12:57]

Dry run: 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, 2016-11-23 10:41:11]

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

[commit-bot: I haz the power, 2016-11-23 10:41:11]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-11-23 12:23:59]

lgtm

[Hannes Payer (out of office), 2016-11-23 16:53:04]

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

[Hannes Payer (out of office), 2016-11-23 16:53:05]

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

https://codereview.chromium.org/2524733007/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:858: RuntimeCallTimerScope(isolate(),
&RuntimeCallStats::GC_AllAvailableGarbage);
Not sure what your naming scheme is but shouldn't this be called
GCCollectAllAvailableGarbage?

[Hannes Payer (out of office), 2016-11-23 16:58:21]

LGTM btw

[Camillo Bruni, 2016-11-23 17:27:16]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-11-23 17:27:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlippautz@chromium.org
Link to the patchset: https://codereview.chromium.org/2524733007/#ps20001
(title: "adding more gc counters")

[commit-bot: I haz the power, 2016-11-23 17:27:26]

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, 2016-11-23 18:00:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479922036261020,
"parent_rev": "6492d558472dd5128671a086d6b33ee8c618c2f5", "commit_rev":
"390adc09c05f09ee1f11ad42785e162b3b086386"}

[commit-bot: I haz the power, 2016-11-23 18:00:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-23 18:00:25]

Description was changed from

==========
[gc] Add runtime timer for gc prologue and epilogue callbacks

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

to

==========
[gc] Add runtime timer for gc prologue and epilogue callbacks

BUG=

Committed: https://crrev.com/c3e0ef8b003e43c9fcb1d21d01f505dcb46bc495
Cr-Commit-Position: refs/heads/master@{#41233}
==========

[commit-bot: I haz the power, 2016-11-23 18:00:26]

Patchset 2 (id:??) landed as
https://crrev.com/c3e0ef8b003e43c9fcb1d21d01f505dcb46bc495
Cr-Commit-Position: refs/heads/master@{#41233}