Closed Bug 1363266 Opened 4 years ago Closed 4 years ago
Avoid busy waiting in safebrowsing gtest test case: Sync
59 bytes, text/x-review-board-request
This is in preparation for bug 1359490. SpinEventLoopUntil requires using NS_ProcessNextEvent(..., true) so we have to set |done| on the main thread.
Hi Nathan, I've submitted the fix and the try result looks good. Would you like to have it a review to see if it meets your requirement? Thanks :)
Comment on attachment 8865757 [details] Bug 1363266 - Post an event to main thread to set |done| to true. https://reviewboard.mozilla.org/r/137368/#review140598 Thank you!
Attachment #8865757 - Flags: review?(nfroyd) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/dac53c5cc129 Post an event to main thread to set |done| to true. r=froydnj
You need to log in before you can comment on or make changes to this bug.