Remove AsTaskQueue() call often used with MozPromise

RESOLVED FIXED in Firefox 65

Status

()

enhancement
RESOLVED FIXED
7 months ago
6 months ago

People

(Reporter: jya, Assigned: jya)

Tracking

unspecified
mozilla65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(3 attachments)

Assignee

Description

7 months ago
There's many places still using MozPromise::Then() with AsTaskQueue() from the day MozPromise were MediaPromise and could only work with TaskQueue.

MozPromise are using nsISerialEventTarget now, we should reflect that everywhere, removing the unnecessary call.
Assignee

Comment 2

7 months ago
Make IsCurrentThreadIn() const.

Depends on D11491

Comment 4

6 months ago
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1dab4c7062ae
P1. Remove taskqueue's shutdown in Benchmark. r=gerald
https://hg.mozilla.org/integration/autoland/rev/6f08cc82afe3
P2. Remove use of AsTaskQueue(). r=cpearce
https://hg.mozilla.org/integration/autoland/rev/bcf48a8dba79
P3. Remove now unused AsTaskQueue(). r=gerald

Comment 5

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1dab4c7062ae
https://hg.mozilla.org/mozilla-central/rev/6f08cc82afe3
https://hg.mozilla.org/mozilla-central/rev/bcf48a8dba79
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.