Closed Bug 1311212 Opened 8 years ago Closed 8 years ago

Add dead CPOW debugging facility

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: billm, Assigned: billm)

References

Details

Attachments

(1 file)

I found this useful while working on bug 1211270. It just forces a GC before handling any CPOW request.
Attached patch patchSplinter Review
Most of the work here is in holding strong references to CPOWs until the next turn of the event loop. When the parent finishes an event, it tells the child the ID of the last CPOW it saw. We assume that all CPOWs whose IDs are <= that number are eligible for collection in the child. CPOWs whose IDs are > that number are traced so they stay alive.

Without this strong reference thing, the debugging mode finds all sorts of problems that aren't real.
Attachment #8802339 - Flags: review?(mrbkap)
Comment on attachment 8802339 [details] [diff] [review]
patch

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

This is awesome.
Attachment #8802339 - Flags: review?(mrbkap) → review+
Pushed by wmccloskey@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/99bf3401b43f
Add dead CPOW debugging facility (r=mrbkap)
https://hg.mozilla.org/mozilla-central/rev/99bf3401b43f
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Depends on: 1330018
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: