Open Bug 1620367 Opened 5 years ago Updated 2 years ago

Convert the thread 'COM Intcpt Log' to use the background thread pool

Categories

(Core :: IPC: MSCOM, task, P3)

task

Tracking

()

People

(Reporter: KrisWright, Unassigned)

References

(Blocks 1 open bug)

Details

The thread COM Intcpt Log [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/ipc/mscom/InterceptorLog.cpp#141
[2] https://searchfox.org/mozilla-central/rev/91f6c02fcf4c16f78fdc4417f61f192688294066/xpcom/threads/nsThreadUtils.h#1730-1757

Yes, we should be able to do this. The good news is that this thread doesn't run unless explicitly enabled by an environment variable, so this thread will not be around in practice.

Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.