Use onResponeStarted for executing userscripts

Potential solution for greasemonkey#2574
Sxderp · Nov 12, 2017 · 8c47ef0 · 8c47ef0
1 parent 6141b3d
commit 8c47ef0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/manifest.json b/manifest.json
@@ -65,6 +65,6 @@
     "storage",
     "tabs",
     "unlimitedStorage",
-    "webNavigation"
+    "webRequest"
   ]
 }
diff --git a/src/bg/execute.js b/src/bg/execute.js
@@ -5,7 +5,7 @@ content script executions.
 TODO: Make document_start execution time work as intended.
 */
 
-function executeUserscriptOnNavigation(detail) {
+function executeUserscriptOnResponseStarted(detail) {
   if (false === getGlobalEnabled()) return;
 
   var userScriptIterator = UserScriptRegistry.scriptsToRunAt(detail.url);

diff --git a/src/bg/execute.run.js b/src/bg/execute.run.js
@@ -1 +1,3 @@
-chrome.webNavigation.onCommitted.addListener(executeUserscriptOnNavigation);
+chrome.webRequest.onResponseStarted.addListener(
+  executeUserscriptOnResponseStarted,
+  { 'urls': ['<all_urls>'], 'types': ['main_frame', 'sub_frame'] });