Index: chrome/browser/notifications/sync_notifier/synced_notification.cc |
diff --git a/chrome/browser/notifications/sync_notifier/synced_notification.cc b/chrome/browser/notifications/sync_notifier/synced_notification.cc |
index cddced0d0dc35641cf3f0664cc7febacee87e3db..1b7d1c948e7497a2d4d4f1edb9e70d64afdc429e 100644 |
--- a/chrome/browser/notifications/sync_notifier/synced_notification.cc |
+++ b/chrome/browser/notifications/sync_notifier/synced_notification.cc |
@@ -120,8 +120,6 @@ bool SyncedNotification::AreAllBitmapsFetched() { |
// dpi bitmap when appropriate. |
void SyncedNotification::OnFetchComplete(const GURL url, |
const SkBitmap* bitmap) { |
- // TODO(petewil): Add timeout mechanism in case bitmaps take too long. Do we |
- // already have one built into URLFetcher? |
// Make sure we are on the thread we expect. |
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); |
@@ -386,6 +384,27 @@ void SyncedNotification::Show(NotificationUIManager* notification_manager, |
return; |
} |
+// Display the notification if it has the specified app_id_name. |
+void SyncedNotification::ShowIfNewlyEnabled( |
+ NotificationUIManager* notification_manager, |
+ ChromeNotifierService* notifier_service, |
+ Profile* profile, |
+ std::string app_id_name) { |
+ if (app_id_name == GetAppId()) |
+ Show(notification_manager, notifier_service, profile); |
+} |
+ |
+// Remove the notification if it has the specified app_id_name. |
+void SyncedNotification::HideIfNewlyRemoved( |
+ NotificationUIManager* notification_manager, |
+ ChromeNotifierService* notifier_service, |
+ Profile* profile, |
+ std::string app_id_name) { |
+ if (app_id_name == GetAppId()) { |
+ notification_manager->CancelById(GetKey()); |
+ } |
+} |
+ |
// This should detect even small changes in case the server updated the |
// notification. We ignore the timestamp if other fields match. |
bool SyncedNotification::EqualsIgnoringReadState( |
@@ -746,14 +765,6 @@ std::string SyncedNotification::GetContainedNotificationMessage( |
collapsed_info(index).simple_collapsed_layout().description(); |
} |
-std::string SyncedNotification::GetSendingServiceId() const { |
- // TODO(petewil): We are building a new protocol (a new sync datatype) to send |
- // the service name and icon from the server. For now this method is |
- // hardcoded to the name of our first service using synced notifications. |
- // Once the new protocol is built, remove this hardcoding. |
- return kFirstSyncedNotificationServiceId; |
-} |
- |
const gfx::Image& SyncedNotification::GetAppIcon() const { |
return app_icon_bitmap_; |
} |