Browse Source

fix: don't store ids of @noframes scripts

tophf 6 years ago
parent
commit
7c24691b67
1 changed files with 7 additions and 5 deletions
  1. 7 5
      src/injected/content/inject.js

+ 7 - 5
src/injected/content/inject.js

@@ -26,11 +26,13 @@ export function triageScripts(data) {
     [INJECT_CONTENT]: [],
   };
   if (data.scripts) {
-    data.scripts = data.scripts.filter((script) => {
-      bridge.ids.push(script.props.id);
-      if ((window.top === window || !script.meta.noframes) && script.config.enabled) {
-        bridge.enabledIds.push(script.props.id);
-        return true;
+    data.scripts = data.scripts.filter(({ meta, props, config }) => {
+      if (!meta.noframes || window.top === window) {
+        bridge.ids.push(props.id);
+        if (config.enabled) {
+          bridge.enabledIds.push(props.id);
+          return true;
+        }
       }
       return false;
     });