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

Unified Diff: chrome/common/chrome_notification_types.h

Issue 10800031: Remove details from BROWSER_CLOSING and BROWSER_CLOSED notifications. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed automation Created 8 years, 5 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
« no previous file with comments | « chrome/browser/ui/tabs/pinned_tab_service.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/chrome_notification_types.h
diff --git a/chrome/common/chrome_notification_types.h b/chrome/common/chrome_notification_types.h
index 8d88e677e10f40b02635ec30eb42ce0017683588..339ca59366dc7745e3c21169365dac3e3f6b2436 100644
--- a/chrome/common/chrome_notification_types.h
+++ b/chrome/common/chrome_notification_types.h
@@ -25,20 +25,13 @@ enum NotificationType {
NOTIFICATION_BROWSER_WINDOW_READY,
// This message is sent when a browser is closing. The source is a
- // Source<Browser> containing the affected Browser. Details is a boolean
- // that if true indicates that the application will be closed as a result of
- // this browser window closure (i.e. this was the last opened browser
- // window on win/linux). This is sent prior to BROWSER_CLOSED, and may be
- // sent more than once for a particular browser.
+ // Source<Browser> containing the affected Browser. No details are expected.
+ // This is sent prior to BROWSER_CLOSED, and may be sent more than once for a
+ // particular browser.
NOTIFICATION_BROWSER_CLOSING,
// This message is sent after a window has been closed. The source is a
- // Source<Browser> containing the affected Browser. Details is a boolean
- // that if true indicates that the last browser window has closed - this
- // does not indicate that the application is exiting (observers should
- // listen for APP_TERMINATING if they want to detect when the application
- // will shut down). Note that the boolean pointed to by details is only
- // valid for the duration of this call.
+ // Source<Browser> containing the affected Browser. No details are exptected.
NOTIFICATION_BROWSER_CLOSED,
// This message is sent when closing a browser has been cancelled, either by
« no previous file with comments | « chrome/browser/ui/tabs/pinned_tab_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698