1789981 - Migrate devtools/server/actors/watcher/SessionDataHelpers.jsm to ESM

Status: RESOLVED FIXED

(DevTools :: Framework, task)

Description

[Alexandre Poirot [:ochameau]]

This is one of the rare (only one?) JSM loaded from a worker thread in DevTools.
We have to wait for bug 1247687 before being able to convert this JSM to ESM.

Comment 1

[Alexandre Poirot [:ochameau]]

This also block removing support for loading jsm as commonjs in the devtools module loader.
Once we are able to migrate it to ESM, we can move from using require to ChromeUtils.import* method and remove a few code in the loader:
https://searchfox.org/mozilla-central/rev/a26af613a476fafe6c3eba05a81bef63dff3c9f1/devtools/shared/loader/base-loader.js#343-349
(and a few dependencies of this code)

Comment 2

[Alexandre Poirot [:ochameau]]

Note that there is also this JSM which is a dependency:
https://searchfox.org/mozilla-central/source/devtools/shared/validate-breakpoint.jsm

Comment 3

[Mark Banner (:standard8)]

Changing to only be under the devtools esm-ification bug to clarify the dependency tree.

Comment 4

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1789981 - [devtools] Convert last DevTools JSM to ESM.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4eec27066227
[devtools] Convert last DevTools JSM to ESM. r=devtools-reviewers,nchevobbe

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/4eec27066227