Closed Bug 1442443 Opened 2 years ago Closed 2 years ago

Remove event spinning code from the tree

Categories

(Cloud Services :: Firefox: Common, enhancement)

enhancement
Not set

Tracking

(firefox60 fixed)

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: markh, Assigned: markh)

Details

Attachments

(1 file)

Async.js has code to support async blocking via spinning nested event loops. Now that eoger's work and a few small followups have landed, only 1 test still needs something like this. We should hack that test up and remove the main event spinning code from the production code to prevent people accidentally choosing to use it.
Assignee: nobody → markh
Comment on attachment 8955340 [details]
Bug 1442443 - remove all event loop spinning from Async.js.

https://reviewboard.mozilla.org/r/224510/#review230458

Good riddance!
Attachment #8955340 - Flags: review?(eoger) → review+
Pushed by mhammond@skippinet.com.au:
https://hg.mozilla.org/integration/autoland/rev/5cedb30e6477
remove all event loop spinning from Async.js. r=eoger
https://hg.mozilla.org/mozilla-central/rev/5cedb30e6477
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.