Closed Bug 956671 Opened 7 years ago Closed 7 years ago

GC: Don't need to check all compartments in Debugger::sweepAll()

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

Currently we iterate through all compartments in Debugger::sweepAll() to check whether the global of any debuggee is about to be finalized.

Since the debuggee global's compartment must be in the same zone group as the debugger's compartment, we don't need to do this and can just iterate the compartments in the current zone group.
Attachment #8356052 - Flags: review?(wmccloskey)
Attachment #8356052 - Flags: review?(wmccloskey) → review+
https://hg.mozilla.org/mozilla-central/rev/34b3cb13258b
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.