fix: run content scripts only once

src/background/main.js

@@ -712,13 +712,7 @@ async function openContentView(message, view) {
     return;
   }
 
-  const [isContentModule] = await executeScript({
-    func: () => typeof initContent !== 'undefined',
-    code: `typeof initContent !== 'undefined'`,
-    tabId
-  });
-
-  if (!isContentModule) {
+  if (!(await hasModule({tabId, module: 'content'}))) {
     await executeScript({files: ['/src/content/script.js'], tabId});
   }
 
@@ -1043,20 +1037,20 @@ async function execEngine(tabId, engine, taskId) {
   await executeScript({
     func: taskId => (self.taskId = taskId),
     args: [taskId],
-    code: `var taskId = '${taskId}';`,
+    code: `self.taskId = '${taskId}'`,
     tabId
   });
   await executeScript({files: ['/src/commons-engine/script.js'], tabId});
   await executeScript({files: [`/src/engines/${engine}/script.js`], tabId});
 }
 
 async function searchClickTarget(session) {
-  const [isParseModule] = await executeScript({
-    func: () => typeof initParse !== 'undefined',
-    code: `typeof initParse !== 'undefined'`,
+  const isParseModule = await hasModule({
     tabId: session.sourceTabId,
-    frameIds: [session.sourceFrameId]
+    frameId: session.sourceFrameId,
+    module: 'parse'
   });
+
   if (!isParseModule) {
     await executeScript({
       files: ['/src/parse/script.js'],

src/base/main.js

@@ -1,14 +1,8 @@
 import storage from 'storage/storage';
 import {isIncomingShareContext, processIncomingShare} from 'utils/app';
+import {runOnce} from 'utils/common';
 
 function main() {
-  // Script may be injected multiple times.
-  if (self.baseModule) {
-    return;
-  } else {
-    self.baseModule = true;
-  }
-
   self.touchTarget = {
     node: null,
     dx: 0,
@@ -156,4 +150,6 @@ function main() {
   }
 }
 
-main();
+if (runOnce('baseModule')) {
+  main();
+}

src/content/main.js

@@ -5,10 +5,10 @@ import {
   getAppTheme,
   addThemeListener
 } from 'utils/app';
-import {addCssClass} from 'utils/common';
+import {addCssClass, runOnce} from 'utils/common';
 import {targetEnv} from 'utils/config';
 
-var contentStorage = {
+const contentStorage = {
   viewFrame: null,
   viewFrameId: 0,
   viewMessagePort: null,
@@ -163,13 +163,15 @@ async function configTheme(viewFrame) {
   await setTheme();
 }
 
-self.initContent = function () {
+function main() {
   addViewFrame();
 
   browser.runtime.onMessage.addListener(onMessage);
   if (targetEnv !== 'firefox') {
     browser.runtime.onConnect.addListener(onConnect);
   }
-};
+}
 
-initContent();
+if (runOnce('contentModule')) {
+  main();
+}

src/engines/123rf.js

@@ -1,4 +1,5 @@
 import {validateUrl} from 'utils/app';
+import {runOnce} from 'utils/common';
 import {initSearch, prepareImageForUpload, sendReceipt} from 'utils/engines';
 
 const engine = '123rf';
@@ -45,4 +46,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/adobestock.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'adobestock';
@@ -20,4 +20,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/alamy.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'alamy';
@@ -20,4 +20,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/alibabaChina.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'alibabaChina';
@@ -28,4 +28,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/ascii2d.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'ascii2d';
@@ -18,4 +18,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/auDesign.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'auDesign';
@@ -27,4 +27,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/auTrademark.js

@@ -1,4 +1,4 @@
-import {findNode, processNode} from 'utils/common';
+import {findNode, processNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'auTrademark';
@@ -40,4 +40,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/baidu.js

@@ -1,5 +1,5 @@
 import {validateUrl} from 'utils/app';
-import {findNode, isMobile} from 'utils/common';
+import {findNode, isMobile, runOnce} from 'utils/common';
 import {
   initSearch,
   prepareImageForUpload,
@@ -76,4 +76,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/bing.js

@@ -1,4 +1,4 @@
-import {findNode, isMobile} from 'utils/common';
+import {findNode, isMobile, runOnce} from 'utils/common';
 import {
   initSearch,
   prepareImageForUpload,
@@ -59,4 +59,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/branddb.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'branddb';
@@ -33,4 +33,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/clipretrieval.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'clipretrieval';
@@ -20,4 +20,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/depositphotos.js

@@ -1,4 +1,4 @@
-import {findNode, makeDocumentVisible} from 'utils/common';
+import {findNode, makeDocumentVisible, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'depositphotos';
@@ -28,4 +28,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/dreamstime.js

@@ -2,7 +2,8 @@ import {
   findNode,
   processNode,
   isMobile,
-  executeScriptMainContext
+  executeScriptMainContext,
+  runOnce
 } from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 import {targetEnv} from 'utils/config';
@@ -39,4 +40,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/enterpix.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'enterpix';
@@ -24,4 +24,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/esearch.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'esearch';
@@ -22,4 +22,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/freepik.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'freepik';
@@ -20,4 +20,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/getty.js

@@ -1,4 +1,4 @@
-import {findNode, sleep} from 'utils/common';
+import {findNode, runOnce, sleep} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'getty';
@@ -29,4 +29,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/google.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'google';
@@ -35,4 +35,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/googleLens.js

@@ -1,4 +1,4 @@
-import {findNode, processNode, sleep} from 'utils/common';
+import {findNode, processNode, runOnce, sleep} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'googleLens';
@@ -62,4 +62,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/haveibeentrained.js

@@ -1,4 +1,4 @@
-import {findNode} from 'utils/common';
+import {findNode, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'haveibeentrained';
@@ -18,4 +18,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}

src/engines/icons8.js

@@ -1,4 +1,4 @@
-import {findNode, makeDocumentVisible} from 'utils/common';
+import {findNode, makeDocumentVisible, runOnce} from 'utils/common';
 import {setFileInputData, initSearch, sendReceipt} from 'utils/engines';
 
 const engine = 'icons8';
@@ -21,4 +21,6 @@ function init() {
   initSearch(search, engine, taskId);
 }
 
-init();
+if (runOnce('search')) {
+  init();
+}