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

Unified Diff: Source/modules/notifications/ServiceWorkerGlobalScopeNotifications.idl

Issue 720873002: Expose the notificationclick and notificationerror events on ServiceWorkerGlobalScope (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: comments and test Created 6 years, 1 month 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: Source/modules/notifications/ServiceWorkerGlobalScopeNotifications.idl
diff --git a/Source/modules/notifications/NotificationEvent.idl b/Source/modules/notifications/ServiceWorkerGlobalScopeNotifications.idl
similarity index 54%
copy from Source/modules/notifications/NotificationEvent.idl
copy to Source/modules/notifications/ServiceWorkerGlobalScopeNotifications.idl
index b089670e6e811fdb6fbba7a0eee7c132989691e7..b2a61e5dbf84558ee4342060c0b1121f1476d710 100644
--- a/Source/modules/notifications/NotificationEvent.idl
+++ b/Source/modules/notifications/ServiceWorkerGlobalScopeNotifications.idl
@@ -4,11 +4,9 @@
// https://notifications.spec.whatwg.org/#service-worker-api
-// FIXME: NotificationEvent should support being constructed from JavaScript.
[
- Exposed=ServiceWorker,
RuntimeEnabled=ServiceWorkerNotifications,
- TypeChecking=Interface,
-] interface NotificationEvent : ExtendableEvent {
- readonly attribute Notification notification;
+] partial interface ServiceWorkerGlobalScope {
+ attribute EventHandler onnotificationclick;
+ attribute EventHandler onnotificationerror;
};

Powered by Google App Engine
This is Rietveld 408576698