Add cd3 to toolbar and faster connections between background and content

mozilla · Nov 2, 2017 · 5989b5f · 5989b5f
1 parent fa4c80f
commit 5989b5f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/extension/background.js b/extension/background.js
@@ -32,19 +32,17 @@ browser.browserAction.onClicked.addListener(function() {
   });
   creating.
   then(tab => {
-            console.log(`Created new tab: ${tab.id}`);
-            setTimeout( function() {
-
+            const intervalConnection = setInterval( function() {
                 browser.tabs.sendMessage(
                     tab.id,
                     {msg: "background script syn"}
                 ).then(response => {
-                    console.log("Message from the content script", response.response);
+                    clearInterval(intervalConnection);
                 }).catch(error => {
-                    console.error(`Error: ${error}`);
+                    //console.error(`Not connected yet. Retrying ${error}`);
                 });
 
-            }, 2000);
+            }, 100);
 
         },
           error => {

diff --git a/extension/content.js b/extension/content.js
@@ -279,6 +279,7 @@
         }
     }
     browser.runtime.onMessage.addListener(request => {
+      metrics.start_session('toolbar');
       SpeakToMePopup.showAt(0, 0);
       stm_init();
       return Promise.resolve({response: "content script ack"});