1662846 - Add reporter for loaded JSMs

Status: RESOLVED FIXED

(Core :: XPConnect, task, P2)

Description

[Andrew McCreight (out until 2026) [:mccr8]]

It would be nice to tell at a glance in a memory report which JSMs are loaded.

Comment 1

[Andrew McCreight (out until 2026) [:mccr8]]

It would also make it easier to tell if some change caused an additional JSM to be loaded.

This would be an "other measurements" reporter that would iterate over the set of loaded modules from somewhere in mozJSComponentLoader.

Comment 2

[Kris Maglione [:kmag]]

Note we already have a list of these in Cu.loadedModules for the sake of the startup performance mochitests.

Comment 3

[Chris Peterson [:cpeterson]]

Nika says this would be helpful and not hard to do.

Comment 4

[Andrew McCreight (out until 2026) [:mccr8]]

This would be a good introductory bug for somebody who wants to become more familiar with our JSM loading (should such a person exist).

Comment 5

[Kris Maglione [:kmag]]

Attachment: Bug 1662846: Add memory reporter for loaded JS modules and components. r=mccr8

This makes it much easier to determine which modules and components are loaded
in which processes now that they no longer each have their own compartments.

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:kmag, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/748d8c28435d
Add memory reporter for loaded JS modules and components. r=mccr8

Comment 8

[Cristian Brindusan [:cbrindusan]]

Backed out changeset 748d8c28435d (Bug 1662846) for mochitest failures at test_memoryReporters.xhtml
https://hg.mozilla.org/integration/autoland/rev/22360273336512474a9aaf82b55794a5960d1be6

Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&revision=748d8c28435d2f022a78bf1b26d6f060afe4e2e1&selectedTaskRun=Sr1vqH84Swm59-wvFnOrsg.0

Failure log:
https://treeherder.mozilla.org/logviewer?job_id=326233255&repo=autoland&lineNumber=111627

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e67edffd1e83
Add memory reporter for loaded JS modules and components. r=mccr8

Comment 10

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/e67edffd1e83