Closed Bug 981033 Opened 7 years ago Closed 7 years ago
Fix intermittent "chrome/test
_cyclecollector .xul | Observer should have been called!" failure with ICC
No description provided.
I don't like the look of this intermittent failure, either.
I think this is a silly mistake. The scenario is triggering a full CC when we're in the middle of an ICC. At the start of Collect(), it sets the reentrancy guard, then finishes off the ICC that is in progress. Then it recusively invokes Collect() again, to do the full CC, but the reentrancy guard is already set, so we return immediately. To fix this, I just clear the reentrancy guard before we check if we need a recursive call. All the real CC work is still guarded by the flag, so it should be okay.
Attachment #8388201 - Flags: review?(bugs)
Well, I should do a try run first, just in case.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.