Named all zones in the project

Named all zones in the project

This adds more useful information to the v8-heap-stats tool.

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

[heimbuef, 2016-10-07 09:21:22]

Description was changed from

==========
Named all zones in the project

This adds more useful information to the v8-heap-stats tool.

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

to

==========
Named all zones in the project

This adds more useful information to the v8-heap-stats tool.

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

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

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

[heimbuef, 2016-10-11 11:25:00]

PTAL; we might need more owners as reviewers or circumvent the checks.

[Toon Verwaest, 2016-10-11 12:13:05]

lgtm

[Yang, 2016-10-11 13:00:52]

On 2016/10/11 12:13:05, Toon Verwaest wrote:
> lgtm

lgtm.

[heimbuef, 2016-10-11 13:29:16]

The CQ bit was checked by heimbuef@google.com

[commit-bot: I haz the power, 2016-10-11 13:29:22]

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 13:33:02]

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

[commit-bot: I haz the power, 2016-10-11 13:33:03]

Try jobs failed on following builders:
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/14324)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/10536)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26148)

[heimbuef, 2016-10-11 14:57:22]

The CQ bit was checked by heimbuef@google.com

[heimbuef, 2016-10-11 14:57:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org, yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2394213003/#ps40001
(title: "Added missing zone name")

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

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 15:05:41]

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

[commit-bot: I haz the power, 2016-10-11 15:05:42]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26161)

[heimbuef, 2016-10-12 11:13:37]

heimbuef@google.com changed reviewers:
+ bmeurer@chromium.org, marja@chromium.org, titzer@chromium.org,
yangguo@chromium.org

[titzer, 2016-10-12 11:22:18]

On 2016/10/11 15:05:42, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   v8_presubmit on master.tryserver.v8 (JOB_FAILED,
> http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26161)

uggh, this adds hundreds, if not thousands, of new strings to the v8 binary.

[heimbuef, 2016-10-12 12:20:25]

On 2016/10/12 at 11:22:18, titzer wrote:
> On 2016/10/11 15:05:42, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   v8_presubmit on master.tryserver.v8 (JOB_FAILED,
> > http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26161)
> 
> uggh, this adds hundreds, if not thousands, of new strings to the v8 binary.

It's ~300.

[titzer, 2016-10-12 12:41:59]

On 2016/10/12 12:20:25, heimbuef wrote:
> On 2016/10/12 at 11:22:18, titzer wrote:
> > On 2016/10/11 15:05:42, commit-bot: I haz the power wrote:
> > > Try jobs failed on following builders:
> > >   v8_presubmit on master.tryserver.v8 (JOB_FAILED,
> > >
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26161)
> > 
> > uggh, this adds hundreds, if not thousands, of new strings to the v8 binary.
> 
> It's ~300.

Since this turned out to be on 5kb, LGTM

[marja, 2016-10-12 12:52:27]

lgtm

[heimbuef, 2016-10-12 14:20:01]

The CQ bit was checked by heimbuef@google.com

[commit-bot: I haz the power, 2016-10-12 14:20:05]

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-12 14:25:13]

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

[commit-bot: I haz the power, 2016-10-12 14:25:14]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/26292)

[Benedikt Meurer, 2016-10-12 17:29:48]

LGTM (rubber-stamped)

[heimbuef, 2016-10-14 10:40:11]

The CQ bit was checked by heimbuef@google.com

[commit-bot: I haz the power, 2016-10-14 10:40:21]

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-14 10:42:10]

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

[commit-bot: I haz the power, 2016-10-14 10:42:11]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/10839)

[heimbuef, 2016-10-17 11:43:07]

The CQ bit was checked by heimbuef@google.com

[heimbuef, 2016-10-17 11:43:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, titzer@chromium.org, verwaest@chromium.org,
yangguo@chromium.org, marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2394213003/#ps60001
(title: "Merge branch 'master' into zonenames")

[commit-bot: I haz the power, 2016-10-17 11:43:10]

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-17 12:12:40]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-17 12:12:51]

Description was changed from

==========
Named all zones in the project

This adds more useful information to the v8-heap-stats tool.

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

to

==========
Named all zones in the project

This adds more useful information to the v8-heap-stats tool.

BUG=v8:5489

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

[commit-bot: I haz the power, 2016-10-17 12:12:52]

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