Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: notification.js

Issue 5464830253203456: Refactored the abstraction layer to address prerendered pages on Safari caused by leaky abstraction (Closed)
Patch Set: Created Feb. 22, 2014, 10:45 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« ext/common.js ('K') | « metadata.safari ('k') | popup.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: notification.js
===================================================================
--- a/notification.js
+++ b/notification.js
@@ -85,7 +85,7 @@
return;
event.preventDefault();
event.stopPropagation();
- ext.windows.getLastFocused(function(win) { win.openTab(link.href); });
+ ext.pages.open(link.href);
Wladimir Palant 2014/04/04 14:00:35 Please use spaces for indentation.
Sebastian Noack 2014/04/07 13:15:25 Done.
});
var notificationElement = document.getElementById("notification");
« ext/common.js ('K') | « metadata.safari ('k') | popup.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld