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

Side by Side Diff: LayoutTests/http/tests/serviceworker/getregistrations.html

Issue 1211623005: Unify multiple sequential_promise_test definitions. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: update fetch templates and remove new duplicate of this method Created 5 years, 6 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 <title>Service Worker: getRegistrations()</title> 2 <title>Service Worker: getRegistrations()</title>
3 <script src="../resources/testharness.js"></script> 3 <script src="../resources/testharness.js"></script>
4 <script src="../resources/testharnessreport.js"></script> 4 <script src="../resources/testharnessreport.js"></script>
5 <script src="../resources/testharness-helpers.js"></script>
5 <script src="resources/test-helpers.js"></script> 6 <script src="resources/test-helpers.js"></script>
6 <script src="../resources/get-host-info.js"></script> 7 <script src="../resources/get-host-info.js"></script>
7 <script src="../fetch/resources/fetch-test-helpers.js"></script>
8 <script> 8 <script>
9 // Purge the existing registrations for the origin. 9 // Purge the existing registrations for the origin.
10 // getRegistrations() is used in order to avoid adding additional complexity 10 // getRegistrations() is used in order to avoid adding additional complexity
11 // e.g. adding an internal function. 11 // e.g. adding an internal function.
12 sequential_promise_test(function(t) { 12 sequential_promise_test(function(t) {
13 return navigator.serviceWorker.getRegistrations() 13 return navigator.serviceWorker.getRegistrations()
14 .then(function(registrations) { 14 .then(function(registrations) {
15 return registrations.reduce(function(sequence, registration) { 15 return registrations.reduce(function(sequence, registration) {
16 return sequence.then(function() { 16 return sequence.then(function() {
17 return registration.unregister(); 17 return registration.unregister();
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 }; 132 };
133 frame.contentWindow.postMessage('unregister', '*', [channel.port2]); 133 frame.contentWindow.postMessage('unregister', '*', [channel.port2]);
134 return p; 134 return p;
135 }) 135 })
136 .then(function() { 136 .then(function() {
137 frame.remove(); 137 frame.remove();
138 return service_worker_unregister(t, scope); 138 return service_worker_unregister(t, scope);
139 }); 139 });
140 }, 'getRegistrations promise resolves only with same origin registrations.'); 140 }, 'getRegistrations promise resolves only with same origin registrations.');
141 141
142 sequential_promise_test_done();
143 done(); 142 done();
144 </script> 143 </script>
OLDNEW
« no previous file with comments | « LayoutTests/http/tests/resources/testharness-helpers.js ('k') | LayoutTests/resources/testharness-helpers.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698