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.
« 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') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | 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 »
