Closed Bug 1442443 Opened 3 years ago Closed 3 years ago

Remove event spinning code from the tree

Categories

(Cloud Services :: Firefox: Common, enhancement)

Product:
Cloud Services
Component:
Firefox: Common
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(firefox60 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 60
Tracking Flags:
Tracking Status
firefox60 --- fixed

People

(Reporter: markh, Assigned: markh)

Details

Attachments

(1 file)

Bug 1442443 - remove all event loop spinning from Async.js.
59 bytes, text/x-review-board-request
eoger
: review+
Details 
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: 3 years ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.