Closed Bug 1487258 Opened Last year Closed Last year
load enterprise roots on a background thread
46 bytes, text/x-phabricator-request
|Details | Review|
At startup, if the enterprise roots pref is enabled, we do the finding/loading on the main thread. Since this can involve certificate verification, it should probably be done on a background thread. Luckily we already have a handy background thread that we use to load the loadable roots module, so this should be a straightforward change.
Loading enterprise roots could potentially take a while, so we certainly shouldn't do it on the main thread at startup. Note that this doesn't address the case where a user enables the feature while Firefox is running. This isn't great but since it's an about:config preference rather than a first-class preference exposed in about:preferences, we can probably get away with it for now.
Comment on attachment 9005284 [details] bug 1487258 - load enterprise roots on a background at startup r?jcj J.C. Jones [:jcj] (he/him) has approved the revision.
Attachment #9005284 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/3a800dfc6ad8 load enterprise roots on a background at startup r=jcj
You need to log in before you can comment on or make changes to this bug.