Make Cache Storage tests resilient to execution order

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

Index: third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-storage.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-storage.js b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-storage.js
index 673cf50f01ca85146983862c9de6bb81967c6002..faf5f6d2fc55c8a367322f1241629e9629e8d418 100644
--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-storage.js
+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-storage.js
@@ -204,8 +204,14 @@ promise_test(function(t) {
 
 promise_test(function(t) {
     var bad_name = 'unpaired\uD800';

[jkarlin, 2016/11/22 15:08:34]

bad_name is very confusing. It makes me think that it's a name that shouldn't
exist. Perhaps call it unpaired_name instead?

[jsbell, 2016/11/22 18:02:39]

Agreed, will do.

-    var converted_name = 'unpaired\uFFFD'; // Don't create cache with this name.
-    return self.caches.has(converted_name)
+    var converted_name = 'unpaired\uFFFD';
+    return Promise.all([
+        self.caches.delete(bad_name),
+        self.caches.delete(converted_name)

[jkarlin, 2016/11/22 15:08:34]

How was this issue (not deleting the caches) not causing flakiness before?

[jsbell, 2016/11/22 18:02:39]

This test runs 3 times (as window, worker, and serviceworker). Normally these
are sharded into 3 different processes, but if the order is randomized they can
end up in the same process; or if you turn off sharding you can make it fail:

run-webkit-tests --no-retry-failures --child-processes=1
http/tests/cachestorage/*/cache-storage.html

[jkarlin, 2016/11/22 18:05:47]

Acknowledged.

+      ])
+      .then(function() {
+          return self.caches.has(converted_name);
+      })
       .then(function(cache_exists) {
           assert_false(cache_exists,
                        'Test setup failure: cache should not exist');