1812628 - Introduce flag for worker modules

Status: RESOLVED FIXED

(Core :: DOM: Workers, task)

Description

[Yulia Startsev [:yulia]]

Add a flag for module workers

Comment 1

[Yulia Startsev [:yulia]]

Attachment: Bug 1812628 - put worker modules behind a flag; r=jonco

Comment 2

[Yulia Startsev [:yulia]]

Attachment: Bug 1812628 - add new tests for beta/release; r=jonco

Depends on D168129

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cc6317f62e54
put worker modules behind a flag; r=jonco
https://hg.mozilla.org/integration/autoland/rev/da2c075d4524
add new tests for beta/release; r=jonco

Comment 4

[Norisz Fay [:noriszfay]]

Backed out for causing wpt failures on pointerevent_pointerrawupdate_in_pointerlock.html

Backout link

Push with failures

Failure log

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1f5f65f28752
put worker modules behind a flag; r=jonco
https://hg.mozilla.org/integration/autoland/rev/b8b31239ec9f
add new tests for beta/release; r=jonco

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/1f5f65f28752
https://hg.mozilla.org/mozilla-central/rev/b8b31239ec9f

Comment 7

[Brian Birtles (:birtles)]

Just a note that there is potentially a Web compat issue here since it will cause content that specifies { type: 'module' } to fail.

I have received multiple reports of an extension I maintain breaking in beta because of this. It works fine in Nightly where { type: 'module' } is enabled, and in Release where { type: 'module' } is ignored but breaks in Beta. I can easily fix the add-on but there may be other sites affected by this.

Comment 8

[Yulia Startsev [:yulia]]

Thanks for the report. I'll change this to ignore rather than throw.

Comment 9

[Brian Birtles (:birtles)]

Thank you!