Closed Bug 1095443 Opened 7 years ago Closed 7 years ago

Ensure processNextEvent never blocks after processing a Promise microtask

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal
Points:
1

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla36
Iteration:
36.3

People

(Reporter: Paolo, Assigned: Paolo)

References

Details

Attachments

(1 file)

The patch
4.83 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
Attached patch The patch — — Splinter Review 
This allows us to use DOM Promise scheduling with Promise.jsm.

The try run is now green for the entire tree:

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=b6b8172550ad
Attachment #8518864 - Flags: review?(bzbarsky)
Flags: qe-verify-
Flags: firefox-backlog+
Assignee: nobody → paolo.mozmail
Status: NEW → ASSIGNED
Comment on attachment 8518864 [details] [diff] [review]
The patch

>+class DummyRunnable : public nsRunnable {

Please put this in an anonymous namespace.

r=me
Attachment #8518864 - Flags: review?(bzbarsky) → review+
Iteration: 36.2 → 36.3
https://hg.mozilla.org/mozilla-central/rev/d8c5f67ac4cd
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.