Adding malloc allocator metric for objects allocated through shim

Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191
Review-Url: https://codereview.chromium.org/2801153004
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/43f23c5d1d94df7282984a968e14731d6f7c9dd3

[etienneb, 2017-05-23 16:59:42]

Description was changed from

==========
[draft] malloc internal metrics

BUG=catapult:#
==========

to

==========
[draft] malloc internal metrics

BUG=713191
==========

[etienneb, 2017-05-23 17:00:38]

Description was changed from

==========
[draft] malloc internal metrics

BUG=713191
==========

to

==========
[draft] malloc internal metrics

BUG=chrome:713191
==========

[etienneb, 2017-05-23 17:01:54]

Description was changed from

==========
[draft] malloc internal metrics

BUG=chrome:713191
==========

to

==========
Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191
==========

[etienneb, 2017-05-23 17:06:03]

etienneb@chromium.org changed reviewers:
+ benjhayden@chromium.org

[etienneb, 2017-05-23 17:06:03]

PTAL

This CL is related to:
  https://codereview.chromium.org/2865333002

[etienneb, 2017-05-23 18:54:18]

Description was changed from

==========
Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191
==========

to

==========
Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191
==========

[etienneb, 2017-05-23 20:31:06]

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

[commit-bot: I haz the power, 2017-05-23 20:31:10]

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, 2017-05-23 20:34:04]

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

[commit-bot: I haz the power, 2017-05-23 20:34:05]

Dry run: Try jobs failed on following builders:
  Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)

[benjhayden, 2017-05-24 21:53:34]

benjhayden@chromium.org changed reviewers:
+ hjd@chromium.org

[benjhayden, 2017-05-24 21:53:35]

I defer to hjd

[benjhayden, 2017-05-24 21:53:46]

benjhayden@chromium.org changed reviewers:
- benjhayden@chromium.org

[hjd, 2017-05-25 12:15:34]

On 2017/05/24 21:53:35, benjhayden wrote:
> I defer to hjd

lgtm, thanks!

[etienneb, 2017-05-25 15:13:16]

The CQ bit was checked by etienneb@chromium.org

[commit-bot: I haz the power, 2017-05-25 15:13: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, 2017-05-25 15:16:16]

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

[commit-bot: I haz the power, 2017-05-25 15:16:18]

Try jobs failed on following builders:
  Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)

[etienneb, 2017-05-25 15:23:21]

I believe hjd@ can't approve that CL:

"""
  Missing LGTM from an OWNER for these files:
    tracing/tracing/metrics/system_health/memory_metric.html
  
  tracing/tracing/metrics/system_health/memory_metric_test.html

"""

[hjd, 2017-05-25 15:25:40]

On 2017/05/25 15:23:21, etienneb wrote:
> I believe hjd@ can't approve that CL:
> 
> """
>   Missing LGTM from an OWNER for these files:
>     tracing/tracing/metrics/system_health/memory_metric.html
>   
>   tracing/tracing/metrics/system_health/memory_metric_test.html
> 
> """

Sorry I should have been explicit that it was a non-owner lgtm, you'll still
need Ben's stamp.

[etienneb, 2017-05-25 15:27:53]

etienneb@chromium.org changed reviewers:
+ benjhayden@chromium.org

[etienneb, 2017-05-25 15:28:08]

Ben, we need a owner approval.

[benjhayden, 2017-05-25 17:07:39]

lgtm

[etienneb, 2017-05-25 19:25:47]

The CQ bit was checked by etienneb@chromium.org

[commit-bot: I haz the power, 2017-05-25 19:25:58]

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, 2017-05-25 19:28:27]

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

[commit-bot: I haz the power, 2017-05-25 19:28:28]

Try jobs failed on following builders:
  Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)

[etienneb, 2017-05-25 19:45:34]

The CQ bit was checked by etienneb@chromium.org

[etienneb, 2017-05-25 19:45:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
benjhayden@chromium.org, hjd@chromium.org
Link to the patchset: https://codereview.chromium.org/2801153004/#ps80001
(title: "presubmit")

[commit-bot: I haz the power, 2017-05-25 19:45:38]

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, 2017-05-25 20:09:32]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1495741534110600,
"parent_rev": "81141992b3456e0c9f12eb9da3f2728b0fcefa3f", "commit_rev":
"43f23c5d1d94df7282984a968e14731d6f7c9dd3"}

[commit-bot: I haz the power, 2017-05-25 20:09:35]

Description was changed from

==========
Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191
==========

to

==========
Adding malloc allocator metric for objects allocated through shim

BUG=chromium:713191

Review-Url: https://codereview.chromium.org/2801153004
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-05-25 20:09:37]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...