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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js

Issue 1485283002: Sync Cache API tests w/ web-platform-tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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: third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js
index 480109005f48b5f7600db59868437f5e0c83f32c..287b4c3792215e0a283d2c55d9482d7029a20e05 100644
--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js
+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/script-tests/cache-matchAll.js
@@ -16,24 +16,26 @@ prepopulated_cache_test(simple_entries, function(cache, entries) {
prepopulated_cache_test(simple_entries, function(cache, entries) {
return cache.matchAll(entries.a.request.url)
.then(function(result) {
- assert_response_array_equivalent(result, [entries.a.response],
- 'Cache.matchAll should match by URL.');
+ assert_response_array_equals(result, [entries.a.response],
+ 'Cache.matchAll should match by URL.');
});
}, 'Cache.matchAll with URL');
prepopulated_cache_test(simple_entries, function(cache, entries) {
return cache.matchAll(entries.a.request)
.then(function(result) {
- assert_response_array_equivalent(result, [entries.a.response],
- 'Cache.matchAll should match by Request.');
+ assert_response_array_equals(
+ result, [entries.a.response],
+ 'Cache.matchAll should match by Request.');
});
}, 'Cache.matchAll with Request');
prepopulated_cache_test(simple_entries, function(cache, entries) {
return cache.matchAll(new Request(entries.a.request.url))
.then(function(result) {
- assert_response_array_equivalent(result, [entries.a.response],
- 'Cache.matchAll should match by Request.');
+ assert_response_array_equals(
+ result, [entries.a.response],
+ 'Cache.matchAll should match by Request.');
});
}, 'Cache.matchAll with new Request');
@@ -107,7 +109,6 @@ prepopulated_cache_test(vary_entries, function(cache, entries) {
assert_response_array_equivalent(
result,
[
- entries.vary_wildcard.response,
entries.vary_cookie_absent.response
],
'Cache.matchAll should exclude matches if a vary header is ' +
@@ -124,7 +125,6 @@ prepopulated_cache_test(vary_entries, function(cache, entries) {
assert_response_array_equivalent(
result,
[
- entries.vary_wildcard.response
],
'Cache.matchAll should exclude matches if a vary header is ' +
'missing in the cached request, but is present in the query ' +
@@ -154,8 +154,7 @@ prepopulated_cache_test(vary_entries, function(cache, entries) {
[
entries.vary_cookie_is_cookie.response,
entries.vary_cookie_is_good.response,
- entries.vary_cookie_absent.response,
- entries.vary_wildcard.response
+ entries.vary_cookie_absent.response
],
'Cache.matchAll should honor "ignoreVary" parameter.');
});

Powered by Google App Engine
This is Rietveld 408576698