third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt - Issue 1578363009: CacheStorage: Add ignoreSearch option to cache.matchAll().

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 1578363009: CacheStorage: Add ignoreSearch option to cache.matchAll(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: rebase Created 4 years, 10 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

« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-match-expected.txt ('k') | third_party/WebKit/Source/modules/cachestorage/Cache.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt

diff --git a/third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt b/third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt

index b5d627119e5bdc38d2f5a6c1bb2267cba3c62541..619debf316a628c375a9f9480ab0aaea04af5eda 100644

--- a/third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt

+++ b/third_party/WebKit/LayoutTests/http/tests/cachestorage/worker/cache-matchAll-expected.txt

@@ -1,13 +1,11 @@

-CONSOLE WARNING: Cache.match() does not support 'ignoreSearch' option yet. See http://crbug.com/520784

CONSOLE WARNING: 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

PASS Cache.matchAll with Request

PASS Cache.matchAll with new Request

-FAIL Cache.matchAll with ignoreSearch option (request with no search parameters) assert_equals: Cache.matchAll with ignoreSearch should ignore the search parameters of cached request. expected 2 but got 1

-FAIL Cache.matchAll with ignoreSearch option (request with search parameter) assert_equals: Cache.matchAll with ignoreSearch should ignore the search parameters of request. expected 2 but got 1

+PASS Cache.matchAll with ignoreSearch option (request with no search parameters)

+PASS Cache.matchAll with ignoreSearch option (request with search parameters)

PASS Cache.matchAll with URL containing fragment

PASS Cache.matchAll with string fragment "http" as query

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