Closed Bug 1620379 Opened 6 years ago Closed 6 years ago

Convert the thread 'GraphRunner' to use the background thread pool

Categories

(Core :: Web Audio, task)

task
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: KrisWright, Unassigned)

References

(Blocks 1 open bug)

Details

The thread GraphRunner [1] uses the NS_NewNamedThread method, so it might be possible to move it to the background thread pool.

In an effort to reduce the number of ad-hoc threads in the browser, we want to convert instances of NS_NewNamedThread to use the background thread pool [2] wherever possible. If your thread may block, you can also include the NS_DISPATCH_EVENT_MAY_BLOCK flag.

[1] https://searchfox.org/mozilla-central/rev/c79c0d65a183d9d38676855f455a5c6a7f7dadd3/dom/media/GraphRunner.cpp#38
[2] https://searchfox.org/mozilla-central/rev/91f6c02fcf4c16f78fdc4417f61f192688294066/xpcom/threads/nsThreadUtils.h#1730-1757

GraphRunner is a high priority media processing thread. I don't think the work it does is suitable for the background thread pool.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
See Also: → 1473469, 1458448
See Also: → 1620386
You need to log in before you can comment on or make changes to this bug.