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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: chrome/test/data/notifications/platform_notification_service.html

Issue 1750083004: Add badge to web notifications. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test. Created 4 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/notifications/platform_notification_service_impl.cc ('k') | content/browser/notifications/notification_database_data.proto » ('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 lang="en"> 2 <html lang="en">
3 <head> 3 <head>
4 <meta charset="utf-8"> 4 <meta charset="utf-8">
5 <title>Platform Notification Service BrowserTest service page</title> 5 <title>Platform Notification Service BrowserTest service page</title>
6 </head> 6 </head>
7 <body> 7 <body>
8 <!-- This page is intended to be used by the cross-platform 8 <!-- This page is intended to be used by the cross-platform
9 PlatformNotificationServiceBrowserTest. --> 9 PlatformNotificationServiceBrowserTest. -->
10 <script src="notification_test_utils.js"></script> 10 <script src="notification_test_utils.js"></script>
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 // Displays a persistent notification having every field in its options 48 // Displays a persistent notification having every field in its options
49 // bag filled out with non-default values. 49 // bag filled out with non-default values.
50 function DisplayPersistentAllOptionsNotification() { 50 function DisplayPersistentAllOptionsNotification() {
51 DisplayPersistentNotification('Title', { 51 DisplayPersistentNotification('Title', {
52 dir: 'rtl', 52 dir: 'rtl',
53 lang: 'nl-NL', 53 lang: 'nl-NL',
54 body: 'Contents', 54 body: 'Contents',
55 tag: 'replace-id', 55 tag: 'replace-id',
56 icon: 'icon.png', 56 icon: 'icon.png',
57 badge: 'icon.png',
57 timestamp: 621046800000, 58 timestamp: 621046800000,
58 renotify: true, 59 renotify: true,
59 silent: true, 60 silent: true,
60 requireInteraction: true, 61 requireInteraction: true,
61 data: [ 62 data: [
62 { property: 'value' } 63 { property: 'value' }
63 ], 64 ],
64 actions: [ 65 actions: [
65 { action: 'actionId', title: 'actionTitle', icon: 'icon.png' } 66 { action: 'actionId', title: 'actionTitle', icon: 'icon.png' }
66 ] 67 ]
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 if (!messageStack.length) { 124 if (!messageStack.length) {
124 expectingMessage = true; 125 expectingMessage = true;
125 return; 126 return;
126 } 127 }
127 128
128 domAutomationController.send('' + messageStack.pop()); 129 domAutomationController.send('' + messageStack.pop());
129 } 130 }
130 </script> 131 </script>
131 </body> 132 </body>
132 </html> 133 </html>
OLDNEW
« no previous file with comments | « chrome/browser/notifications/platform_notification_service_impl.cc ('k') | content/browser/notifications/notification_database_data.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698