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

Unified Diff: LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js

Issue 927723003: Removing prefixMatch attribute of CacheQueryOptions- Blink Side (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: After rebase Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js
diff --git a/LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js b/LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js
index 44b3d67f8409bbb78e9d94ce9900c1fa06e55c49..98204fddf96e73ec4f14155b08055a4e6833b0dc 100644
--- a/LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js
+++ b/LayoutTests/http/tests/serviceworker/resources/cache-delete-worker.js
@@ -114,44 +114,3 @@ function prepopulated_cache_test(test_function, description) {
});
}, description);
}
-
-// Note that these tests don't exercise the full gamut of CacheQueryOptions.
-// That's left for the Cache.match() and Cache.matchAll() tests. The objective
-// is to test that CacheQueryOptions, if specified, are used.
-
-prepopulated_cache_test(function(cache) {
- return cache.delete('http://example.com/a', { prefixMatch: true })
- .then(function(result) {
- assert_true(result,
- 'Cache.delete should resolve with "true" if an entry ' +
- 'was successfully deleted.');
- })
- .then(function() {
- return Promise.all(
- [].concat(
- // The entries 'a' and 'a_with_query' should have been deleted.
- ['a', 'a_with_query'].map(function(k) {
- return assert_promise_rejects(
- cache.match(cache_entries[k].request.url),
- 'NotFoundError',
- 'Cache.delete should respect "prefixMatch" option.');
- }),
- // The entry 'b' should still be in the cache.
- cache.match(cache_entries.b.request.url)
- .catch(function() {
- assert_unreached('Cache.delete should respect ' +
- '"prefixMatch" option.');
- })
- ));
- });
- }, 'Cache.delete with CacheQueryOptions.*');
-
-prepopulated_cache_test(function(cache) {
- return cache.delete('http://example.com/ac', { prefixMatch: true })
- .then(function(result) {
- assert_false(result,
- 'Cache.delete should resolve with "false" if there ' +
- 'are no matching entries.');
- });
- }, 'Cache.delete with CacheQueryOptions.* that don\'t match');
-

Powered by Google App Engine
This is Rietveld 408576698