Closed Bug 884509 Opened 11 years ago Closed 2 years ago

Gecko Profiler add-on holds 100MBs of memory even if disabled

Categories

(Core :: Gecko Profiler, defect)

defect
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: vladan, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached file about:memory snapshot
STR:

1. Open a profile
2. Check SymbolicateWorker.js memory use in about:memory
3. Disable add-on & click "Minimize memory usage" in about:memory

Expected: SymbolicateWorker.js memory use goes to 0 in about:memory
Actual: SymbolicateWorker.js keeps using >100MB of memory

Kyle, can you suggest any extra diagnostic steps? Could it be that the add-on simply isn't shutting down its worker?
Summary: Gecko Profiler add-on holds hundreds of MBs of memory even if disabled → Gecko Profiler add-on holds 100MBs of memory even if disabled
(In reply to Vladan Djeric (:vladan) from comment #0)
> Kyle, can you suggest any extra diagnostic steps? Could it be that the
> add-on simply isn't shutting down its worker?

The GC/CC should clean it up. Why is it staying alive after the addon is removed?
Well the addon is clearly still around in that about:memory report.
Could disabling the addon not release everything required?
I don't know.  That's a jetpack question.
Flags: needinfo?(dtownsend+bugmail)
(In reply to Benoit Girard (:BenWa) from comment #3)
> Could disabling the addon not release everything required?

It should, but this could be a leak in the SDK modules or the add-on's code not properly listening for the unload event.
Flags: needinfo?(dtownsend+bugmail)
Keep in mind that workers will hold themselves alive if they're doing things (handling messages, timeouts, loading scripts, etc.), so that could cause the addon to stick around longer than it expects. Calling terminate() on it would kill any internal activity so you might be able to use that here.
Blocks: 1329163
Blocks: 1329219

Not an add-on anymore.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: