Use a PersistentRooted to automate tracing of unwrappedException_

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla43
Points:
---

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

The use of |throwing| rather than |unwrappedException.isMarkable()| to chose when we keep the exception live seems worryingly prone to leaving dead pointers around in the heap, although I was not able to find any places where they (obviously) get out of sync.
Attachment #8657342 - Flags: review?(sphink)
Comment on attachment 8657342 [details] [diff] [review]
un_special_case_unwrappedException_rooting-v0.diff

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

I don't know, this could badly regress the cases where you have >1M JSContexts live...
Attachment #8657342 - Flags: review?(sphink) → review+
https://hg.mozilla.org/mozilla-central/rev/0fda784c6682
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.