Add UMA to record WorkerThread runtime

Add UMA to record WorkerThread runtime

We have one for ServiceWorker but don't have the stat for
other workers.

Having it would be useful to add other stats like
WorkerThread CPU Load UMA (https://codereview.chromium.org/2749383003/)

BUG=692906
Review-Url: https://codereview.chromium.org/2766263005
Cr-Commit-Position: refs/heads/master@{#459349}
Committed: https://chromium.googlesource.com/chromium/src/+/c71427629a142302c08179360116f7a0125b90cc

[kinuko, 2017-03-23 08:36:49]

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

[commit-bot: I haz the power, 2017-03-23 08:37:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kinuko, 2017-03-23 09:38:58]

kinuko@chromium.org changed reviewers:
+ altimin@chromium.org, falken@chromium.org, nhiroki@chromium.org,
skyostil@chromium.org

[kinuko, 2017-03-23 09:38:59]

PTL

(Context: https://codereview.chromium.org/2749383003/#msg19, we want to know how
long worker threads typically last)

[kinuko, 2017-03-23 09:39:42]

Description was changed from

==========
Add UMA to record WorkerThread runtime

We have one for ServiceWorker but don't have the stat for
other workers.

BUG=692906
==========

to

==========
Add UMA to record WorkerThread runtime

We have one for ServiceWorker but don't have the stat for
other workers.

Having it would be useful to add other stats like
WorkerThread CPU Load UMA (https://codereview.chromium.org/2749383003/)

BUG=692906
==========

[nhiroki, 2017-03-23 10:31:50]

lgtm

[commit-bot: I haz the power, 2017-03-23 10:44:27]

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

[commit-bot: I haz the power, 2017-03-23 10:44:28]

Dry run: This issue passed the CQ dry run.

[altimin, 2017-03-23 11:18:35]

lgtm

[kinuko, 2017-03-23 11:30:49]

kinuko@chromium.org changed reviewers:
+ asvitkine@chromium.org

[kinuko, 2017-03-23 11:31:28]

asvitkine@chromium.org: could you review histogram change?

[kinuko, 2017-03-23 11:33:05]

kinuko@chromium.org changed reviewers:
+ isherman@chromium.org
- asvitkine@chromium.org

[kinuko, 2017-03-23 11:34:16]

On 2017/03/23 11:31:28, kinuko wrote:
> mailto:asvitkine@chromium.org: could you review histogram change?

oops he seems busy...

isherman@chromium.org could you review histogram changes?

[Ilya Sherman, 2017-03-23 19:31:00]

metrics lgtm

[kinuko, 2017-03-24 02:59:48]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2017-03-24 03:00: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, 2017-03-24 03:06:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490324388000250, "parent_rev":
"85f49a916b4835ec93379a0eaeb4f125b66f6270", "commit_rev":
"c71427629a142302c08179360116f7a0125b90cc"}

[commit-bot: I haz the power, 2017-03-24 03:08:03]

Description was changed from

==========
Add UMA to record WorkerThread runtime

We have one for ServiceWorker but don't have the stat for
other workers.

Having it would be useful to add other stats like
WorkerThread CPU Load UMA (https://codereview.chromium.org/2749383003/)

BUG=692906
==========

to

==========
Add UMA to record WorkerThread runtime

We have one for ServiceWorker but don't have the stat for
other workers.

Having it would be useful to add other stats like
WorkerThread CPU Load UMA (https://codereview.chromium.org/2749383003/)

BUG=692906

Review-Url: https://codereview.chromium.org/2766263005
Cr-Commit-Position: refs/heads/master@{#459349}
Committed:
https://chromium.googlesource.com/chromium/src/+/c71427629a142302c08179360116...
==========

[commit-bot: I haz the power, 2017-03-24 03:08:04]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c71427629a142302c08179360116...