Chromium Code Reviews| Index: content/browser/ssl/ssl_policy.cc |
| diff --git a/content/browser/ssl/ssl_policy.cc b/content/browser/ssl/ssl_policy.cc |
| index 18fdde4dfe2bf9823e716ae01e48db73f1641fb8..816dd85f89b1d446a9c05083ecc9bf8880a13832 100644 |
| --- a/content/browser/ssl/ssl_policy.cc |
| +++ b/content/browser/ssl/ssl_policy.cc |
| @@ -34,21 +34,19 @@ SSLPolicy::SSLPolicy(SSLPolicyBackend* backend) |
| void SSLPolicy::OnCertError(SSLCertErrorHandler* handler) { |
| bool expired_previous_decision; |
| // First we check if we know the policy for this error. |
| - net::CertPolicy::Judgment judgment = |
| + SSLHostStateDelegate::CertJudgment judgment = |
| backend_->QueryPolicy(handler->ssl_info().cert.get(), |
| handler->request_url().host(), |
| handler->cert_error(), |
| &expired_previous_decision); |
| - if (judgment == net::CertPolicy::ALLOWED) { |
| + if (judgment == SSLHostStateDelegate::ALLOWED) { |
| handler->ContinueRequest(); |
| return; |
| } |
| - // The judgment is either DENIED or UNKNOWN. |
| - // For now we handle the DENIED as the UNKNOWN, which means a blocking |
| - // page is shown to the user every time he comes back to the page. |
| - |
| + // For all other hosts, which must be DENIED, a blocking page is shown to the |
| + // user every time he comes back to the page. |
|
Ryan Sleevi
2014/09/04 00:17:25
s/he comes/they come/
jww
2014/09/04 01:06:30
Done.
|
| int options_mask = 0; |
| switch (handler->cert_error()) { |
| case net::ERR_CERT_COMMON_NAME_INVALID: |
| @@ -180,13 +178,6 @@ void SSLPolicy::OnAllowCertificate(scoped_refptr<SSLCertErrorHandler> handler, |
| handler->ContinueRequest(); |
| } else { |
| // Default behavior for rejecting a certificate. |
| - // |
| - // While DenyCertForHost() executes synchronously on this thread, |
| - // CancelRequest() gets posted to a different thread. Calling |
| - // DenyCertForHost() first ensures deterministic ordering. |
| - backend_->DenyCertForHost(handler->ssl_info().cert.get(), |
| - handler->request_url().host(), |
| - handler->cert_error()); |
| handler->CancelRequest(); |
| } |
| } |