Unified Diff: popupBlocker.js
Issue 5464830253203456:
Refactored the abstraction layer to address prerendered pages on Safari caused by leaky abstraction (Closed)
Patch Set: Rebased and addressed comments
Created April 7, 2014, 1:14 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« popup.js ('k')
|
safari/ext/background.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: popupBlocker.js |
| =================================================================== |
| --- a/popupBlocker.js |
| +++ b/popupBlocker.js |
| @@ -21,10 +21,10 @@ |
| chrome.webNavigation.onCreatedNavigationTarget.addListener(function(details) |
| { |
| - var sourceTab = new Tab({id: details.sourceTabId}); |
| - var sourceFrame = new Frame({id: details.sourceFrameId, tab: sourceTab}); |
| + var sourcePage = new ext.Page({id: details.sourceTabId}); |
| + var sourceFrame = new ext.Frame({frameId: details.sourceFrameId, tabId: details.sourceTabId}); |
| - if (isFrameWhitelisted(sourceTab, sourceFrame)) |
| + if (isFrameWhitelisted(sourcePage, sourceFrame)) |
| return; |
| var openerUrl = sourceFrame.url; |
« no previous file with comments
|
« popup.js ('k')
|
safari/ext/background.js » ('j')
|
no next file with comments »
