Closed Bug 1594858 Opened 2 years ago Closed 2 years ago

Move user cert checks to the dedicated background thread

Categories

(Core :: Networking, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: KrisWright, Assigned: KrisWright)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In Bug 1435141 this check was moved to a LazyIdleThread[1], and it looks like a good candidate to move into our background thread instance[2] instead of using its own lazy thread.

[1]https://searchfox.org/mozilla-central/rev/d061ba55ac76f41129618d638f4ef674303ec103/netwerk/protocol/http/nsHttpHandler.cpp#446
[2] https://searchfox.org/mozilla-central/rev/b2b0077c2e6a516a76bf8077d6f0237c58f5959a/xpcom/threads/nsThreadUtils.cpp#528

Instead of spinning up a LazyIdleThread for this check, just use the background thread pool.

Pushed by kwright@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ce47bb582ecb
Move CanEnableSpeculativeConnect call to the background thread. r=mayhemer
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Assignee: nobody → kwright
You need to log in before you can comment on or make changes to this bug.