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

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

Issue 1042513002: Add the vibrate attribute to the Notification object (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 9 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: Source/modules/notifications/NotificationOptions.idl
diff --git a/Source/modules/notifications/NotificationOptions.idl b/Source/modules/notifications/NotificationOptions.idl
index 716ce68f4ea9ae8c1bc3eadbb29dbcf4536f7859..c4df71cc6b84d54fe69e79328ce53f8d8d98d24a 100644
--- a/Source/modules/notifications/NotificationOptions.idl
+++ b/Source/modules/notifications/NotificationOptions.idl
@@ -10,6 +10,8 @@ enum NotificationDirection {
"rtl"
};
+typedef (unsigned long or sequence<unsigned long>) VibratePattern;
Peter Beverloo 2015/04/01 14:26:29 Why can't we share this definition with the Vibrat
timvolodine 2015/04/01 16:27:17 +1
Sanghyun Park 2015/04/01 17:46:08 Unfortunately, NavigatorVibration.idl did not supp
+
dictionary NotificationOptions {
NotificationDirection dir = "auto";
DOMString lang = "";
@@ -17,5 +19,6 @@ dictionary NotificationOptions {
DOMString tag = "";
DOMString icon;
boolean silent = false;
+ [RuntimeEnabled=NotificationExperimental] VibratePattern vibrate;
[RuntimeEnabled=NotificationExperimental] any data = null;
};

Powered by Google App Engine
This is Rietveld 408576698