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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt

Issue 2242883002: [CacheStorage] Use QueryCache everywhere (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments from PS9 Created 4 years, 4 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: third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt
diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt b/third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt
index cfed7e6ed156acf1c44d5421f111d09dce077492..c1f964f46260d43b9d6ae1ee5b7dd091f452ca10 100644
--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt
+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/window/cache-matchAll-expected.txt
@@ -1,4 +1,3 @@
-CONSOLE WARNING: line 172: Cache.match() does not support 'ignoreVary' option yet. See http://crbug.com/499216
This is a testharness.js-based test.
PASS Cache.matchAll with no matching entries
PASS Cache.matchAll with URL
@@ -7,10 +6,12 @@ PASS Cache.matchAll with new Request
PASS Cache.matchAll with HEAD
PASS Cache.matchAll with ignoreSearch option (request with no search parameters)
PASS Cache.matchAll with ignoreSearch option (request with search parameters)
+PASS Cache.matchAll supports ignoreMethod
+PASS Cache.matchAll supports ignoreVary
PASS Cache.matchAll with URL containing fragment
PASS Cache.matchAll with string fragment "http" as query
PASS Cache.matchAll without parameters
FAIL Cache.matchAll with responses containing "Vary" header assert_equals: Cache.matchAll should match the entire header if a vary header is present in both the query and cached requests. expected 1 but got 0
-FAIL Cache.matchAll with "ignoreVary" parameter assert_equals: Cache.matchAll should honor "ignoreVary" parameter. expected 3 but got 1
+FAIL Cache.matchAll with multiple vary pairs assert_equals: Cache.matchAll should support multiple vary request/response pairs. expected 3 but got 1
Harness: the test ran to completion.

Powered by Google App Engine
This is Rietveld 408576698