Unified Diff: chrome/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
|
« no previous file
|
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: chrome/ext/background.js |
| =================================================================== |
| --- a/chrome/ext/background.js |
| +++ b/chrome/ext/background.js |
| @@ -325,7 +325,14 @@ |
| if (requestType == "other" && parseInt(navigator.userAgent.match(/\bChrome\/(\d+)/)[1], 10) >= 38) |
| requestType = "object"; |
| - if (!ext.webRequest.onBeforeRequest._dispatch(details.url, requestType, new Page({id: details.tabId}), frame)) |
| + var results = ext.webRequest.onBeforeRequest._dispatch( |
| + details.url, |
| + requestType, |
| + new Page({id: details.tabId}), |
| + frame |
| + ); |
| + |
| + if (results.indexOf(false) != -1) |
| return {cancel: true}; |
| } |
| } |
« no previous file with comments
|
« no previous file
|
ext/common.js » ('j')
|
no next file with comments »
