IgnoreCertError should check if the error is a certificate error.

IgnoreCertError should check if the error is a certificate error.

This only affects one of the two callers. --ignore-certificate-error ends up
causing some metrics to record on all errors rather than just certificate ones,
so we end up recording things based on a default-initialized SSLInfo.

BUG=none

Committed: https://crrev.com/d3f1515455e2b8ce6d7d33dffea95bc7a86fa559
Cr-Commit-Position: refs/heads/master@{#317442}

[davidben, 2015-02-20 20:52:31]

davidben@chromium.org changed reviewers:
+ rsleevi@chromium.org

[davidben, 2015-02-20 20:52:31]

Here are the two callers of this function. One already assumes
IsCertificateError, so it's a no-op. The other is the problem call.

https://code.google.com/p/chromium/codesearch#search/&q=IgnoreCertError%20cas...

[Ryan Sleevi, 2015-02-20 22:24:16]

The CQ bit was checked by rsleevi@chromium.org

[Ryan Sleevi, 2015-02-20 22:24:16]

lgtm

[commit-bot: I haz the power, 2015-02-20 22:25:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/947453003/1

[commit-bot: I haz the power, 2015-02-20 23:43:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-20 23:43:46]

Patchset 1 (id:??) landed as
https://crrev.com/d3f1515455e2b8ce6d7d33dffea95bc7a86fa559
Cr-Commit-Position: refs/heads/master@{#317442}