Closed Bug 1730581 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 30699 - [WPT] Check base URLs in worker-interception-redirect.https.html

Categories

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

Product:
Core
Component:
DOM: Service Workers
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
94 Branch
Tracking Flags:
Tracking Status
firefox94 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

b'Hiroshige Hayashizaki <hiroshige@chromium.org>' wrote:

[WPT] Check base URLs in worker-interception-redirect.https.html

This CL

  • Changes the response URL of a worker top-level script to
    .../subdir/... when intercepted by a service worker,
    to make worker's base URL directory different depending on
    whether the response URL or the current request URL is used.
  • Makes worker-interception-redirect-webworker.js/py
    to fetch()/importScripts() relative to its
    settings object's base URL, not
    /service-workers/service-worker/resources/, and checks
    the URLs intercepted by serviceworkers.

This checks the settings object's base URL of the workers,
and thus exposes that Chromium uses current request URLs
instead of response URLs as base URL, which will be fixed by
https://chromium-review.googlesource.com/c/chromium/src/+/3145772.

Bug: 861564
Change-Id: I32dbb4aae55aebd12c75b26bd4a503ce366b57a6
Reviewed-on: https://chromium-review.googlesource.com/3146235
WPT-Export-Revision: fb13f1accf730ec7af4cc404915f526631ac82a6

PR 30699 applied with additional changes from upstream: 02b44873e7e4fcaaa9c59d7feaaadac82c59e982
Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core

CI Results

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

Total 1 tests and 66 subtests

Status Summary

Firefox

OK : 1
PASS: 58
FAIL: 8

Chrome

OK : 1
PASS: 30
FAIL: 24

Safari

OK : 1
PASS: 12
FAIL: 18

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/service-workers/service-worker/worker-interception-redirect.https.html
Case #1: network scope1->scope2 (module DedicatedWorker, importScripts()): FAIL
Case #2: network scope1->out-scope (module DedicatedWorker, importScripts()): FAIL

New Tests That Don't Pass

/service-workers/service-worker/worker-interception-redirect.https.html
Case #1: network scope1->scope2 (module DedicatedWorker, importScripts()): FAIL (Chrome: PASS, Safari: PASS)
Case #1: network scope1->scope2 (module SharedWorker, importScripts()): FAIL (Chrome: PASS)
Case #2: network scope1->out-scope (module DedicatedWorker, importScripts()): FAIL (Chrome: PASS, Safari: PASS)
Case #2: network scope1->out-scope (module SharedWorker, importScripts()): FAIL (Chrome: PASS)
Case #3: sw scope1->scope2 (module DedicatedWorker, importScripts()): FAIL
Case #3: sw scope1->scope2 (module SharedWorker, importScripts()): FAIL (Chrome: PASS)
Case #4: sw scope1->out-scope (module DedicatedWorker, importScripts()): FAIL
Case #4: sw scope1->out-scope (module SharedWorker, importScripts()): FAIL (Chrome: PASS)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b875760430c5
[wpt PR 30699] - [WPT] Check base URLs in worker-interception-redirect.https.html, a=testonly
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/48100b493ace
[wpt PR 30699] - [WPT] Check base URLs in worker-interception-redirect.https.html, a=testonly

https://hg.mozilla.org/mozilla-central/rev/48100b493ace

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