Minimize the delegate dependencies for non persistent notifications.

content/public/browser/notification_event_dispatcher.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_PUBLIC_BROWSER_NOTIFICATION_EVENT_DISPATCHER_H_
 #define CONTENT_PUBLIC_BROWSER_NOTIFICATION_EVENT_DISPATCHER_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 14 matching lines @@
 // notifications on a Service Worker. This class is a singleton, the instance of
 // which can be retrieved using the static GetInstance() method. All methods
 // must be called on the UI thread.
 class CONTENT_EXPORT NotificationEventDispatcher {
  public:
   static NotificationEventDispatcher* GetInstance();
 
   using NotificationDispatchCompleteCallback =
       base::Callback<void(PersistentNotificationStatus)>;
 
+  // Dispatch methods for persistent (SW backed) notifications.
+  // TODO(miguelg) consider merging them with the non persistent ones below.
+
   // Dispatches the "notificationclick" event on the Service Worker associated
   // with |notification_id| belonging to |origin|. The |callback| will be
   // invoked when it's known whether the event successfully executed.
   virtual void DispatchNotificationClickEvent(
       BrowserContext* browser_context,
       const std::string& notification_id,
       const GURL& origin,
       int action_index,
       const base::NullableString16& reply,
       const NotificationDispatchCompleteCallback&
           dispatch_complete_callback) = 0;
 
   // Dispatches the "notificationclose" event on the Service Worker associated
   // with |notification_id| belonging to |origin|. The
   // |dispatch_complete_callback| will be invoked when it's known whether the
   // event successfully executed.
   virtual void DispatchNotificationCloseEvent(
       BrowserContext* browser_context,
       const std::string& notification_id,
       const GURL& origin,
       bool by_user,
       const NotificationDispatchCompleteCallback&
           dispatch_complete_callback) = 0;
 
+  // Dispatch methods for the different in-page (not backed by a service worker)
+  // notification events.

[Peter Beverloo, 2017/06/01 17:56:27]

Just call these "non-persistent", please don't diverge terminology even further.

[Miguel Garcia, 2017/06/02 12:37:13]

Done.

+  virtual void DispatchNonPersistentShowEvent(
+      const std::string& notification_id) = 0;
+  virtual void DispatchNonPersistentClickEvent(
+      const std::string& notification_id) = 0;
+  virtual void DispatchNonPersistentCloseEvent(
+      const std::string& notification_id) = 0;
+
  protected:
   virtual ~NotificationEventDispatcher() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NOTIFICATION_EVENT_DISPATCHER_H_