[CacheStorage] Sort QueryCache results by time entered into cache

third_party/WebKit/LayoutTests/http/tests/cachestorage/resources/test-helpers.js

 (function() {
   var next_cache_index = 1;
 
   // Returns a promise that resolves to a newly created Cache object. The
   // returned Cache will be destroyed when |test| completes.
   function create_temporary_cache(test) {
     var uniquifier = String(++next_cache_index);
     var cache_name = self.location.pathname + '/' + uniquifier;
 
     test.add_cleanup(function() {
@@ skipping 123 matching lines @@
     name: 'vary_cookie_absent',
     request: new Request('http://example.com/c'),
     response: new Response('',
                            {headers: {'Vary': 'Cookies'}})
   }
 ];
 
 // Run |test_function| with a Cache object and a map of entries. Prior to the
 // call, the Cache is populated by cache entries from |entries|. The latter is
 // expected to be an Object mapping arbitrary keys to objects of the form
-// {request: <Request object>, response: <Response object>}. There's no
-// guarantee on the order in which entries will be added to the cache.
+// {request: <Request object>, response: <Response object>}. The entries are
+// added sequentially so that tests can verify the ordering of the cache
+// methods.
 //
 // |test_function| should return a Promise that can be used with promise_test.
 function prepopulated_cache_test(entries, test_function, description) {
   cache_test(function(cache) {
       var p = Promise.resolve();

[nhiroki, 2016/09/08 03:54:56]

Is this used?

[jkarlin, 2016/09/08 12:03:13]

Nope, thanks!

       var hash = {};
-      return Promise.all(entries.map(function(entry) {
+      var resolveMethod = null;
+
+      var promise = new Promise(function(resolve, reject) {
+          resolveMethod = resolve;
+        })
+      .then(function() {
+          assert_equals(Object.keys(hash).length, entries.length);
+        })
+      .then(function() {
+          return test_function(cache, hash);

[nhiroki, 2016/09/08 03:54:56]

You could combine this '.then' with the previous '.then'.

.then(function() {
    assert_equals(Object.keys(hash).length, entries.length);
    return test_function(cache, hash);
  })

[jkarlin, 2016/09/08 12:03:13]

Done.

+        });
+
+      // Add the entries to the cache sequentially.
+      // TODO(jkarlin): Once async/await is available use it to prettify this
+      // code.
+      var i = 0;
+      var processNextEntry = function(i) {
+          if (i == entries.length) {
+            resolveMethod();
+            return;
+          }
+          entry = entries[i];
           hash[entry.name] = entry;
-          return cache.put(entry.request.clone(),
-                           entry.response.clone())
-            .catch(function(e) {
-                assert_unreached(
-                  'Test setup failed for entry ' + entry.name + ': ' + e);
-            });
-        }))
-        .then(function() {
-            assert_equals(Object.keys(hash).length, entries.length);
-        })
-        .then(function() {
-            return test_function(cache, hash);
-        });
+          cache.put(entry.request.clone(), entry.response.clone())
+          .then(function() {
+              processNextEntry(i+1);
+            })
+          .catch(function(e) {
+              assert_unreached(
+              'Test setup failed for entry ' + entry.name + ': ' + e);

[nhiroki, 2016/09/08 03:54:56]

+2-spaces indent for an argument?

[jkarlin, 2016/09/08 12:03:13]

Done.

+            })
+        }
+
+      processNextEntry(0);
+      return promise;
     }, description);
 }
 
 // Helper for testing with Headers objects. Compares Headers instances
 // by serializing |expected| and |actual| to arrays and comparing.
 function assert_header_equals(actual, expected, description) {
     assert_class_string(actual, "Headers", description);
     var header;
     var actual_headers = [];
     var expected_headers = [];
@@ skipping 10 matching lines @@
 // does not compare the response bodies.
 function assert_response_equals(actual, expected, description) {
     assert_class_string(actual, "Response", description);
     ["type", "url", "status", "ok", "statusText"].forEach(function(attribute) {
         assert_equals(actual[attribute], expected[attribute],
                       description + " Attributes differ: " + attribute + ".");
     });
     assert_header_equals(actual.headers, expected.headers, description);
 }
 
-// Assert that the two arrays |actual| and |expected| contain the same
-// set of Responses as determined by assert_response_equals. The order
-// is not significant.
-//
-// |expected| is assumed to not contain any duplicates.
-function assert_response_array_equivalent(actual, expected, description) {
-    assert_true(Array.isArray(actual), description);
-    assert_equals(actual.length, expected.length, description);
-    expected.forEach(function(expected_element) {
-        // assert_response_in_array treats the first argument as being
-        // 'actual', and the second as being 'expected array'. We are
-        // switching them around because we want to be resilient
-        // against the |actual| array containing duplicates.
-        assert_response_in_array(expected_element, actual, description);
-    });
-}
-
 // Asserts that two arrays |actual| and |expected| contain the same
 // set of Responses as determined by assert_response_equals(). The
 // corresponding elements must occupy corresponding indices in their
 // respective arrays.
 function assert_response_array_equals(actual, expected, description) {
     assert_true(Array.isArray(actual), description);
     assert_equals(actual.length, expected.length, description);
     actual.forEach(function(value, index) {
         assert_response_equals(value, expected[index],
                                description + " : object[" + index + "]");
@@ skipping 60 matching lines @@
 function assert_request_in_array(actual, expected_array, description) {
     assert_true(expected_array.some(function(element) {
         try {
             assert_request_equals(actual, element);
             return true;
         } catch (e) {
             return false;
         }
     }), description);
 }