Inject script only once

[rbdr/junction] / extension / junction.js

diff --git a/extension/junction.js b/extension/junction.js
index 889e23f975dfdd3dccb9ce7dd081367fe33023e8..776096927473e635738e8577f8988675b7e9645e 100644 (file)
--- a/extension/junction.js
+++ b/extension/junction.js
@@ -1,19 +1,19 @@
 const internals = {
-
-  promisesSupported: !!(window.browser),
+  promisesSupported: !!window.browser,
+  injectedScript: false,
   port: null,
   currentUrl: null,
 
   icons: {
     call: {
-      16: 'icons/action-16.png',
-      32: 'icons/action-32.png'
+      16: "icons/action-16.png",
+      32: "icons/action-32.png",
     },
 
     hangUp: {
-      16: 'icons/hang_up-16.png',
-      32: 'icons/hang_up-32.png'
-    }
+      16: "icons/hang_up-16.png",
+      32: "icons/hang_up-32.png",
+    },
   },
 
   onClick() {
@@ -21,93 +21,92 @@ const internals = {
       return internals.hangUp();
     }
 
-    return internals.joinAudioCall();
+    return internals.initializeContentScript();
   },
 
-  onConnect(port) {
-
-    internals.port = port;
-    port.onDisconnect.addListener(internals.onDisconnect);
-    port.onMessage.addListener(internals.onMessage);
-    port.postMessage({
-      action: 'joinAudioCall',
+  joinAudioCall() {
+    internals.port.postMessage({
+      action: "joinAudioCall",
       data: {
         currentUrl: internals.currentUrl,
-        tada: internals.getRoot().runtime.getURL('sounds/tada.wav')
-      }
+        tada: internals.getRoot().runtime.getURL("sounds/tada.wav"),
+      },
     });
     internals.getRoot().browserAction.enable();
-    internals.setIcon('hangUp');
+    internals.setIcon("hangUp");
   },
 
-  onMessage(message) {
+  onConnect(port) {
+    internals.port = port;
+    port.onDisconnect.addListener(internals.onDisconnect);
+    port.onMessage.addListener(internals.onMessage);
+    internals.joinAudioCall();
+  },
 
-    if (message.action === 'error') {
-      internals.getRoot().browserAction.setBadgeText({ text: 'x' }, () => {});
+  onMessage(message) {
+    if (message.action === "error") {
+      internals.getRoot().browserAction.setBadgeText({ text: "x" }, () => {});
     }
   },
 
   onDisconnect() {
-    internals.getRoot().browserAction.setBadgeText({ text: '' }, () => {});
-    internals.setIcon('call');
+    internals.getRoot().browserAction.setBadgeText({ text: "" }, () => {});
+    internals.setIcon("call");
     internals.currentUrl = null;
     internals.port = null;
     internals.getRoot().browserAction.enable();
   },
 
-  async joinAudioCall() {
-
+  async initializeContentScript() {
     internals.getRoot().browserAction.disable();
     const activeTabs = await internals.getActiveTabs();
 
     internals.currentUrl = activeTabs[0].url;
-    const execution = await internals.getRoot().tabs.executeScript(activeTabs[0].id, {
-      file: '/build/content_script.js'
-    }, () => {
-
-      if (internals.getRoot().runtime.lastError) {
-        internals.onDisconnect();
-      }
-    });
+    if (!internals.injectedScript) {
+      const execution = await internals.getRoot().tabs.executeScript(
+        activeTabs[0].id,
+        {
+          file: "/build/content_script.js",
+        },
+        () => {
+          internals.injectedScript = true;
+        },
+      );
+    } else {
+      internals.getRoot().tabs.connect(activeTabs[0].id)
+    }
 
     if (execution && !execution[0]) {
       internals.onDisconnect();
     }
-  }
-  ,
-
+  },
   hangUp() {
-
     internals.getRoot().browserAction.disable();
     internals.port.postMessage({
-      action: 'hangUp'
+      action: "hangUp",
     });
   },
 
   isInCall() {
-
-    return !!(internals.port);
+    return !!internals.port;
   },
 
   setIcon(iconSet) {
-
     internals.getRoot().browserAction.setIcon({
-      path: internals.icons[iconSet]
+      path: internals.icons[iconSet],
     });
   },
 
   getRoot() {
-
     return window.browser || window.chrome;
   },
 
   // Chrome doesn't yet implement the promise based tabs.query :'(
 
   getActiveTabs() {
-
     const query = {
       currentWindow: true,
-      active: true
+      active: true,
     };
 
     if (internals.promisesSupported) {
@@ -115,9 +114,7 @@ const internals = {
     }
 
     return new Promise((resolve) => {
-
       internals.getRoot().tabs.query(query, (tabs) => {
-
         return resolve(tabs);
       });
     });