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

Side by Side Diff: LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js

Issue 1286123004: Ensure that Service Worker clients are always returned in MRU order (2) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Restructure tests for readability. 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 | « LayoutTests/http/tests/serviceworker/clients-matchall-include-uncontrolled.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 self.onmessage = function(e) { 1 self.onmessage = function(e) {
2 var port = e.data.port; 2 var port = e.data.port;
3 var options = e.data.options; 3 var options = e.data.options;
4 4
5 self.clients.matchAll(options).then(function(clients) { 5 self.clients.matchAll(options).then(function(clients) {
6 var message = []; 6 var message = [];
7 clients.forEach(function(client) { 7 clients.forEach(function(client) {
8 message.push([client.visibilityState, 8 message.push([client.visibilityState,
9 client.focused, 9 client.focused,
10 client.url, 10 client.url,
11 client.frameType]); 11 client.frameType]);
12 }); 12 });
13 // Sort by url
14 message.sort(function(a, b) { return a[2] > b[2] ? 1 : -1; });
15 port.postMessage(message); 13 port.postMessage(message);
16 }); 14 });
17 }; 15 };
OLDNEW
« no previous file with comments | « LayoutTests/http/tests/serviceworker/clients-matchall-include-uncontrolled.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698