The 1st patch to disambiguate message center notifications

ui/message_center/notification.h

 // Copyright (c) 2013 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.
 
 #ifndef UI_MESSAGE_CENTER_NOTIFICATION_H_
 #define UI_MESSAGE_CENTER_NOTIFICATION_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 46 matching lines @@
   Notification(NotificationType type,
                const std::string& id,
                const base::string16& title,
                const base::string16& message,
                const gfx::Image& icon,
                const base::string16& display_source,
                const NotifierId& notifier_id,
                const RichNotificationData& optional_fields,
                NotificationDelegate* delegate);
 
+  Notification(const std::string& id, const Notification& other);

[dewittj, 2014/06/09 20:00:30]

by the way, are you using this new constructor in this patch?  If not, save it
for the next patch.

[juyik, 2014/06/09 21:28:08]

Done.

+
   Notification(const Notification& other);
+
   Notification& operator=(const Notification& other);
+
   virtual ~Notification();
 
   // Copies the internal on-memory state from |base|, i.e. shown_as_popup,
   // is_read, and never_timeout.
   void CopyState(Notification* base);
 
   NotificationType type() const { return type_; }
   void set_type(NotificationType type) { type_ = type; }
 
+  // Uniquely identifies a notification in the message center. For
+  // notification front ends that support multiple profiles, this id should
+  // identify a unique profile + frontend_notification_id combination. You can
+  // Use this id against the MessageCenter interface but not the
+  // NotificationUIManager interface.
   const std::string& id() const { return id_; }
 
   const base::string16& title() const { return title_; }
   void set_title(const base::string16& title) { title_ = title; }
 
   const base::string16& message() const { return message_; }
   void set_message(const base::string16& message) { message_ = message; }
 
   // A display string for the source of the notification.
   const base::string16& display_source() const { return display_source_; }
@@ skipping 129 matching lines @@
   bool is_read_;  // True if this has been seen in the message center.
 
   // A proxy object that allows access back to the JavaScript object that
   // represents the notification, for firing events.
   scoped_refptr<NotificationDelegate> delegate_;
 };
 
 }  // namespace message_center
 
 #endif  // UI_MESSAGE_CENTER_NOTIFICATION_H_