Support running tasks on nsISerialEventTarget shutdown
Categories
(Core :: XPCOM, enhancement)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox100 | --- | fixed |
People
(Reporter: nika, Assigned: nika)
References
Details
Attachments
(5 files)
This can be used to simplify the logic around DelayedRunnable as we can shut down the delayed task nsITimer instances using a shutdown task, and can also be used for IPC to ensure that IPDL actors are torn down before the thread or event target they're hosted on is shut down.
This should help simplify background tasks using IPDL actors as well as hopefully avoid shutdown hang issues.
|
Assignee | |
Comment 1•3 years ago
|
||
|
|
Assignee | |
Comment 2•3 years ago
|
||
This makes the behaviour of TaskQueue more in-line with nsThread, which may be
useful when converting code to using taskqueue instead of threads.
Depends on D129837
|
|
Assignee | |
Comment 3•3 years ago
|
||
The existing behaviour of the code is not fully compatible with nsIEventTarget.
Depends on D129838
|
|
Assignee | |
Comment 4•3 years ago
|
||
Tasks registered with shutdown tasks are called when the target
nsISerialEventTarget is about to be destroyed.
Depends on D129839
|
|
Assignee | |
Comment 5•2 years ago
|
||
Depends on D129840
Pushed by nlayzell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4b29a3fdfb02 Part 1: Remove the unused TailDispatchingTarget type, r=xpcom-reviewers,KrisWright https://hg.mozilla.org/integration/autoland/rev/0bf347bacee2 Part 2: Accept messages for TaskQueue until queue is exhausted during shutdown, r=xpcom-reviewers,KrisWright https://hg.mozilla.org/integration/autoland/rev/ba728a4ee829 Part 3: DelayedRunnable should only be used with nsISerialEventTarget, r=necko-reviewers,valentin https://hg.mozilla.org/integration/autoland/rev/ad682061ae5c Part 4: Add XPCOM support for shutdown tasks, r=necko-reviewers,KrisWright https://hg.mozilla.org/integration/autoland/rev/3ae0fe8f2824 Part 5: Basic TargetShutdownTask tests, r=KrisWright
|
||
Comment 7•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/4b29a3fdfb02
https://hg.mozilla.org/mozilla-central/rev/0bf347bacee2
https://hg.mozilla.org/mozilla-central/rev/ba728a4ee829
https://hg.mozilla.org/mozilla-central/rev/ad682061ae5c
https://hg.mozilla.org/mozilla-central/rev/3ae0fe8f2824
Description
•