[Extensions] Add metrics for API binding generation time

[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561
Committed: https://crrev.com/95cbe926faeb42e0f4fcfa3c42c9cba95e77e4e5
Cr-Commit-Position: refs/heads/master@{#402969}

[Devlin, 2016-06-29 00:54:41]

Description was changed from

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=None
==========

to

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561
==========

[Devlin, 2016-06-29 00:55:11]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org, mpearson@chromium.org

[Devlin, 2016-06-29 00:55:12]

Istiaque and Mark, can you take a look?

[Mark P, 2016-06-29 19:44:14]

https://codereview.chromium.org/2102383002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2102383002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:13662: +    time when the API object is
returned. See also
I see a lot of early returns in the relevant function that would prevent this
metric from being emitted.  Can you mention this (in whatever depth you think is
appropriate) in this description.

[Devlin, 2016-06-29 19:50:18]

https://codereview.chromium.org/2102383002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2102383002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:13662: +    time when the API object is
returned. See also
On 2016/06/29 19:44:14, Mark P wrote:
> I see a lot of early returns in the relevant function that would prevent this
> metric from being emitted.  Can you mention this (in whatever depth you think
is
> appropriate) in this description.

Done.

[Mark P, 2016-06-29 19:55:27]

histograms.xml lgtm

[lazyboy, 2016-06-29 20:09:35]

lgtm

[Devlin, 2016-06-29 20:24:09]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-06-29 20:24:57]

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-06-29 22:41:05]

Description was changed from

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561
==========

to

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561
==========

[commit-bot: I haz the power, 2016-06-29 22:41:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-29 22:41:35]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-29 22:44:30]

Description was changed from

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561
==========

to

==========
[Extensions] Add metrics for API binding generation time

Extension API bindings are generated through some fancy javascript code,
but it's probably pretty slow by comparison to other mechanisms we're
looking at. Add UMA for the generation times so we have a better
picture.

BUG=608561

Committed: https://crrev.com/95cbe926faeb42e0f4fcfa3c42c9cba95e77e4e5
Cr-Commit-Position: refs/heads/master@{#402969}
==========

[commit-bot: I haz the power, 2016-06-29 22:44:31]

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