1880789 - Remove dom.workers.modules.enabled pref

Status: RESOLVED FIXED

(Core :: DOM: Workers, task)

Description

[Gregory Pappas [:gregp]]

This pref has been enabled by default for about a year (bug 1812591). Let's remove this feature rollout pref

Comment 1

[Gregory Pappas [:gregp]]

• dom/workers/ScriptLoader.cpp#246-249
• dom/workers/ScriptLoader.cpp#518-520
• dom/workers/ScriptLoader.cpp#704-707
• dom/workers/ScriptLoader.cpp#1259
• dom/webidl/Worker.webidl#34,36
• dom/fs/test/mochitest/head.js#72
• dom/workers/test/test_bug1824498.html#26-28
• dom/workers/test/test_dynamicImport.html#29-31
• dom/workers/test/test_dynamicImport_early_termination.html#28-30
• modules/libpref/init/StaticPrefList.yaml#4134-4138

Comment 2

[tannal2409]

Attachment: Bug 1880789 - Remove dom.workers.modules.enabled pref. r?gregp

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Yulia, is it okay to remove the worker modules pref or do we potentially need to keep it around as a kill switch?

Comment 4

[Yulia Startsev [:yulia] OOO until July 2026]

yes, you should be able to remove it.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/314ece164d39
Remove dom.workers.modules.enabled pref. r=gregp,webidl,dom-worker-reviewers,saschanaz,dom-storage-reviewers,janv,yulia,asuth

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/314ece164d39