Closed Bug 1619880 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 22080 - Make PauseAfterDownload() work for COEP with a new service worker

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla76
Tracking Flags:
Tracking Status
firefox76 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Makoto Shimazu <shimazu@chromium.org> wrote:

Make PauseAfterDownload() work for COEP with a new service worker

Currently Cross-Origin-Embedder-Policy header is parsed in
ServiceWorkerNewScriptLoader when launching a new service worker.
However, the COEP in the response header for the main script isn't
applied for the network loader created on launching a service worker
because the network loader is created before starting to load the main
script.
This CL is a preparation to make that work. PauseAfterDownload() is
going to be used to pause the service worker's script evaluation until
the network loader is updated with the correct COEP value. In the next
patch, SubresourceLoaderUpdater will be used to replace the subresource
loader factory with the one for which the new policy is applied.

Bug: 1039613
Change-Id: Ice82eef86beec2f2f5c5e91f3aa136229f14c3ab
Reviewed-on: https://chromium-review.googlesource.com/2029403
WPT-Export-Revision: dd59b4e7c0f06a30527388c6c87eaf16b3b178ca

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

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

Total 47 tests

Status Summary

Firefox

OK : 1
PASS: 4[GitHub] 50[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt]
FAIL: 2

Chrome

OK : 1
PASS: 4
FAIL: 2

Safari

OK : 1
PASS: 4
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/cross-origin-embedder-policy/require-corp-sw.https.html
importScripts() fails for a script with no corp.: FAIL (Chrome: FAIL, Safari: FAIL)
fetch() to no CORP response should not succeed.: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c6b243025ff7
[wpt PR 22080] - COEP takes effect on a new service worker except for importScripts, a=testonly
https://hg.mozilla.org/integration/autoland/rev/c623b985abcb
[wpt PR 22080] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/c6b243025ff7
https://hg.mozilla.org/mozilla-central/rev/c623b985abcb

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox76: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.