Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3676)

Unified Diff: chrome/browser/resources/ntp4/recently_closed.js

Issue 7550015: ntp4: record app launch source (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/ntp4/recently_closed.js
diff --git a/chrome/browser/resources/ntp4/recently_closed.js b/chrome/browser/resources/ntp4/recently_closed.js
index 04750a29b7b2d829db180cf971b2a108c8998438..71d332e47cae4b2ddc38b3dc1e57a5a0dc1ce3f9 100644
--- a/chrome/browser/resources/ntp4/recently_closed.js
+++ b/chrome/browser/resources/ntp4/recently_closed.js
@@ -91,7 +91,12 @@ cr.define('ntp4', function() {
function onActivate(e) {
// TODO(estade): don't convert to string.
+ chrome.send('recordAppLaunchByURL',
+ [data.url, String(ntp4.APP_LAUNCH.NTP_RECENTLY_CLOSED)]);
+ // TODO(estade): don't convert to string.
chrome.send('reopenTab', [String(data.sessionId)]);
+ // We are likely deleted by this point!
+
e.preventDefault();
}
a.addEventListener('activate', onActivate);

Powered by Google App Engine
This is Rietveld 408576698