1922726 - [wpt-sync] Sync PR 48463 - Add cache-storage as a value of deliveryType in resource timing

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 48463 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/48463
Details from upstream follow.

Keita Suzuki <[e-mail]> wrote:

Add cache-storage as a value of deliveryType in resource timing

This CL adds a new value |cache-storage| to the deliveryType field in
resource timing. It is specified when the response comes from
CacheStorage either via Fetch handler in the ServiceWorker or from
the static routing API when the cache rule is specified and used.

Explainer: https://github.com/WICG/service-worker-static-routing-api/blob/main/resource-timing-api.md
Bug: 41496865
Change-Id: Iff62aa9394244f28d88cbc73a94ff87c6c3e56a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5901298
Commit-Queue: Keita Suzuki \<[e-mail]>
Reviewed-by: Takashi Toyoshima \<[e-mail]>
Reviewed-by: Shunya Shishido \<[e-mail]>
Reviewed-by: Yoshisato Yanagisawa \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1364026}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=596e28f12b4c5c04905f944d61c07698c1e49138

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 14 subtests

Status Summary

Firefox

OK : 1
FAIL: 14

Chrome

OK : 1
FAIL: 14

Safari

OK : 1
FAIL: 14

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /service-workers/service-worker/tentative/static-router/static-router-resource-timing.https.html [wpt.fyi]
  • Main resource matched the rule with fetch-event source: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource load matched with the condition and resource timing: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource load not matched with the condition and resource timing: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource load matched with the cache source and resource timing: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource fallback to the network when there is no cache entry and resource timing: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load matched the rule fetch-event source: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load not matched with URLPattern condition: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load matched with URLPattern condition: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load matched with the cache source rule: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load did not match with the cache and fallback to the network: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource load matched the rule with race-network-and-fetch-handler source, and the fetch handler response is faster than the server response: FAIL (Chrome: FAIL, Safari: FAIL)
  • Main resource load matched the rule with race-network-and-fetch-handler source, and the server reseponse is faster than the fetch handler: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load matched the rule with race-network-and-fetch-handler source, and the fetch handler response is faster than the server response: FAIL (Chrome: FAIL, Safari: FAIL)
  • Subresource load matched the rule with race-network-and-fetch-handler source, and the server reseponse is faster than the fetch handler: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6fd6dd71a968
[wpt PR 48463] - Add cache-storage as a value of deliveryType in resource timing, a=testonly

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/6fd6dd71a968