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

Unified Diff: chrome/browser/notifications/sync_notifier/chrome_notifier_service.h

Issue 22470006: Process incoming updates and deletes properly. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates and deletes: CR feedback from DewittJ and Zea Created 7 years, 4 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 | « no previous file | chrome/browser/notifications/sync_notifier/chrome_notifier_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/sync_notifier/chrome_notifier_service.h
diff --git a/chrome/browser/notifications/sync_notifier/chrome_notifier_service.h b/chrome/browser/notifications/sync_notifier/chrome_notifier_service.h
index fdbd7f14fa1ebbaf3c242309ef098b70754eec8f..61ff02b853c052025485d626f6b76f8b41b8f0e3 100644
--- a/chrome/browser/notifications/sync_notifier/chrome_notifier_service.h
+++ b/chrome/browser/notifications/sync_notifier/chrome_notifier_service.h
@@ -91,9 +91,15 @@ class ChromeNotifierService : public syncer::SyncableService,
// Add a notification to our list. This takes ownership of the pointer.
void Add(scoped_ptr<notifier::SyncedNotification> notification);
+ // Display this notification in the notification center, or remove it.
+ void UpdateInMessageCenter(notifier::SyncedNotification* notification);
+
// Display a notification in the notification center (eventually).
void Display(notifier::SyncedNotification* notification);
+ // Remove a notification from our store.
+ void FreeNotificationById(const std::string& notification_id);
+
// Back pointer to the owning profile.
Profile* const profile_;
NotificationUIManager* const notification_manager_;
« no previous file with comments | « no previous file | chrome/browser/notifications/sync_notifier/chrome_notifier_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698