Document that entering a compartment roots the global

RESOLVED FIXED in Firefox 58

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jorendorff, Assigned: jorendorff)

Tracking

unspecified
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

(Whiteboard: [js:tech-debt])

Attachments

(1 attachment)

<ajeffrey> A rooting question for globals...
<ajeffrey> Does AutoCompartment root the global?
<ajeffrey> Or do I need to root it myself?
<tcampbell> it does not root

Fortunately, entering a compartment does root the global (although the mechanism is unobvious--JSCompartment::traceRoots() traces the global if necessary).
Assignee

Updated

2 years ago
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Assignee

Updated

2 years ago
Attachment #8911326 - Flags: review?(sphink)
Attachment #8911326 - Flags: review?(sphink) → review+
Priority: -- → P3
Whiteboard: [js:tech-debt]

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c3c6f7e45868
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.