Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1206)

Unified Diff: chrome/common/notification_type.h

Issue 2067003: Track "display" and "run" separately for mixed content, and make the latter d... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/notification_type.h
===================================================================
--- chrome/common/notification_type.h (revision 47175)
+++ chrome/common/notification_type.h (working copy)
@@ -131,16 +131,15 @@
// Updating the SSL security indicators (the lock icon and such) proceeds
// in two phases:
//
- // 1) An SSLManager changes the SSLHostState (which hangs off the profile
- // object). When this happens, the SSLManager broadcasts an
- // SSL_INTERNAL_STATE_CHANGED notification.
+ // 1) The internal SSL state for a host or tab changes. 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 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.
+ // 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.
// 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
@@ -149,9 +148,7 @@
// case. Listen to this notification if you need to refresh SSL-related UI
// elements.
//
- // 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.
+ // There is no source or details.
SSL_VISIBLE_STATE_CHANGED,
// The SSL state of the browser has changed in some internal way. For

Powered by Google App Engine
This is Rietveld 408576698