Closed Bug 982146 Opened 6 years ago Closed 6 years ago
Clean up Sync
Runnable wait code
Our condition variables don't remain triggered like some other event implementations (e.g. manual-reset Windows Event objects). You have to watch your own condition flag.
Attachment #8389235 - Flags: review?(benjamin)
Comment on attachment 8389235 [details] [diff] [review] Patch, v1 As discussed on IRC, I don't think there is actually a bug here: we cannot signal the monitor until the task is complete and we can't enter the monitor until we're waiting. However this patch with the explicit condition is better in general and we should do it.
Attachment #8389235 - Flags: review?(benjamin) → review+
Yep, sorry, I read too quickly! https://hg.mozilla.org/integration/mozilla-inbound/rev/ed30fc4d3e17
Summary: SyncRunnable doesn't wait correctly → Clean up SyncRunnable wait code
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.