Set SSL info when an HTTP auth dialog is triggered by direct navigation.

content/browser/ssl/ssl_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/ssl/ssl_manager.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 62 matching lines @@
                                          id,
                                          resource_type,
                                          url,
                                          render_process_id,
                                          render_frame_id,
                                          ssl_info,
                                          fatal)));
 }
 
 // static
+void SSLManager::OnAuthDialog(int render_process_id,
+                              int render_frame_host,
+                              const std::string& serialized_security_info,
+                              bool is_main_frame) {
+  if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
+    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+                            base::Bind(SSLManager::OnAuthDialog,
+                                       render_process_id,
+                                       render_frame_host,
+                                       serialized_security_info,
+                                       is_main_frame));
+    return;
+  }
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));

[nasko, 2014/07/24 09:32:34]

This DCHECK seems useless because of the if statement above.

[meacer, 2014/07/24 17:32:32]

Done.

+  RenderFrameHost* host =
+      RenderFrameHostImpl::FromID(render_process_id, render_frame_host);
+  WebContentsImpl* web_contents =
+      static_cast<WebContentsImpl*>(WebContents::FromRenderFrameHost(host));

[nasko, 2014/07/24 09:32:34]

Why do you need to cast this? GetController() is available on the public
content/ API.

[meacer, 2014/07/24 17:32:32]

I agree it's confusing. It's now casting NavigationController rather than
WebContents.

+  if (!web_contents)
+    return;
+
+  NavigationControllerImpl* controller = &web_contents->GetController();

[nasko, 2014/07/24 09:32:34]

Ah, this is the place where a cast will be needed.

[meacer, 2014/07/24 17:32:32]

Acknowledged.

+  NavigationEntryImpl* entry =
+      NavigationEntryImpl::FromNavigationEntry(controller->GetVisibleEntry());
+
+  scoped_ptr<SSLManager> ssl_manager(new SSLManager(controller));

[nasko, 2014/07/24 09:32:34]

Why do we need to construct a SSLManager? The nav controller should already have
one of those and it should be registered, so looking it up and calling
UpdateEntry on it seems to make sense.

[meacer, 2014/07/24 17:32:32]

Done.

+  ssl_manager->UpdateEntry(serialized_security_info,
+                           is_main_frame,
+                           entry);
+}
+
+// static
 void SSLManager::NotifySSLInternalStateChanged(BrowserContext* context) {
   SSLManagerSet* managers = static_cast<SSLManagerSet*>(
       context->GetUserData(kSSLManagerKeyName));
 
   for (std::set<SSLManager*>::iterator i = managers->get().begin();
        i != managers->get().end(); ++i) {
     (*i)->UpdateEntry(NavigationEntryImpl::FromNavigationEntry(
                           (*i)->controller()->GetLastCommittedEntry()));
   }
 }
@@ skipping 16 matching lines @@
 SSLManager::~SSLManager() {
   SSLManagerSet* managers = static_cast<SSLManagerSet*>(
       controller_->GetBrowserContext()->GetUserData(kSSLManagerKeyName));
   managers->get().erase(this);
 }
 
 void SSLManager::DidCommitProvisionalLoad(const LoadCommittedDetails& details) {
   NavigationEntryImpl* entry =
       NavigationEntryImpl::FromNavigationEntry(
           controller_->GetLastCommittedEntry());
+  UpdateEntry(details.serialized_security_info,
+              details.is_main_frame,
+              entry);
+}
 
-  if (details.is_main_frame) {
-    if (entry) {
-      // Decode the security details.
-      int ssl_cert_id;
-      net::CertStatus ssl_cert_status;
-      int ssl_security_bits;
-      int ssl_connection_status;
-      SignedCertificateTimestampIDStatusList
-          ssl_signed_certificate_timestamp_ids;
-      DeserializeSecurityInfo(details.serialized_security_info,
-                              &ssl_cert_id,
-                              &ssl_cert_status,
-                              &ssl_security_bits,
-                              &ssl_connection_status,
-                              &ssl_signed_certificate_timestamp_ids);
+void SSLManager::UpdateEntry(const std::string& serialized_security_info,

[nasko, 2014/07/24 09:32:34]

nit: all params should be on new lines

[meacer, 2014/07/24 17:32:32]

Done.

+                 bool is_main_frame,
+                 NavigationEntryImpl* entry) {
+  if (is_main_frame && entry) {
+    // Decode the security details.
+    int ssl_cert_id;
+    net::CertStatus ssl_cert_status;
+    int ssl_security_bits;
+    int ssl_connection_status;
+    SignedCertificateTimestampIDStatusList
+        ssl_signed_certificate_timestamp_ids;
+    DeserializeSecurityInfo(serialized_security_info,
+                            &ssl_cert_id,
+                            &ssl_cert_status,
+                            &ssl_security_bits,
+                            &ssl_connection_status,
+                            &ssl_signed_certificate_timestamp_ids);
 
-      // We may not have an entry if this is a navigation to an initial blank
-      // page. Reset the SSL information and add the new data we have.
-      entry->GetSSL() = SSLStatus();
-      entry->GetSSL().cert_id = ssl_cert_id;
-      entry->GetSSL().cert_status = ssl_cert_status;
-      entry->GetSSL().security_bits = ssl_security_bits;
-      entry->GetSSL().connection_status = ssl_connection_status;
-      entry->GetSSL().signed_certificate_timestamp_ids =
-          ssl_signed_certificate_timestamp_ids;
-    }
+    // We may not have an entry if this is a navigation to an initial blank
+    // page. Reset the SSL information and add the new data we have.
+    entry->GetSSL() = SSLStatus();
+    entry->GetSSL().cert_id = ssl_cert_id;
+    entry->GetSSL().cert_status = ssl_cert_status;
+    entry->GetSSL().security_bits = ssl_security_bits;
+    entry->GetSSL().connection_status = ssl_connection_status;
+    entry->GetSSL().signed_certificate_timestamp_ids =
+        ssl_signed_certificate_timestamp_ids;
   }
-
   UpdateEntry(entry);
 }
 
 void SSLManager::DidDisplayInsecureContent() {
   UpdateEntry(
       NavigationEntryImpl::FromNavigationEntry(
           controller_->GetLastCommittedEntry()));
 }
 
 void SSLManager::DidRunInsecureContent(const std::string& security_origin) {
@@ skipping 56 matching lines @@
 
   WebContentsImpl* contents =
       static_cast<WebContentsImpl*>(controller_->delegate()->GetWebContents());
   policy()->UpdateEntry(entry, contents);
 
   if (!entry->GetSSL().Equals(original_ssl_status))
     contents->DidChangeVisibleSSLState();
 }
 
 }  // namespace content