CacheStorage: Add ignoreSearch option to cache.delete().

third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-delete.js

 if (self.importScripts) {
     importScripts('/resources/testharness.js');
     importScripts('/resources/testharness-helpers.js');
     importScripts('../resources/test-helpers.js');
 }
 
 var test_url = 'https://example.com/foo';
 
 // Construct a generic Request object. The URL is |test_url|. All other fields
 // are defaults.
@@ skipping 46 matching lines @@
 
 cache_test(function(cache) {
     return cache.delete(test_url)
       .then(function(result) {
           assert_false(result,
                        'Cache.delete should resolve with "false" if there ' +
                        'are no matching entries.');
         });
   }, 'Cache.delete with a non-existent entry');
 
-var cache_entries = {

[zino, 2016/02/16 06:45:44]

I'm not sure but these codes looks unnecessary.

[nhiroki, 2016/02/16 08:47:23]

Acknowledged.

-  a: {
-    request: new Request('http://example.com/abc'),
-    response: new Response('')
+prepopulated_cache_test(simple_entries, function(cache, entries) {
+    return cache.matchAll(entries.a_with_query.request,
+                          { ignoreSearch: true })
+      .then(function(result) {
+          assert_response_array_equivalent(
+            result,
+            [
+              entries.a.response,
+              entries.a_with_query.response
+            ]);
+          return cache.delete(entries.a_with_query.request,
+                              { ignoreSearch: true });

[nhiroki, 2016/02/16 08:47:23]

Can you add a test for the case where 'ignoreSearch: false' is explicitly
specified?

cache.delete(entries.a_with_query.request, { ignoreSearch: false })

[zino, 2016/02/16 09:23:09]

Done.

+        })
+      .then(function(result) {
+          return cache.matchAll(entries.a_with_query.request,
+                                { ignoreSearch: true });
+        })
+      .then(function(result) {
+          assert_response_array_equivalent(result, []);
+        });
   },
-
-  b: {
-    request: new Request('http://example.com/b'),
-    response: new Response('')
-  },
-
-  a_with_query: {
-    request: new Request('http://example.com/abc?q=r'),
-    response: new Response('')
-  }
-};
-
-function prepopulated_cache_test(test_function, description) {

[zino, 2016/02/16 06:45:44]

This test doesn't run actually. So, I removed it.

[nhiroki, 2016/02/16 08:47:23]

Acknowledged.

-  cache_test(function(cache) {
-      return Promise.all(Object.keys(cache_entries).map(function(k) {
-          return cache.put(cache_entries[k].request.clone(),
-                           cache_entries[k].response.clone());
-        }))
-        .then(function() {
-            return test_function(cache);
-          });
-    }, description);
-}
+  'Cache.delete with ignoreSearch option (request with search parameters)');
 
 done();