Implement notificationclose event

content/browser/notifications/notification_event_dispatcher_impl.h

 // Copyright 2014 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 CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_EVENT_DISPATCHER_IMPL_H_
 #define CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_EVENT_DISPATCHER_IMPL_H_
 
 #include <stdint.h>
 
 #include "base/macros.h"
 #include "base/memory/singleton.h"
+#include "content/public/browser/notification_database_data.h"
 #include "content/public/browser/notification_event_dispatcher.h"
 
 namespace content {
 
+class PlatformNotificationContext;
+class ServiceWorkerRegistration;
+
 class NotificationEventDispatcherImpl : public NotificationEventDispatcher {
  public:
   // Returns the instance of the NotificationEventDispatcherImpl. Must be called
   // on the UI thread.
   static NotificationEventDispatcherImpl* GetInstance();
 
+  using NotificationActionCallback =
+      base::Callback<void(const ServiceWorkerRegistration*,
+                          const NotificationDatabaseData&)>;
+
+  using CurriedNotificationActionCallback =
+      base::Callback<void(scoped_refptr<PlatformNotificationContext>,
+                          const ServiceWorkerRegistration*,
+                          const NotificationDatabaseData&)>;
+
   // NotificationEventDispatcher implementation.
   void DispatchNotificationClickEvent(
       BrowserContext* browser_context,
       int64_t persistent_notification_id,
       const GURL& origin,
       int action_index,
-      const NotificationClickDispatchCompleteCallback&
-          dispatch_complete_callback) override;
+      const NotificationDispatchCompleteCallback& dispatch_complete_callback)
+      override;
+  void DispatchNotificationCloseEvent(
+      BrowserContext* browser_context,
+      int64_t persistent_notification_id,
+      const GURL& origin,
+      const NotificationDispatchCompleteCallback& dispatch_complete_callback)
+      override;
 
  private:
   NotificationEventDispatcherImpl();
   ~NotificationEventDispatcherImpl() override;
 
+  // Dispatches any notification event. The actual, specific event dispatch
+  // should be done by the |notification_action_callback|.
+  void DispatchNotificationEvent(

[Peter Beverloo, 2016/01/26 16:13:45]

Is there a reason that we'd like to expose this in the header? It could be an
anonymous function in the implementation (and with it, the typedefs + additional
forward declarations).

[Nina, 2016/01/27 18:48:58]

No reason - fixed.

+      BrowserContext* browser_context,
+      int64_t persistent_notification_id,
+      const GURL& origin,
+      const CurriedNotificationActionCallback& notification_action_callback,
+      const NotificationDispatchCompleteCallback& notification_error_callback);
+
   friend struct base::DefaultSingletonTraits<NotificationEventDispatcherImpl>;
 
   DISALLOW_COPY_AND_ASSIGN(NotificationEventDispatcherImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_EVENT_DISPATCHER_IMPL_H_