Closed Bug 1357936 Opened 3 years ago Closed 3 years ago

Don't leak ProfilerIOInterposeObserver

Categories

(Core :: Gecko Profiler, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: njn, Assigned: njn)

Details

Attachments

(1 file)

The patch adds a missing |delete|. (This leak only occurred when the
"mainthreadio" feature was enabled, which is not the default.)

The patch also makes the unregistering of the interpose observer conditional on
there being one in the first place, avoiding a harmless but useless
unregistering of |nullptr|.
Attachment #8859784 - Flags: review?(mstange)
Attachment #8859784 - Flags: review?(mstange) → review+
https://hg.mozilla.org/mozilla-central/rev/1b37b1ed3bc1
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.