Minimize the delegate dependencies for non persistent notifications.

content/public/browser/notification_event_dispatcher.h

Index: content/public/browser/notification_event_dispatcher.h
diff --git a/content/public/browser/notification_event_dispatcher.h b/content/public/browser/notification_event_dispatcher.h
index ce2261b3044b878d67a1110329b87ce2ecf792f5..2687ca1de2514f4e723959d93b4086f6dd7cbefc 100644
--- a/content/public/browser/notification_event_dispatcher.h
+++ b/content/public/browser/notification_event_dispatcher.h
@@ -32,6 +32,9 @@ class CONTENT_EXPORT NotificationEventDispatcher {
   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.
@@ -56,6 +59,15 @@ class CONTENT_EXPORT NotificationEventDispatcher {
       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() {}
 };