Reordered startup sequence to be more friendly with embedders like the Tor Browser.

2 files changed, 45 insertions, 45 deletions

diff --git a/src/bg/defaults.js b/src/bg/defaults.js
index 4d071b4..7170743 100644
--- a/src/bg/defaults.js
+++ b/src/bg/defaults.js
@@ -1,36 +1,38 @@
 'use strict';
 
-ns.defaults = (async () => {
-  let defaults = {
-    local: {
-      debug: false,
-      showCtxMenuItem: true,
-      showCountBadge: true,
-      showFullAddresses: false,
-    },
-    sync: {
-      "global": false,
-      "xss": true,
-      "clearclick": true
-    }
-  };
-  let defaultsClone = JSON.parse(JSON.stringify(defaults));
+var Defaults = {
+  async init()  {
+    let defaults = {
+      local: {
+        debug: false,
+        showCtxMenuItem: true,
+        showCountBadge: true,
+        showFullAddresses: false,
+      },
+      sync: {
+        "global": false,
+        "xss": true,
+        "clearclick": true
+      }
+    };
+    let defaultsClone = JSON.parse(JSON.stringify(defaults));
 
-  for (let [k, v] of Object.entries(defaults)) {
-    let store = await Storage.get(k, k);
-    if (k in store) {
-      Object.assign(v, store[k]);
+    for (let [k, v] of Object.entries(defaults)) {
+      let store = await Storage.get(k, k);
+      if (k in store) {
+        Object.assign(v, store[k]);
+      }
+      v.storage = k;
     }
-    v.storage = k;
-  }
 
-  Object.assign(ns, defaults);
+    Object.assign(ns, defaults);
 
-  // dynamic settings
-  if (!ns.local.uuid) {
-    ns.local.uuid = uuid();
-    await ns.save(ns.local);
-  }
+    // dynamic settings
+    if (!ns.local.uuid) {
+      ns.local.uuid = uuid();
+      await ns.save(ns.local);
+    }
 
-  return ns.defaults = defaultsClone;
-})();
+    return ns.defaults = defaultsClone;
+  }
+};
diff --git a/src/bg/main.js b/src/bg/main.js
index 72ef5de..58e7aef 100644
--- a/src/bg/main.js
+++ b/src/bg/main.js
@@ -23,8 +23,7 @@
   }
 
   async function init() {
-    await include("/bg/defaults.js");
-    await ns.defaults;
+    await Defaults.init();
     
     let policyData = (await Storage.get("sync", "policy")).policy;
     if (policyData && policyData.DEFAULT) {
@@ -35,16 +34,22 @@
       ns.policy = await Legacy.createOrMigratePolicy();
       ns.savePolicy();
     }
-    
-    
-    
-    await include("/bg/RequestGuard.js");
+
     await RequestGuard.start();
     await XSS.start(); // we must start it anyway to initialize sub-objects
     if (!ns.sync.xss) {
       XSS.stop();
     }
-    Commands.install();
+
+    Messages.addHandler(messageHandler);
+
+    try {
+      await Messages.send("started");
+    } catch (e) {
+      // no embedder to answer us
+    }
+    log("STARTED");
+
   };
 
   let Commands = {
@@ -65,8 +70,6 @@
 
     togglePermissions() {},
     install() {
-
-
       if ("command" in browser) {
         // keyboard shortcuts
         browser.commands.onCommand.addListener(cmd => {
@@ -182,13 +185,8 @@
 
       deferWebTraffic(init(), 
         async () => {
-
-          await include("/bg/Settings.js");
-          Messages.addHandler(messageHandler);
-
-          await Messages.send("started");
-          log("STARTED");
-
+          Commands.install();
+            
           this.devMode = (await browser.management.getSelf()).installType === "development";
           if (this.local.debug) {
             if (this.devMode) {