Closed Bug 1753780 Opened 4 years ago Closed 3 years ago

[wpt-sync] Sync PR 32704 - import.meta.url should include the URL fragment

Categories

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

task

Tracking

()

RESOLVED FIXED
99 Branch
Tracking Status
firefox99 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Depends on 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Chris Dumez <cdumez@apple.com> wrote:

import.meta.url should include the URL fragment

Update workers/modules/shared-worker-import-meta.html to expect import.meta.url to include
the URL fragment. This matches the behavior of both WebKit and Blink.
See https://github.com/whatwg/html/issues/5162 for more information.

Component: web-platform-tests → DOM: Workers
Product: Testing → Core

CI Results

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

Total 1 tests and 3 subtests

Status Summary

Firefox

OK : 1
FAIL: 3

Chrome

OK : 1
PASS: 3

Safari

OK : 1
FAIL: 3

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/workers/modules/shared-worker-import-meta.html
Test import.meta.url on the top-level module script.: FAIL (Chrome: PASS, Safari: FAIL)
Test import.meta.url on the imported module script.: FAIL (Chrome: PASS, Safari: FAIL)
Test import.meta.url on the imported module script with a fragment.: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/86c635e00d20 [wpt PR 32704] - import.meta.url should include the URL fragment, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b103e973a460 [wpt PR 32704] - import.meta.url should include the URL fragment, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
You need to log in before you can comment on or make changes to this bug.