1480236 - Implement queueMicrotask()

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P3)

Description

[Domenic Denicola]

Spec: https://html.spec.whatwg.org/multipage/timers-and-user-prompts.html#microtask-queuing

Tests: https://github.com/web-platform-tests/wpt/pull/6334

Comment 1

[Boris Zbarsky [:bzbarsky]]

What is the status in other engines?  It looked like the Blink intent to implement was withdrawn?

Comment 2

[Anne (:annevk)]

Yeah, I think that was mostly due to it not being in the standard, where it landed a couple days later. I suspect they'll intent again once https://github.com/w3ctag/design-reviews/issues/294 is closed. We could wait for that.

Comment 3

[Domenic Denicola]

Yeah, working on the new intent now in Chrome.

Also worth noting WebKit seems to have implemented: https://bugs.webkit.org/show_bug.cgi?id=188212 (although only on Window for now due to infra issues with microtasks in workers).

Microsoft supports the API, in a sense strong enough to count as "implementer interest", although it's unclear what their prioritization would be: https://github.com/whatwg/html/issues/512#issuecomment-403555829

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: Bug 1480236 - Implement queueMicrotask(), r=baku

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Whoever ends up reviewing this, https://searchfox.org/mozilla-central/source/xpcom/base/CycleCollectedJSContext.cpp#201,249-252 is possibly useful.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=879ee5eccd0abf6ce19bed367c8eb5de09aaf2e1

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

aha, missing one MOZ_KnownLive annotation.

Comment 8

[Olli Pettay [:smaug][[e-mail]]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=606fc87eae18ce33adaf5055ba2cab2f498f8596

Comment 9

[Olli Pettay [:smaug][[e-mail]]]

And more annotations, MOZ_CAN_RUN_SCRIPT_BOUNDARY
https://treeherder.mozilla.org/#/jobs?repo=try&revision=64967c74706762858dc14e75424a4fe244ad240e

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9a5f590f5f75
Implement queueMicrotask(), r=baku

Comment 11

[Andreea Pavel [:apavel]]

Backed out for breaking android builds at nsIGlobalObject.cpp

Push with failures https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&group_state=expanded&selectedJob=248093979&revision=9a5f590f5f7574e186216d510692eab02e242ec8

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=248093979&repo=autoland&lineNumber=21158

Backout: https://hg.mozilla.org/integration/autoland/rev/d451bd35ca1c95f6392cea158332c34e4729ee22

Comment 12

[Boris Zbarsky [:bzbarsky]]

There should be an intent sent somewhere for this, right?

Comment 13

[Olli Pettay [:smaug][[e-mail]]]

yup

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/574e6b299f2d
Implement queueMicrotask(), r=baku

Comment 15

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/574e6b299f2d

Comment 16

[Gregory Pappas [:gregp]]

Docs exist for this feature
https://developer.mozilla.org/docs/Web/API/queueMicrotask