Don't nuke all modules on unload

RESOLVED WONTFIX

Status

Add-on SDK
General
RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: erikvold, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

It's unnecessary to nuke some modules, and I assume that takes some time.  If nuking has no cost then close this bug, otherwise I'd like to see a module.metadata key that allows us to opt-out of nuking some modules, to speed up unload time.
OS: Mac OS X → All
Hardware: x86 → All
Flags: needinfo?(rFobic)
As far as I know we don't nuke all modules, instead we just nuke loader module that keeps refs to all the other modules. Since loader is nuked GC can claim all the other module sandboxes with all the objects in them. Unless someone has changed that and I'm not aware, as far as I can see though it's not the case.
Flags: needinfo?(rFobic)
Flags: needinfo?(evold)
Ah let's just close this, we should measure unload time first.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: needinfo?(evold)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.