Profiles: SSLManger broadcasts SSL_INTERNAL_STATE_CHANGED with a NavigationController.

content/browser/ssl/ssl_manager.cc

Index: content/browser/ssl/ssl_manager.cc
diff --git a/content/browser/ssl/ssl_manager.cc b/content/browser/ssl/ssl_manager.cc
index 0f6fff359f09d2aca87e8311f34d1a7b3bdf8d58..bc492850105d499bd1dbfe1701ba5d23a2e72ebc 100644
--- a/content/browser/ssl/ssl_manager.cc
+++ b/content/browser/ssl/ssl_manager.cc
@@ -18,6 +18,7 @@
 #include "content/browser/tab_contents/provisional_load_details.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/common/notification_service.h"
+#include "content/common/notification_source.h"
 #include "net/base/cert_status_flags.h"
 
 // static
@@ -44,11 +45,12 @@ void SSLManager::OnSSLCertificateError(ResourceDispatcherHost* rdh,
 }
 
 // static
-void SSLManager::NotifySSLInternalStateChanged() {
+void SSLManager::NotifySSLInternalStateChanged(
+    NavigationController* controller) {
   NotificationService::current()->Notify(
       content::NOTIFICATION_SSL_INTERNAL_STATE_CHANGED,
-      NotificationService::AllSources(),
-      NotificationService::NoDetails());
+      Source<content::BrowserContext>(controller->browser_context()),
+      Details<NavigationController>(controller));

[abarth-chromium, 2011/08/04 21:09:32]

These details shouldn't be needed.  I'd skip it.  We can always add it if
needed.

 }
 
 // static
@@ -105,7 +107,8 @@ SSLManager::SSLManager(NavigationController* controller)
   registrar_.Add(this, content::NOTIFICATION_LOAD_FROM_MEMORY_CACHE,
                  Source<NavigationController>(controller_));
   registrar_.Add(this, content::NOTIFICATION_SSL_INTERNAL_STATE_CHANGED,
-                 NotificationService::AllSources());
+                 Source<content::BrowserContext>(
+                     controller_->browser_context()));
 }
 
 SSLManager::~SSLManager() {