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

Unified Diff: safari/ext/background.js

Issue 6751260996796416: Issue 1724 - Fixed memory leak in messaging code, when no response is sent, on Safari (Closed)
Patch Set: Addressed comments Created Jan. 8, 2015, 2:35 p.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 | « ext/common.js ('k') | safari/ext/common.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: safari/ext/background.js
===================================================================
--- a/safari/ext/background.js
+++ b/safari/ext/background.js
@@ -606,13 +606,14 @@
break;
case "webRequest":
var page = pages[event.message.pageId];
-
- event.message = ext.webRequest.onBeforeRequest._dispatch(
+ var results = ext.webRequest.onBeforeRequest._dispatch(
event.message.url,
event.message.type,
page,
page._frames[event.message.frameId]
);
+
+ event.message = (results.indexOf(false) == -1);
break;
case "proxy":
event.message = backgroundPageProxy.handleMessage(event.message);
« no previous file with comments | « ext/common.js ('k') | safari/ext/common.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld