Unified Diff: notification.js
Issue 16067002:
Added Safari Support (Closed)
Patch Set: Rebased and changed icon to 16px (for Safari) and 19px again (for Chrome)
Created Nov. 2, 2013, 5:50 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« metadata.safari ('K')
|
« metadata.safari ('k')
|
options.html » ('j')
|
popup.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: notification.js |
| =================================================================== |
| --- a/notification.js |
| +++ b/notification.js |
| @@ -15,7 +15,7 @@ |
| * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. |
| */ |
| -var backgroundPage = chrome.extension.getBackgroundPage(); |
| +var backgroundPage = ext.backgroundPage.getDOMWindow(); |
| var require = backgroundPage.require; |
| var Utils = require("utils").Utils; |
| @@ -85,7 +85,7 @@ |
| return; |
| event.preventDefault(); |
| event.stopPropagation(); |
| - chrome.tabs.create({url: link.href}); |
| + ext.windows.getLastFocused(function(win) { win.openTab(link.href); }); |
| }); |
| var notificationElement = document.getElementById("notification"); |
