From b9a9188da43a172c6b299e27075c1486fe4458e2 Mon Sep 17 00:00:00 2001 From: "markus@chromium.org" Date: Mon, 30 Mar 2009 22:23:51 +0000 Subject: Reverting, as this changelist broke unittests on Windows. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12816 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/ssl/ssl_policy.cc | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'chrome') diff --git a/chrome/browser/ssl/ssl_policy.cc b/chrome/browser/ssl/ssl_policy.cc index 5221c04..9e796fa 100644 --- a/chrome/browser/ssl/ssl_policy.cc +++ b/chrome/browser/ssl/ssl_policy.cc @@ -318,13 +318,9 @@ SSLErrorInfo SSLPolicy::GetSSLErrorInfo(SSLManager::CertError* error) { void SSLPolicy::OnDenyCertificate(SSLManager::CertError* error) { // 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. + error->CancelRequest(); error->manager()->DenyCertForHost(error->ssl_info().cert, error->request_url().host()); - error->CancelRequest(); } void SSLPolicy::OnAllowCertificate(SSLManager::CertError* error) { @@ -334,13 +330,9 @@ void SSLPolicy::OnAllowCertificate(SSLManager::CertError* error) { // new NavigationEntry will not be set until DidNavigate. This is ok, // because the new NavigationEntry will have its max security style set // within DidNavigate. - // - // While AllowCertForHost() executes synchronously on this thread, - // ContinueRequest() gets posted to a different thread. Calling - // AllowCertForHost() first ensures deterministic ordering. + error->ContinueRequest(); error->manager()->AllowCertForHost(error->ssl_info().cert, error->request_url().host()); - error->ContinueRequest(); } //////////////////////////////////////////////////////////////////////////////// -- cgit v1.1