Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1118)

Unified Diff: content/public/browser/notification_event_dispatcher.h

Issue 1619703002: Implement notificationclose event (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed redundant TODO Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 bb3df945e2364e89cf3eb712087fe24ffe35b635..e84b9bc6e3bf3d37aebdebde2c793f7ba0ca64c6 100644
--- a/content/public/browser/notification_event_dispatcher.h
+++ b/content/public/browser/notification_event_dispatcher.h
@@ -41,6 +41,17 @@ class CONTENT_EXPORT NotificationEventDispatcher {
const NotificationClickDispatchCompleteCallback&
dispatch_complete_callback) = 0;
+ // Dispatches the "notificationclose" event on the Service Worker associated
+ // with |persistent_notification_id| belonging to |origin|. The |callback|
+ // will be invoked when it's known whether the event successfully executed.
+ virtual void DispatchNotificationCloseEvent(
+ BrowserContext* browser_context,
+ int64_t persistent_notification_id,
+ const GURL& origin,
+ const NotificationClickDispatchCompleteCallback&
+ dispatch_complete_callback) = 0;
+
Peter Beverloo 2016/01/21 14:17:37 micro nit: blank line too much.
Nina 2016/01/25 14:54:55 Done.
+
protected:
virtual ~NotificationEventDispatcher() {}
};

Powered by Google App Engine
This is Rietveld 408576698