[wpt-sync] Sync PR 15112 - cache storage: Introduce MultiCacheQueryOptions as per spec
Categories
(Testing :: web-platform-tests, enhancement, P4)
Tracking
(firefox67 fixed)
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 15112 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/15112
Details from upstream follow.
Amos Lim <eui-sang.lim@samsung.com> wrote:
cache storage: Introduce MultiCacheQueryOptions as per spec
MultiCacheQueryOptions inherits CacheQueryOptions and contains cacheName1.
MultiCacheQueryOptions in match of CacheStorage replaces CacheQueryOptions.
On the other hand, match in Cache takes CacheQueryOptions without cacheName.It was added in https://github.com/w3c/ServiceWorker/pull/1341
Bug: 919077
Change-Id: I86b68fe9197dacb431f989fa3958daebe0fd1dba
Reviewed-on: https://chromium-review.googlesource.com/c/1395998
Reviewed-by: Kinuko Yasuda \<kinuko@chromium.org>
Reviewed-by: Kentaro Hara \<haraken@chromium.org>
Reviewed-by: Hiroki Nakagawa \<nhiroki@chromium.org>
Reviewed-by: Ben Kelly \<wanderview@chromium.org>
Commit-Queue: Amos Lim \<eui-sang.lim@samsung.com>
Cr-Commit-Position: refs/heads/master@{#628245}
|
Assignee | |
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=33fd16df3554c6f35a6e150389605eb44d23d04c
|
|
Assignee | |
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=e75d1e470456844d9f54a2b1f4865a999eb05654
|
|
Assignee | |
Comment 3•5 years ago
|
||
Ran 3 tests and 67 subtests
OK : 3
PASS : 61
FAIL : 6
New tests that have failures or other problems:
/service-workers/cache-storage/serviceworker/cache-match.https.html
Cache.match does not support cacheName option: FAIL
/service-workers/cache-storage/window/cache-match.https.html
Cache.match does not support cacheName option: FAIL
/service-workers/cache-storage/worker/cache-match.https.html
Cache.match does not support cacheName option: FAIL|
|
Assignee | |
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=2dd33ddaab50ee9cfef7d5c52c4f9e9c4fc44355
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d089908c83f0 [wpt PR 15112] - cache storage: Introduce MultiCacheQueryOptions as per spec, a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/a778c323510e [wpt PR 15112] - Update wpt metadata, a=testonly
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/d089908c83f0
https://hg.mozilla.org/mozilla-central/rev/a778c323510e
Description
•