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

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

Issue 1907443007: Use promises in notifications tests and enable controlling the page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Address peter's comments. Created 4 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 = 'instrumentation-service-worker.js';
18 18
19 testRunner.setPermission('notifications', 'granted', location.origin, location.origin); 19 testRunner.setPermission('notifications', 'granted', location.origin, location.origin);
20 getActiveServiceWorkerWithMessagePort(test, script, scope).then(functi on(info) { 20 getActiveServiceWorkerWithMessagePort(test, script, scope).then(functi on(info) {
21 // (1) Display a Web Notification with an image that 404s after a 200ms redirect. 21 // (1) Display a Web Notification with an image that 404s after a 200ms redirect.
22 return info.registration.showNotification(scope, { 22 return info.registration.showNotification(scope, {
23 body: 'Hello, world!', 23 body: 'Hello, world!',
24 icon: '/resources/slow-image.php?redirect_name=not_there.png&m imeType=image/png&sleep=200' 24 icon: '/resources/slow-image.php?redirect_name=not_there.png&m imeType=image/png&sleep=200'
25 }); 25 });
26 }).then(function() { 26 }).then(function() {
27 test.done(); 27 test.done();
28 }).catch(unreached_rejection(test)); 28 }).catch(unreached_rejection(test));
29 29
30 }, 'Displaying a notification with an image that 404s after a slow redirec t still resolves the promise.'); 30 }, 'Displaying a notification with an image that 404s after a slow redirec t still resolves the promise.');
31 </script> 31 </script>
32 </body> 32 </body>
33 </html> 33 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698