Introduce a new API to handle native notification clicks

chrome/browser/notifications/in_page_notification_handler.h

Index: chrome/browser/notifications/in_page_notification_handler.h
diff --git a/chrome/browser/notifications/in_page_notification_handler.h b/chrome/browser/notifications/in_page_notification_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..92e0999123769afde2cd8711d4318275050d2dd0
--- /dev/null
+++ b/chrome/browser/notifications/in_page_notification_handler.h
@@ -0,0 +1,47 @@
+// Copyright 2016 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 CHROME_BROWSER_NOTIFICATIONS_IN_PAGE_NOTIFICATION_HANDLER_H_
+#define CHROME_BROWSER_NOTIFICATIONS_IN_PAGE_NOTIFICATION_HANDLER_H_
+
+#include <memory>
+#include <unordered_map>
+
+#include "base/macros.h"
+#include "chrome/browser/notifications/notification_handler.h"
+
+class Notification;
+
+// NotificationHandler implementation for non persistent
+// notifications.
+class InPageNotificationHandler : public NotificationHandler {

[Peter Beverloo, 2016/06/29 00:23:15]

Please be consistent in your terminology. We have persistent and non-persistent
Web Notifications. I would argue that, at this level, we should even emphasize
on the fact that they're *Web* Notifications.

[Miguel Garcia, 2016/07/05 13:54:50]

Renamed to NonPersistentNotificationHandler Adding Web there would make it even
longer so I am shying away from it.

+ public:
+  InPageNotificationHandler();
+  ~InPageNotificationHandler() override;
+
+  void HandleNotificationOperation(
+      notification_operation_common::NotificationOperation operation,
+      Profile* profile,
+      const std::string& origin,
+      const std::string& notification_id,
+      int action_index) override;
+
+  void RegisterNotification(
+      const std::string& notification_id,
+      std::unique_ptr<Notification> notification) override;
+
+  void CloseNotification(Profile* profile,
+                         const std::string& notification_id) override;
+
+  notification_operation_common::NotificationHandlerType NotificationType()
+      override;
+
+ private:
+  // map of in page Notification objects keyed by notification id.
+  std::unordered_map<std::string, std::unique_ptr<Notification>> notifications_;
+
+  DISALLOW_COPY_AND_ASSIGN(InPageNotificationHandler);
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_IN_PAGE_NOTIFICATION_HANDLER_H_