Turn on and use the AppInfo data.

chrome/browser/notifications/sync_notifier/synced_notification.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This class represents the data for a single Synced Notification.
 // It should map 1-1 to all the data in synced_notification_sepcifics.proto,
 // and the data and render protobufs that the specifics protobuf contains.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_H_
 #define CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_H_
@@ skipping 56 matching lines @@
   GURL GetDefaultDestinationUrl() const;
   std::string GetButtonTitle(unsigned int which_button) const;
   GURL GetButtonIconUrl(unsigned int which_button) const;
   GURL GetButtonUrl(unsigned int which_button) const;
   GURL GetProfilePictureUrl(unsigned int which_url) const;
   size_t GetProfilePictureCount() const;
   size_t GetNotificationCount() const;
   size_t GetButtonCount() const;
   std::string GetContainedNotificationTitle(int index) const;
   std::string GetContainedNotificationMessage(int index) const;
-  std::string GetSendingServiceId() const;
   const gfx::Image& GetAppIcon() const;
 
   // Use this to prevent toasting a notification.
   void SetToastState(bool toast_state);
 
   bool EqualsIgnoringReadState(const SyncedNotification& other) const;
 
   void NotificationHasBeenRead();
   void NotificationHasBeenDismissed();
 
   // Fill up the queue of bitmaps to fetch.
   void QueueBitmapFetchJobs(NotificationUIManager* notification_manager,
                             ChromeNotifierService* notifier_service,
                             Profile* profile);
 
   // Start the bitmap fetching.  When it is complete, the callback
   // will call Show().
   void StartBitmapFetch();
 
   // Display the notification in the notification center
   void Show(NotificationUIManager* notification_manager,
             ChromeNotifierService* notifier_service,
             Profile* profile);
 
+  // Display the notification if it has the specified app_id_name.
+  void ShowIfNewlyEnabled(NotificationUIManager* notification_manager,

[dewittj, 2014/03/17 21:43:43]

Can you explain why you sometimes use the stored NotificationUIManager pointer
and sometimes use a function argument?  The inconsistency is really confusing,
especially since it's basically a singleton anyway.

[Pete Williamson, 2014/03/21 01:22:31]

It was to make the code easier to test by putting the dependency on the method
instead of the object.  I added a public setter as the normal way to set this.

+                          ChromeNotifierService* notifier_service,

[dewittj, 2014/03/17 21:43:43]

Same.

[Pete Williamson, 2014/03/21 01:22:31]

Done.

+                          Profile* profile,
+                          std::string app_id_name);
+
+  // Remove the notification if it has the specified app_id_name.
+  void HideIfNewlyRemoved(NotificationUIManager* notification_manager,

[dewittj, 2014/03/17 21:43:43]

Wouldn't this be better named HideAllForAppID?

[Pete Williamson, 2014/03/21 01:22:31]

Done (also renamed ShowIfNewlyEnabled -> ShowAllForAppId.

Thanks! I was having a hard time finding a good name, the one you suggested is
clearly better.

+                          ChromeNotifierService* notifier_service,

[dewittj, 2014/03/17 21:43:43]

Argument unused, please remove.

[Pete Williamson, 2014/03/21 01:22:31]

Done.

+                          Profile* profile,

[dewittj, 2014/03/17 21:43:43]

Same.

[Pete Williamson, 2014/03/21 01:22:31]

Done.

+                          std::string app_id_name);
+
   // This gets a pointer to the SyncedNotificationSpecifics part
   // of the sync data.
   sync_pb::EntitySpecifics GetEntitySpecifics() const;
 
   // Write a notification to the console log.
   void LogNotification();
 
  private:
   // Helper function to mark a notification as read or dismissed.
   void SetReadState(const ReadState& read_state);
@@ skipping 15 matching lines @@
   ScopedVector<chrome::BitmapFetcher> fetchers_;
   gfx::Image app_icon_bitmap_;
   gfx::Image sender_bitmap_;
   gfx::Image image_bitmap_;
   std::vector<gfx::Image> button_bitmaps_;
   bool app_icon_bitmap_fetch_pending_;
   bool sender_bitmap_fetch_pending_;
   bool image_bitmap_fetch_pending_;
   std::vector<bool> button_bitmaps_fetch_pending_;
 
-
   friend class SyncedNotificationTest;
 
   FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, AddBitmapToFetchQueueTest);
   FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, OnFetchCompleteTest);
   FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, QueueBitmapFetchJobsTest);
   FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, EmptyBitmapTest);
+  FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, ShowIfNewlyEnabledTest);
+  FRIEND_TEST_ALL_PREFIXES(SyncedNotificationTest, HideIfNewlyRemovedTest);
 
   DISALLOW_COPY_AND_ASSIGN(SyncedNotification);
 };
 
 }  // namespace notifier
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_SYNCED_NOTIFICATION_H_