Added zone pool metrics to gc-trace.

Added zone pool metrics to gc-trace.

Can be displayed in the external tool as well.

BUG=v8:5489
Committed: https://crrev.com/91a3517e24624776bd894b047190e600d18363be
Cr-Commit-Position: refs/heads/master@{#40166}

[heimbuef, 2016-10-06 12:18:13]

heimbuef@google.com changed reviewers:
+ mlippautz@chromium.org

[heimbuef, 2016-10-06 12:18:13]

PTAL. Please also see the matching pull request on github:
https://github.com/mlippautz/v8-heap-stats/pull/1

[Michael Lippautz, 2016-10-06 12:41:18]

LGTM once the comment is addressed 

If you plan more changes you should probably open a tracking bug and reference
that from all of your related CLs.

https://codereview.chromium.org/2392183004/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/2392183004/diff/1/src/isolate.cc#newcode1936
src/isolate.cc:1936: last_memory_usage_(0),
Initialize last_pool_size_.

[heimbuef, 2016-10-06 12:44:48]

On 2016/10/06 at 12:41:18, mlippautz wrote:
> LGTM once the comment is addressed 
> 
> If you plan more changes you should probably open a tracking bug and reference
that from all of your related CLs.
> 
> https://codereview.chromium.org/2392183004/diff/1/src/isolate.cc
> File src/isolate.cc (right):
> 
> https://codereview.chromium.org/2392183004/diff/1/src/isolate.cc#newcode1936
> src/isolate.cc:1936: last_memory_usage_(0),
> Initialize last_pool_size_.

Done.

[heimbuef, 2016-10-06 17:29:47]

Description was changed from

==========
Added zone pool metrics to gc-trace.

Can be displayed in the external tool as well.

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

to

==========
Added zone pool metrics to gc-trace.

Can be displayed in the external tool as well.

BUG=v8:5489
==========

[heimbuef, 2016-10-11 11:19:59]

The CQ bit was checked by heimbuef@google.com

[heimbuef, 2016-10-11 11:19:59]

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/2392183004/#ps40001
(title: "Added recent master changes")

[commit-bot: I haz the power, 2016-10-11 11:20:08]

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-10-11 11:49:35]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-11 11:49:56]

Description was changed from

==========
Added zone pool metrics to gc-trace.

Can be displayed in the external tool as well.

BUG=v8:5489
==========

to

==========
Added zone pool metrics to gc-trace.

Can be displayed in the external tool as well.

BUG=v8:5489

Committed: https://crrev.com/91a3517e24624776bd894b047190e600d18363be
Cr-Commit-Position: refs/heads/master@{#40166}
==========

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

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