Fix for a crasher in SSLManager

chrome/browser/ssl_manager.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/ssl_manager.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "chrome/app/theme/theme_resources.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 450 matching lines @@
                                        URLRequest* request,
                                        MessageLoop* ui_loop) {
   ui_loop->PostTask(FROM_HERE,
       NewRunnableMethod(new MixedContentHandler(rdh, request, ui_loop),
                         &MixedContentHandler::Dispatch));
 }
 
 void SSLManager::OnCertError(CertError* error) {
   // Ask our delegate to deal with the error.
   NavigationEntry* entry = controller_->GetActiveEntry();
+  // We might not have a navigation entry in some cases (e.g. when a
+  // HTTPS page opens a popup with no URL and then populate it with
+  // document.write()). See bug http://crbug.com/3845.
+  if (!entry)
+    return;
+
   delegate()->OnCertError(entry->url(), error);
 }
 
 void SSLManager::OnMixedContent(MixedContentHandler* mixed_content) {
   // Ask our delegate to deal with the mixed content.
   NavigationEntry* entry = controller_->GetActiveEntry();
+  // We might not have a navigation entry in some cases (e.g. when a
+  // HTTPS page opens a popup with no URL and then populate it with
+  // document.write()). See bug http://crbug.com/3845.
+  if (!entry)
+    return;
+
   delegate()->OnMixedContent(controller_, entry->url(), mixed_content);
 }
 
 void SSLManager::Observe(NotificationType type,
                          const NotificationSource& source,
                          const NotificationDetails& details) {
   // We should only be getting notifications from our controller.
   DCHECK(source == Source<NavigationController>(controller_));
 
   // Dispatch by type.
@@ skipping 217 matching lines @@
 
   if (ca_name) {
     // TODO(wtc): should we show the root CA's name instead?
     *ca_name = l10n_util::GetStringF(
         IDS_SECURE_CONNECTION_EV_CA,
         UTF8ToWide(cert.issuer().organization_names[0]));
   }
   return true;
 }