Bug 1404934 wired up Intermediate Preloading by initializing it in blocklist-clients.js, while it's not a blocklist.

It should be initialized (and maintained) someplace a little more clear for the long-haul.

You could move the initialization to BrowserGlue.jsm:

    Services.tm.idleDispatchToMainThread(() => {
      SecurityClients.initialize();
    });

But we better get Thunderbird folks in the loop since it does not have browserglue

Jorg, do you have something like BrowserGlue.jsm in Thunderbird?

https://searchfox.org/mozilla-central/rev/11cfa0462a6b5d8c5e2111b8cfddcf78098f0141/browser/components/BrowserGlue.jsm#1816-1818

Sure :-) - https://searchfox.org/comm-central/source/mail/components/mailGlue.js

Thanks for the heads-up.

Jorg, I'm sending the patch to autoland.
How do we proceed to port the few lines to maiGlue.js? I guess I could do it, but I don't even have cloned TB...

Hi :-)
Well, I suggest you paste the lines here and tell me after which line they go using:
https://searchfox.org/comm-central/source/mail/components/mailGlue.js
I'll take care of the rest.

Would that work for you?

I didn't find exactly the same structure as in browserGlue, but I guess it's this (_onMailStartupDone())

--- mailGlue.js	2019-05-22 12:55:11.678982616 +0200
+++ mailGlue.js.orig	2019-05-22 12:48:51.174554255 +0200
@@ -14,7 +14,6 @@
 var {ExtensionSupport} = ChromeUtils.import("resource:///modules/ExtensionSupport.jsm");
 var { L10nRegistry, FileSource } = ChromeUtils.import("resource://gre/modules/L10nRegistry.jsm");
 var {AppConstants} = ChromeUtils.import("resource://gre/modules/AppConstants.jsm");
-var {RemoteSecuritySettings} = ChromeUtils.import("resource://gre/modules/psm/RemoteSecuritySettings.jsm");
 
 // lazy module getters
 
@@ -183,11 +182,6 @@
 
     const {ExtensionsUI} = ChromeUtils.import("resource:///modules/ExtensionsUI.jsm");
     ExtensionsUI.checkForSideloadedExtensions();
-
-    // Certificates revocation list, etc.
-    Services.tm.idleDispatchToMainThread(() => {
-      RemoteSecuritySettings.init();
-    });
   },
 
   _handleLink(aSubject, aData) {

In case you don't have Services.tm.idleDispatchToMainThread(), you can call .init() directly. It only instantiates cheap objects, and won't block startup. I just mimic what's done in browserGlue for other initializations :)

Thanks!

Thanks, Mathieu. I guess you meat to add the code rather than remove it ;-)

I'll see how this works once the M-C patch has landed, that is, if I need to pull out the Services.tm.idleDispatchToMainThread() call.

Thanks for your help.

Thanks, Mathieu. I guess you meat to add the code rather than remove it ;-)

Oh! I haven't even paid attention, I did the diff the other way :/

It looks good to me!

https://hg.mozilla.org/mozilla-central/rev/134250706ea6

Thanks Mathieu, I checked with some debugging that this is really run.

Comment on attachment 9181067 [details]
Bug 1526018 - Initialize OneCRL when GeckoView starts.

Revision D93250 was moved to bug 1667179. Setting attachment 9181067 [details] to obsolete.