Add UpdateEngine.CertificateCheck.* to histograms.xml

Add UpdateEngine.CertificateCheck.* to histograms.xml

This CL adds two Chrome OS specific histograms:
* UpdateEngine.CertificateCheck.Download
* UpdateEngine.CertificateCheck.UpdateCheck

BUG=563800
Committed: https://crrev.com/98f90bfc930621229ba9b28e7a453d603cfddc59
Cr-Commit-Position: refs/heads/master@{#363040}

[bsimonnet, 2015-12-01 01:35:30]

bsimonnet@chromium.org changed reviewers:
+ deymo@chromium.org

[deymo, 2015-12-01 18:15:16]

deymo@chromium.org changed reviewers:
+ rkaplow@chromium.org

[deymo, 2015-12-01 18:15:16]

lgtm w/nit.

You will need OWNER review. +rkaplow for that.

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

https://codereview.chromium.org/1490643002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:45240: +    This is reported on every
payload download over HTTPS.
This is reported on every HTTPS connection to the payload download server.
Connection drops on the same payload may report different values.

[bsimonnet, 2015-12-01 18:58:07]

On 2015/12/01 18:15:16, deymo wrote:
> lgtm w/nit.
> 
> You will need OWNER review. +rkaplow for that.
> 
>
https://codereview.chromium.org/1490643002/diff/1/tools/metrics/histograms/hi...
> File tools/metrics/histograms/histograms.xml (right):
> 
>
https://codereview.chromium.org/1490643002/diff/1/tools/metrics/histograms/hi...
> tools/metrics/histograms/histograms.xml:45240: +    This is reported on every
> payload download over HTTPS.
> This is reported on every HTTPS connection to the payload download server.
> Connection drops on the same payload may report different values.

Done.
Thanks, I cced an owner instead of adding it to the review -_-

[bsimonnet, 2015-12-02 20:45:53]

bsimonnet@chromium.org changed reviewers:
+ asvitkine@google.com

[rkaplow, 2015-12-03 15:41:08]

lgtm

oops, sorry for delay

[bsimonnet, 2015-12-03 17:58:54]

The CQ bit was checked by bsimonnet@chromium.org

[bsimonnet, 2015-12-03 17:58:55]

The patchset sent to the CQ was uploaded after l-g-t-m from deymo@chromium.org
Link to the patchset: https://codereview.chromium.org/1490643002/#ps20001
(title: "Fixed nits.")

[commit-bot: I haz the power, 2015-12-03 17:59:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1490643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1490643002/20001

[commit-bot: I haz the power, 2015-12-03 20:08:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 20:10:34]

Description was changed from

==========
Add UpdateEngine.CertificateCheck.* to histograms.xml

This CL adds two Chrome OS specific histograms:
* UpdateEngine.CertificateCheck.Download
* UpdateEngine.CertificateCheck.UpdateCheck

BUG=563800
==========

to

==========
Add UpdateEngine.CertificateCheck.* to histograms.xml

This CL adds two Chrome OS specific histograms:
* UpdateEngine.CertificateCheck.Download
* UpdateEngine.CertificateCheck.UpdateCheck

BUG=563800

Committed: https://crrev.com/98f90bfc930621229ba9b28e7a453d603cfddc59
Cr-Commit-Position: refs/heads/master@{#363040}
==========

[commit-bot: I haz the power, 2015-12-03 20:10:34]

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