Add a UMA histogram for measuring DetachableResourceHandler duration

Add a UMA histogram for measuring DetachableResourceHandler duration

BUG=709910
Review-Url: https://codereview.chromium.org/2820303002
Cr-Commit-Position: refs/heads/master@{#465171}
Committed: https://chromium.googlesource.com/chromium/src/+/56866c20d1252bb4ad9a6a0ed51a14fa73b7632f

[yhirano, 2017-04-18 05:27:31]

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

[commit-bot: I haz the power, 2017-04-18 05:27:51]

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

[yhirano, 2017-04-18 05:29:20]

yhirano@chromium.org changed reviewers:
+ kinuko@chromium.org, tyoshino@chromium.org

[yhirano, 2017-04-18 05:29:20]

I think measuring the duration is enough - or do you want the completion status
as well?

[tyoshino (SeeGerritForStatus), 2017-04-18 05:37:33]

lgtm

[tyoshino (SeeGerritForStatus), 2017-04-18 05:39:13]

On 2017/04/18 05:29:20, yhirano (slow) wrote:
> I think measuring the duration is enough - or do you want the completion
status
> as well?

This is interesting but maybe we don't need it for the keepalive design
decision?

[kinuko, 2017-04-18 05:43:25]

lgtm

[kinuko, 2017-04-18 05:43:26]

lgtm

[yhirano, 2017-04-18 05:54:10]

yhirano@chromium.org changed reviewers:
+ isherman@chromium.org

[yhirano, 2017-04-18 05:54:10]

+isherman@ for histograms.xml change.

[Ilya Sherman, 2017-04-18 06:04:57]

metrics lgtm

[yhirano, 2017-04-18 06:50:03]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2017-04-18 06:50:04]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2017-04-18 06:50: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, 2017-04-18 07:35:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492498204106240, "parent_rev":
"8a759adb94636e0b6a1a1b3b07593ea81b2ba576", "commit_rev":
"56866c20d1252bb4ad9a6a0ed51a14fa73b7632f"}

[commit-bot: I haz the power, 2017-04-18 07:35:47]

Description was changed from

==========
Add a UMA histogram for measuring DetachableResourceHandler duration

BUG=709910
==========

to

==========
Add a UMA histogram for measuring DetachableResourceHandler duration

BUG=709910

Review-Url: https://codereview.chromium.org/2820303002
Cr-Commit-Position: refs/heads/master@{#465171}
Committed:
https://chromium.googlesource.com/chromium/src/+/56866c20d1252bb4ad9a6a0ed51a...
==========

[commit-bot: I haz the power, 2017-04-18 07:35:48]

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