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

Side by Side Diff: LayoutTests/http/tests/notifications/notification-properties.html

Issue 1254423011: Add Notification.maxActions so websites know how many buttons we support (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@staticreadonlypeter
Patch Set: Address review nits 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 unified diff | Download patch
« no previous file with comments | « no previous file | Source/modules/notifications/Notification.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!doctype html> 1 <!doctype html>
2 <html> 2 <html>
3 <head> 3 <head>
4 <title>Notifications: The Notification object exposes the expected propertie s.</title> 4 <title>Notifications: The Notification object exposes the expected propertie s.</title>
5 <script src="../resources/testharness.js"></script> 5 <script src="../resources/testharness.js"></script>
6 <script src="../resources/testharnessreport.js"></script> 6 <script src="../resources/testharnessreport.js"></script>
7 </head> 7 </head>
8 <body> 8 <body>
9 <script> 9 <script>
10 // Tests that the Notification object exposes the properties per the 10 // Tests that the Notification object exposes the properties per the
11 // semantics defined by the specification. When the test is being ran 11 // semantics defined by the specification. When the test is being ran
12 // manually, grant Notification permission first. 12 // manually, grant Notification permission first.
13 test(function () { 13 test(function () {
14 assert_greater_than_equal(Notification.maxActions, 0);
15
14 var options = { 16 var options = {
15 dir: "rtl", 17 dir: "rtl",
16 lang: "nl-NL", 18 lang: "nl-NL",
17 body: "Hallo, wereld!", 19 body: "Hallo, wereld!",
18 tag: "notification", 20 tag: "notification",
19 icon: "http://localhost/my_icon.png", 21 icon: "http://localhost/my_icon.png",
20 silent: true, 22 silent: true,
21 data: "my data" 23 data: "my data"
22 }; 24 };
23 25
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 var notification = new Notification("My Notification", { 101 var notification = new Notification("My Notification", {
100 silent: true, 102 silent: true,
101 vibrate: 1000 103 vibrate: 1000
102 }); 104 });
103 }, 'Set vibrate, when silent is true.'); 105 }, 'Set vibrate, when silent is true.');
104 106
105 }, 'Checks the properties exposed on the Notification object.'); 107 }, 'Checks the properties exposed on the Notification object.');
106 </script> 108 </script>
107 </body> 109 </body>
108 </html> 110 </html>
OLDNEW
« no previous file with comments | « no previous file | Source/modules/notifications/Notification.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698