Add UMA histogram to show how many retries we had to do to execute a DeviceManagementRequestJob.

Add UMA histogram to show how many retries we had to do to execute a DeviceManagementRequestJob.

BUG=655652
Committed: https://crrev.com/43f6aa3711de2334be84bdf28ec68e9c27d29052
Cr-Commit-Position: refs/heads/master@{#425364}

[Marton Hunyady, 2016-10-13 14:59:52]

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

[commit-bot: I haz the power, 2016-10-13 15:00:09]

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

[Marton Hunyady, 2016-10-13 15:00:42]

hunyadym@chromium.org changed reviewers:
+ atwilson@chromium.org

[Marton Hunyady, 2016-10-13 15:00:42]

PTAL

[Marton Hunyady, 2016-10-13 15:17:05]

hunyadym@chromium.org changed reviewers:
+ jwd@chromium.org

[Marton Hunyady, 2016-10-13 15:17:06]

jwd: PTAL at histograms.xml

[commit-bot: I haz the power, 2016-10-13 16:07:20]

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

[commit-bot: I haz the power, 2016-10-13 16:07:22]

Dry run: This issue passed the CQ dry run.

[jwd, 2016-10-13 19:58:37]

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

https://codereview.chromium.org/2413993003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12740: +    Number of retries the client
needed to do to execute a
nit: "to do to"

https://codereview.chromium.org/2413993003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12741: +    DeviceManagementServer
request.
Mention it's recorded on success or failure of the request.

[Marton Hunyady, 2016-10-14 08:40:34]

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

https://codereview.chromium.org/2413993003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12740: +    Number of retries the client
needed to do to execute a
On 2016/10/13 19:58:37, jwd wrote:
> nit: "to do to"

Done.

https://codereview.chromium.org/2413993003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:12741: +    DeviceManagementServer
request.
On 2016/10/13 19:58:37, jwd wrote:
> Mention it's recorded on success or failure of the request. 

Done.

[Andrew T Wilson (Slow), 2016-10-14 09:18:28]

lgtm

[jwd, 2016-10-14 15:28:38]

lgtm

[Marton Hunyady, 2016-10-14 15:44:58]

The CQ bit was checked by hunyadym@chromium.org

[commit-bot: I haz the power, 2016-10-14 15:45:31]

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 16:57:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-14 16:59:36]

Description was changed from

==========
Add UMA histogram to show how many retries we had to do to execute a
DeviceManagementRequestJob.

BUG=655652
==========

to

==========
Add UMA histogram to show how many retries we had to do to execute a
DeviceManagementRequestJob.

BUG=655652

Committed: https://crrev.com/43f6aa3711de2334be84bdf28ec68e9c27d29052
Cr-Commit-Position: refs/heads/master@{#425364}
==========

[commit-bot: I haz the power, 2016-10-14 16:59:36]

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