Revert 47347 - (Original patch reviewed at http://codereview.chromium.org/206...

chrome/common/notification_type.h

 // Copyright (c) 2009 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.
 
 #ifndef CHROME_COMMON_NOTIFICATION_TYPE_H_
 #define CHROME_COMMON_NOTIFICATION_TYPE_H_
 
 // This file describes various types used to describe and filter notifications
 // that pass through the NotificationService.
 //
@@ skipping 113 matching lines @@
     // a Source<NavigationController> corresponding to the tab in which the
     // request was issued.  Details in the form of a ResourceRedirectDetails
     // are provided.
     RESOURCE_RECEIVED_REDIRECT,
 
     // SSL ---------------------------------------------------------------------
 
     // Updating the SSL security indicators (the lock icon and such) proceeds
     // in two phases:
     //
-    // 1) The internal SSL state for a host or tab changes.  When this happens,
-    //    the SSLManager broadcasts an SSL_INTERNAL_STATE_CHANGED notification.
+    // 1) An SSLManager changes the SSLHostState (which hangs off the profile
+    //    object).  When this happens, the SSLManager broadcasts an
+    //    SSL_INTERNAL_STATE_CHANGED notification.
     //
     // 2) The SSLManager for each tab receives this notification and might or
     //    might not update the navigation entry for its tab, depending on
-    //    whether the change in state affects that tab.  If the SSLManager does
-    //    change the navigation entry, then the SSLManager broadcasts an
-    //    SSL_VISIBLE_STATE_CHANGED notification to the user interface can
-    //    redraw properly.
+    //    whether the change in SSLHostState affects that tab.  If the
+    //    SSLManager does change the navigation entry, then the SSLManager
+    //    broadcasts an SSL_VISIBLE_STATE_CHANGED notification to the user
+    //    interface can redraw properly.
 
     // The SSL state of a page has changed in some visible way.  For example,
     // if an insecure resource is loaded on a secure page.  Note that a
     // toplevel load commit will also update the SSL state (since the
     // NavigationEntry is new) and this message won't always be sent in that
     // case.  Listen to this notification if you need to refresh SSL-related UI
     // elements.
     //
-    // There is no source or details.
+    // The source will be the navigation controller associated with the load.
+    // There are no details.  The entry changed will be the active entry of the
+    // controller.
     SSL_VISIBLE_STATE_CHANGED,
 
     // The SSL state of the browser has changed in some internal way.  For
     // example, the user might have explicitly allowed some broken certificate
     // or a secure origin might have included some insecure content.  Listen to
     // this notifiation if you need to keep track of our internal SSL state.
     //
     // The source will be the navigation controller associated with the state
     // change.  There are no details.
     SSL_INTERNAL_STATE_CHANGED,
@@ skipping 846 matching lines @@
 };
 
 inline bool operator==(NotificationType::Type a, NotificationType b) {
   return a == b.value;
 }
 inline bool operator!=(NotificationType::Type a, NotificationType b) {
   return a != b.value;
 }
 
 #endif  // CHROME_COMMON_NOTIFICATION_TYPE_H_