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

Side by Side Diff: LayoutTests/http/tests/notifications/serviceworkerregistration-document-image-slow-404.html

Issue 1097383006: Notifications: use testRunner.setPermission() in addition of grantWebNotificationPermission. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@cleanup
Patch Set: fix expectations Created 5 years, 8 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
OLDNEW
1 <!doctype html> 1 <!doctype html>
2 <html> 2 <html>
3 <head> 3 <head>
4 <title>Notifications: Showing a notification with an image that 404s after a slow redirect.</title> 4 <title>Notifications: Showing a notification with an image that 404s after a slow redirect.</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 <script src="../serviceworker/resources/test-helpers.js"></script> 7 <script src="../serviceworker/resources/test-helpers.js"></script>
8 <script src="resources/test-helpers.js"></script> 8 <script src="resources/test-helpers.js"></script>
9 </head> 9 </head>
10 <body> 10 <body>
11 <script> 11 <script>
12 // Tests that showing a notification with an image that creates a 404 resp onse 12 // Tests that showing a notification with an image that creates a 404 resp onse
13 // after a slow redirect works properly, and does not block the actual dis play of it. 13 // after a slow redirect works properly, and does not block the actual dis play of it.
14 14
15 async_test(function(test) { 15 async_test(function(test) {
16 var scope = 'resources/scope/' + location.pathname, 16 var scope = 'resources/scope/' + location.pathname,
17 script = 'resources/instrumentation-service-worker.js'; 17 script = 'resources/instrumentation-service-worker.js';
18 18
19 testRunner.setPermission('notifications', 'granted', location.origin, location.origin);
19 testRunner.grantWebNotificationPermission(location.origin, true); 20 testRunner.grantWebNotificationPermission(location.origin, true);
20 getActiveServiceWorkerWithMessagePort(test, script, scope).then(functi on(info) { 21 getActiveServiceWorkerWithMessagePort(test, script, scope).then(functi on(info) {
21 // (1) Display a Web Notification with an image that 404s after a 200ms redirect. 22 // (1) Display a Web Notification with an image that 404s after a 200ms redirect.
22 return info.registration.showNotification(scope, { 23 return info.registration.showNotification(scope, {
23 body: 'Hello, world!', 24 body: 'Hello, world!',
24 icon: '/resources/slow-image.php?redirect_name=not_there.png&m imeType=image/png&sleep=200' 25 icon: '/resources/slow-image.php?redirect_name=not_there.png&m imeType=image/png&sleep=200'
25 }); 26 });
26 }).then(function() { 27 }).then(function() {
27 test.done(); 28 test.done();
28 }).catch(unreached_rejection(test)); 29 }).catch(unreached_rejection(test));
29 30
30 }, 'Displaying a notification with an image that 404s after a slow redirec t still resolves the promise.'); 31 }, 'Displaying a notification with an image that 404s after a slow redirec t still resolves the promise.');
31 </script> 32 </script>
32 </body> 33 </body>
33 </html> 34 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698