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

Unified Diff: chrome/browser/notifications/notification_ui_manager_mac.h

Issue 11414215: Add CloseAllByProfile to NotificationUIManager and call before destroyng a Profile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix mac Created 8 years, 1 month 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/browser/notifications/notification_ui_manager_mac.h
diff --git a/chrome/browser/notifications/notification_ui_manager_mac.h b/chrome/browser/notifications/notification_ui_manager_mac.h
index 10184608a8660a32f866b7fd51c11b4f954acf74..e962fdb3d06a8099158a4fafd6342d625020b701 100644
--- a/chrome/browser/notifications/notification_ui_manager_mac.h
+++ b/chrome/browser/notifications/notification_ui_manager_mac.h
@@ -37,6 +37,7 @@ class NotificationUIManagerMac : public NotificationUIManager {
Profile* profile) OVERRIDE;
virtual bool CancelById(const std::string& notification_id) OVERRIDE;
virtual bool CancelAllBySourceOrigin(const GURL& source_origin) OVERRIDE;
+ virtual bool CancelAllByProfile(Profile* profile) OVERRIDE;
virtual void CancelAll() OVERRIDE;
virtual BalloonCollection* balloon_collection() OVERRIDE;
virtual NotificationPrefsManager* prefs_manager() OVERRIDE;
@@ -57,10 +58,12 @@ class NotificationUIManagerMac : public NotificationUIManager {
// (model) notification. This assumes ownership (i.e. does not retain a_view)
// of both objects and will delete them on destruction.
struct ControllerNotification {
- ControllerNotification(id<CrUserNotification> a_view,
+ ControllerNotification(Profile* a_profile,
+ id<CrUserNotification> a_view,
Notification* a_model);
~ControllerNotification();
+ Profile* profile;
id<CrUserNotification> view;
Notification* model;

Powered by Google App Engine
This is Rietveld 408576698