Unified Diff: safari/ext/popup.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
|
« safari/ext/content.js ('k')
|
stats.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/popup.js |
| =================================================================== |
| --- a/safari/ext/popup.js |
| +++ b/safari/ext/popup.js |
| @@ -63,14 +63,12 @@ |
| // import ext into the javascript context of the popover. This code might fail, |
| // when the background page isn't ready yet. So it is important to put it below |
| // the reloading code above. |
| - var backgroundPage = safari.extension.globalPage.contentWindow; |
| + window.ext = { |
| + __proto__: safari.extension.globalPage.contentWindow.ext, |
| - window.ext = { |
| - __proto__: backgroundPage.ext, |
| closePopup: function() |
| { |
| safari.self.hide(); |
| } |
| }; |
| - window.TabMap = backgroundPage.TabMap; |
| })(); |
« no previous file with comments
|
« safari/ext/content.js ('k')
|
stats.js » ('j')
|
no next file with comments »
