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

Unified Diff: Source/modules/notifications/NotificationData.cpp

Issue 1309273007: Add requireInteraction to Notification & NotificationOptions (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@constructor
Patch Set: Add to NotificationDataTest.cpp Created 5 years, 4 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/NotificationData.cpp
diff --git a/Source/modules/notifications/NotificationData.cpp b/Source/modules/notifications/NotificationData.cpp
index 4615425367a9282dd32a8b7fddd8497f119b0a68..865c1a2f9106d952e3f34d977deb587df3730f0f 100644
--- a/Source/modules/notifications/NotificationData.cpp
+++ b/Source/modules/notifications/NotificationData.cpp
@@ -57,6 +57,7 @@ WebNotificationData createWebNotificationData(ExecutionContext* executionContext
webData.icon = iconUrl;
webData.vibrate = NavigatorVibration::sanitizeVibrationPattern(options.vibrate());
webData.silent = options.silent();
+ webData.requireInteraction = options.requireInteraction();
if (options.hasData()) {
RefPtr<SerializedScriptValue> serializedScriptValue = SerializedScriptValueFactory::instance().create(options.data().isolate(), options.data(), nullptr, exceptionState);
« no previous file with comments | « Source/modules/notifications/Notification.idl ('k') | Source/modules/notifications/NotificationDataTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698