Introduce a new API to handle native notification clicks

chrome/browser/notifications/message_center_display_service.cc

Index: chrome/browser/notifications/message_center_display_service.cc
diff --git a/chrome/browser/notifications/message_center_display_service.cc b/chrome/browser/notifications/message_center_display_service.cc
index 8dc7420cafe2b9fdc9811c8ff68980b7a12b6de2..d5a8cfd2a17408db82be68dea2e35a33d5ea2f57 100644
--- a/chrome/browser/notifications/message_center_display_service.cc
+++ b/chrome/browser/notifications/message_center_display_service.cc
@@ -15,12 +15,19 @@ MessageCenterDisplayService::MessageCenterDisplayService(
 
 MessageCenterDisplayService::~MessageCenterDisplayService() {}
 
-void MessageCenterDisplayService::Display(const std::string& notification_id,
-                                          const Notification& notification) {
+void MessageCenterDisplayService::Display(
+    NotificationCommon::Type notification_type,
+    const std::string& notification_id,
+    const Notification& notification) {
+  // TODO(miguelg): MCDS should stop relying on the
+  // |notification|'s delegate for Close/Click operations
+  // once the Notification object becomes a mojom type.

[Peter Beverloo, 2016/07/08 16:06:31]

nit: weird line wrapping

[Miguel Garcia, 2016/07/08 16:27:27]

Done.

   ui_manager_->Add(notification, profile_);
 }
 
-void MessageCenterDisplayService::Close(const std::string& notification_id) {
+void MessageCenterDisplayService::Close(
+    NotificationCommon::Type notification_type,
+    const std::string& notification_id) {
   ui_manager_->CancelById(notification_id,
                           NotificationUIManager::GetProfileID(profile_));
 }