Closed
Bug 956671
Opened 10 years ago
Closed 10 years ago
GC: Don't need to check all compartments in Debugger::sweepAll()
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla29
People
(Reporter: jonco, Assigned: jonco)
Details
Attachments
(1 file)
|
1.30 KB,
patch
|
billm
:
review+
|
Details | Diff | Splinter Review |
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+
|
Assignee | |
Comment 1•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/34b3cb13258b
|
||
Comment 2•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/34b3cb13258b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in
before you can comment on or make changes to this bug.
Description
•