Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/ext/background.js

Issue 5162273421131776: Issue 1768 - Pass value returned by ext.onMessage listeners to chrome.runtime.onMessage listener (Closed)
Patch Set: Created Jan. 9, 2015, 10:56 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/ext/content.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/ext/background.js
===================================================================
--- a/chrome/ext/background.js
+++ b/chrome/ext/background.js
@@ -254,7 +254,7 @@
};
ext.webRequest = {
- onBeforeRequest: new ext._EventTarget(true),
+ onBeforeRequest: new ext._EventTarget(),
Sebastian Noack 2015/01/09 10:58:25 Beside passing the return value below, I also forg
handlerBehaviorChanged: chrome.webRequest.handlerBehaviorChanged
};
@@ -392,7 +392,7 @@
}
};
- return ext.onMessage._dispatch(message, sender, sendResponse);
+ return ext.onMessage._dispatch(message, sender, sendResponse).indexOf(true) != -1;
});
« no previous file with comments | « no previous file | chrome/ext/content.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld