Closed Bug 1296488 Opened 3 years ago Closed 3 years ago

Lift isBeingDestroyed checks above traceRuntimeForGC

Categories

(Core :: JavaScript: GC, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

Now that finishRoots is actually asserting that all our roots are gone, we can do this safely.
Attachment #8782712 - Flags: review?(jcoppeard)
Comment on attachment 8782712 [details] [diff] [review]
2_lift_isBeingDestroyed-v0.diff

Review of attachment 8782712 [details] [diff] [review]:
-----------------------------------------------------------------

Nice.
Attachment #8782712 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/a39b3debfe37
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.