Factor out ResourceDispatcherHost dependent code around SSLManager

content/browser/ssl/ssl_manager.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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 "base/bind.h"
 #include "base/utf_string_conversions.h"
 #include "content/browser/load_from_memory_cache_details.h"
 #include "content/browser/renderer_host/resource_dispatcher_host.h"
 #include "content/browser/renderer_host/resource_dispatcher_host_request_info.h"
@@ skipping 12 matching lines @@
 #include "content/public/common/ssl_status.h"
 
 using content::BrowserThread;
 using content::NavigationController;
 using content::NavigationEntry;
 using content::NavigationEntryImpl;
 using content::SSLStatus;
 using content::WebContents;
 
 // static
-void SSLManager::OnSSLCertificateError(ResourceDispatcherHost* rdh,
-                                       net::URLRequest* request,
+void SSLManager::OnSSLCertificateError(SSLErrorHandler::Delegate* delegate,
+                                       const content::GlobalRequestID& id,
+                                       const ResourceType::Type resource_type,
+                                       const GURL& url,
+                                       int render_process_id,
+                                       int render_view_id,
                                        const net::SSLInfo& ssl_info,
                                        bool fatal) {
+  DCHECK(delegate);
   DVLOG(1) << "OnSSLCertificateError() cert_error: "
            << net::MapCertStatusToNetError(ssl_info.cert_status)
-           << " url: " << request->url().spec()
+           << " id: " << id.child_id << "," << id.request_id
+           << " resource_type: " << resource_type
+           << " url: " << url
+           << " render_process_id: " << render_process_id
+           << " render_view_id: " << render_view_id
            << " cert_status: " << std::hex << ssl_info.cert_status;
 
-  ResourceDispatcherHostRequestInfo* info =
-      ResourceDispatcherHost::InfoForRequest(request);
-
   // A certificate error occurred.  Construct a SSLCertErrorHandler object and
   // hand it over to the UI thread for processing.
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       base::Bind(&SSLCertErrorHandler::Dispatch,
-                 new SSLCertErrorHandler(rdh,
-                                         request,
-                                         info->resource_type(),
+                 new SSLCertErrorHandler(delegate,
+                                         id,
+                                         resource_type,
+                                         url,
+                                         render_process_id,
+                                         render_view_id,
                                          ssl_info,
                                          fatal)));
 }
 
 // static
 void SSLManager::NotifySSLInternalStateChanged(
     NavigationControllerImpl* controller) {
   content::NotificationService::current()->Notify(
       content::NOTIFICATION_SSL_INTERNAL_STATE_CHANGED,
       content::Source<content::BrowserContext>(controller->GetBrowserContext()),
@@ skipping 149 matching lines @@
 
   policy()->UpdateEntry(entry, controller_->tab_contents());
 
   if (!entry->GetSSL().Equals(original_ssl_status)) {
     content::NotificationService::current()->Notify(
         content::NOTIFICATION_SSL_VISIBLE_STATE_CHANGED,
         content::Source<NavigationController>(controller_),
         content::NotificationService::NoDetails());
   }
 }