Closed Bug 1609863 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 21229 - SharedWorker: Add static import with 'module' type for SharedWorker

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla74
Tracking Status
firefox74 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(Regressed 2 open bugs, )

Details

(Whiteboard: [wptsync downstream])

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

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

Eriko Kurimoto <elkurin@google.com> wrote:

SharedWorker: Add static import with 'module' type for SharedWorker

SharedWorker with type 'module' Web Platform Tests are added.
Dynamic import is not yet supported, so WPTs which include dyamic import
is expected to fail.

Bug: 824646
Change-Id: I3b4a239f392736d226a736c011e3f5eb0a3e0d78
Reviewed-on: https://chromium-review.googlesource.com/2006995
WPT-Export-Revision: 44f8f954c73f440a320912cbd3fe5350dbb2444c

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

CI Results

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

Total 6 tests and 7 subtests

Status Summary

Firefox

OK : 6
PASS: 11
FAIL: 23

Chrome

OK : 6
PASS: 18
FAIL: 16

Safari

OK : 6
PASS: 8
FAIL: 26

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/workers/modules/dedicated-worker-import-failure.html
Worker construction for a file URL should throw an exception.: FAIL

New Tests That Don't Pass

/workers/modules/shared-worker-options-type.html
Test worker construction with an empty worker type.: FAIL (Chrome: PASS, Safari: FAIL)
Test worker construction with an unknown worker type.: FAIL (Chrome: PASS, Safari: FAIL)
/workers/modules/dedicated-worker-import-failure.html
Dynamic import for non-existent script should throw an exception.: FAIL (Chrome: PASS, Safari: FAIL)
Worker construction for a file URL should throw an exception.: FAIL (Chrome: PASS, Safari: PASS)
importScripts() on module worker should throw an exception.: FAIL (Chrome: PASS, Safari: FAIL)
/workers/modules/shared-worker-import-blob-url.any.html
Dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
Static import and then dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
eval(import()).: FAIL (Chrome: FAIL, Safari: FAIL)
Dynamic import and then static import.: FAIL (Chrome: FAIL, Safari: FAIL)
Static import.: FAIL (Chrome: FAIL, Safari: FAIL)
Nested dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
Nested static import.: FAIL (Chrome: FAIL, Safari: FAIL)
/workers/modules/shared-worker-import-failure.html
SharedWorker construction for a file URL should throw an exception.: FAIL (Chrome: PASS, Safari: FAIL)
importScripts() on module worker should throw an exception.: FAIL (Chrome: FAIL, Safari: FAIL)
/workers/modules/dedicated-worker-options-type.html
Test worker construction with an empty worker type.: FAIL (Chrome: PASS, Safari: FAIL)
Test worker construction with an unknown worker type.: FAIL (Chrome: PASS, Safari: FAIL)
/workers/modules/shared-worker-import.any.html
Dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
Static import and then dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
eval(import()).: FAIL (Chrome: FAIL, Safari: FAIL)
Dynamic import and then static import.: FAIL (Chrome: FAIL, Safari: FAIL)
Static import.: FAIL (Chrome: FAIL, Safari: FAIL)
Nested dynamic import.: FAIL (Chrome: FAIL, Safari: FAIL)
Nested static import.: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/12eded4b0cfe
[wpt PR 21229] - SharedWorker: Add static import with 'module' type for SharedWorker, a=testonly
https://hg.mozilla.org/integration/autoland/rev/7ca6056156b6
[wpt PR 21229] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4190ff506fce
Revert expectations in modules/shared-worker-options-credentials.html.ini for causing failures. a=testonly

The changes reverted in Comment 5 in modules/shared-worker-options-credentials.html.ini were pushed in Bug 1614907.

See Also: → 1614907
Flags: needinfo?(james)
See Also: → 1615753
Regressions: 1615753
Flags: needinfo?(james)
Regressions: 1592239
You need to log in before you can comment on or make changes to this bug.