1789595 - Convert devtools/server JSMs to ES Modules

Status: RESOLVED FIXED

(DevTools :: General, task)

Description

[Alexandre Poirot [:ochameau]]

JSMs in devtools/server seems to be mostly internals of the server, so should be easy to migrate to ESM:
devtools/server/actors/targets/target-actor-registry.jsm
devtools/server/actors/watcher/SessionDataHelpers.jsm
devtools/server/actors/watcher/WatcherRegistry.jsm
devtools/server/actors/watcher/browsing-context-helpers.jsm
devtools/server/connectors/js-window-actor/DevToolsFrameChild.jsm
devtools/server/connectors/js-window-actor/DevToolsFrameParent.jsm
devtools/server/connectors/js-window-actor/DevToolsWorkerChild.jsm
devtools/server/connectors/js-window-actor/DevToolsWorkerParent.jsm
devtools/server/connectors/js-window-actor/WindowGlobalLogger.jsm
devtools/server/startup/content-process.jsm

Comment 1

[Alexandre Poirot [:ochameau]]

Oh wait, some of these modules are loaded in the worker thread and are blocked by bug 1247687.

Actually, it may only be this one module:
devtools/server/actors/watcher/SessionDataHelpers.jsm

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1789595 - [devtools] Convert devtools/server JSM to ES Modules.

This is the result of:
$ ./mach esmify devtools/server/

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1789595 - [devtools] Migrate require to new devtools/server's ESM to ChromeUtils.importESModule.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ec840ebd0d22
[devtools] Convert devtools/server JSM to ES Modules. r=nchevobbe
https://hg.mozilla.org/integration/autoland/rev/35ccddb0757d
[devtools] Migrate require to new devtools/server's ESM to ChromeUtils.importESModule. r=nchevobbe

Comment 5

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/ec840ebd0d22
https://hg.mozilla.org/mozilla-central/rev/35ccddb0757d