Allow partial update for notification update API

chrome/browser/notifications/notification_ui_manager.h

Index: chrome/browser/notifications/notification_ui_manager.h
diff --git a/chrome/browser/notifications/notification_ui_manager.h b/chrome/browser/notifications/notification_ui_manager.h
index 625793917a68a33c645d5b196e742acabf71f90d..68aee3aa578f71daababa20da00e0bf4236a82c1 100644
--- a/chrome/browser/notifications/notification_ui_manager.h
+++ b/chrome/browser/notifications/notification_ui_manager.h
@@ -30,8 +30,10 @@ class NotificationUIManager {
                    Profile* profile) = 0;
 
   // Returns true if any notifications match the supplied ID, either currently
-  // displayed or in the queue.
-  virtual bool DoesIdExist(const std::string& notification_id) = 0;
+  // displayed or in the queue. If |matched_notification| is not NULL, it
+  // will contain the matched notification upon return.
+  virtual bool DoesIdExist(const std::string& notification_id,
+                           Notification* matched_notification) = 0;

[dewittj, 2013/07/24 23:09:53]

Should we just return Notification* instead of bool?

[jianli, 2013/07/25 01:02:41]

Changed to return Notification pointer and renamed the method to FindById.

 
   // Removes any notifications matching the supplied ID, either currently
   // displayed or in the queue.  Returns true if anything was removed.