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

Status: RESOLVED FIXED

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

Description

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

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 <[e-mail]> 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

Comment 1

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

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

Comment 2

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

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)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
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

Comment 4

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/12eded4b0cfe
https://hg.mozilla.org/mozilla-central/rev/7ca6056156b6

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4190ff506fce
Revert expectations in modules/shared-worker-options-credentials.html.ini for causing failures. a=testonly

Comment 6

[Alexandru Michis [:malexandru]]

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

Comment 7

[Alexandru Michis [:malexandru]]

These are the failures caused by the wpt sync push:
https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=success%2Ctestfailed%2Cbusted%2Cexception&searchStr=%28wpt&tochange=4190ff506fce51fc5c38c82166a6f9ee299f1389&fromchange=a056e19dce0ee6c130688e5a078cf8a5ce3aafb8&selectedJob=289001260

Comment 8

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/4190ff506fce