Add Net.CertVerifier_First_Job_Latency metric.

Add Net.CertVerifier_First_Job_Latency metric.

Specifically counts only the first CertVerifer job in order to capture any
initialization costs.

BUG=361166
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=262816

[Ryan Sleevi, 2014-04-08 20:19:01]

https://codereview.chromium.org/225583009/diff/1/net/cert/multi_threaded_cert...
File net/cert/multi_threaded_cert_verifier.cc (right):

https://codereview.chromium.org/225583009/diff/1/net/cert/multi_threaded_cert...
net/cert/multi_threaded_cert_verifier.cc:563: CertVerifierJob* job = j->second;
Rather than at a bool member to the CertVerifierJob (which will increase the
time of all jobs), why not store the time&first job in the CertVerifier, and do
the comparison here in HandleResult? That will reduce the run-time memory
overhead, and still offer the same run-time CPU overhead (a comparison to a
pointer, with presumably a sentinel/dead value to indicate you've recorded
things)

[davidben, 2014-04-08 23:33:20]

https://codereview.chromium.org/225583009/diff/1/net/cert/multi_threaded_cert...
File net/cert/multi_threaded_cert_verifier.cc (right):

https://codereview.chromium.org/225583009/diff/1/net/cert/multi_threaded_cert...
net/cert/multi_threaded_cert_verifier.cc:563: CertVerifierJob* job = j->second;
On 2014/04/08 20:19:01, Ryan Sleevi wrote:
> Rather than at a bool member to the CertVerifierJob (which will increase the
> time of all jobs), why not store the time&first job in the CertVerifier, and
do
> the comparison here in HandleResult? That will reduce the run-time memory
> overhead, and still offer the same run-time CPU overhead (a comparison to a
> pointer, with presumably a sentinel/dead value to indicate you've recorded
> things)

Done.

[Ryan Sleevi, 2014-04-09 16:52:10]

lgtm

[Alexei Svitkine (slow), 2014-04-09 18:48:50]

lgtm

[davidben, 2014-04-09 18:51:59]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2014-04-09 18:52:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davidben@chromium.org/225583009/10001

[commit-bot: I haz the power, 2014-04-09 21:13:44]

Change committed as 262816