Ensure that Service Worker clients are always returned in MRU order (2)

LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js

Index: LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js
diff --git a/LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js b/LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js
index be69c4b23e3eb37716046ea8f58e581d5b0dc981..dfa5c277df80f1c738a8b622ddd398457904a7a5 100644
--- a/LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js
+++ b/LayoutTests/http/tests/serviceworker/resources/clients-matchall-worker.js
@@ -1,6 +1,7 @@
 self.onmessage = function(e) {
   var port = e.data.port;
   var options = e.data.options;
+  var retainOriginalOrder = e.data.retainOriginalOrder;
 
   self.clients.matchAll(options).then(function(clients) {
       var message = [];
@@ -11,7 +12,8 @@ self.onmessage = function(e) {
                         client.frameType]);
         });
       // Sort by url
-      message.sort(function(a, b) { return a[2] > b[2] ? 1 : -1; });
+      if (!retainOriginalOrder)
+        message.sort(function(a, b) { return a[2] > b[2] ? 1 : -1; });

[falken, 2015/08/17 07:10:19]

This extra flag is kind of tricky, I'd just remove the sorting from this
function and let the caller deal with whether they want to sort it or not. It
looks like the only other test using this
clients-matchall-include-uncontrolled.html is already sorting the expected urls
so it's natural to also sort the actual urls.

       port.postMessage(message);
     });
 };