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

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

Issue 11958025: Start delegating notifications to MessageCenter on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix androif build Created 7 years, 11 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/browser/notifications/notification_ui_manager_impl.h
diff --git a/chrome/browser/notifications/notification_ui_manager_impl.h b/chrome/browser/notifications/notification_ui_manager_impl.h
index 5f89c9726e7e6e77e59702e5250aeebb789a81c5..6dc30b0c9a460dbc86e55c022183e974f57879b9 100644
--- a/chrome/browser/notifications/notification_ui_manager_impl.h
+++ b/chrome/browser/notifications/notification_ui_manager_impl.h
@@ -52,7 +52,8 @@ class NotificationUIManagerImpl
// Replace an existing notification of the same id with this one if applicable;
// subclass returns 'true' if the replacement happened.
- virtual bool UpdateNotification(const Notification& notification) = 0;
+ virtual bool UpdateNotification(const Notification& notification,
+ Profile* profile) = 0;
// Attempts to display notifications from the show_queue. Invoked by subclasses
// if they previously returned 'false' from ShowNotifications, which may happen
@@ -72,7 +73,7 @@ class NotificationUIManagerImpl
// Replace an existing notification with this one if applicable;
// returns true if the replacement happened.
- bool TryReplacement(const Notification& notification);
+ bool TryReplacement(const Notification& notification, Profile* profile);
// Checks the user state to decide if we want to show the notification.
void CheckUserState();

Powered by Google App Engine
This is Rietveld 408576698