Closed
Bug 1311212
Opened 8 years ago
Closed 8 years ago
Add dead CPOW debugging facility
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: billm, Assigned: billm)
References
Details
Attachments
(1 file)
25.78 KB,
patch
|
mrbkap
:
review+
|
Details | Diff | Splinter Review |
I found this useful while working on bug 1211270. It just forces a GC before handling any CPOW request.
Assignee | ||
Comment 1•8 years ago
|
||
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 2•8 years ago
|
||
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)
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/99bf3401b43f
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in
before you can comment on or make changes to this bug.
Description
•