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

Unified Diff: chrome/browser/resources/local_ntp/most_visited_util.js

Issue 17526008: Log NTP hovers in 1993 clients (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed OVERRIDE error Created 7 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/local_ntp/most_visited_util.js
diff --git a/chrome/browser/resources/local_ntp/most_visited_util.js b/chrome/browser/resources/local_ntp/most_visited_util.js
index 3b9589e672d65a6dd67bf75a7ba3c8b543c38b15..ad49a6d21cc0817e2044a82feb615157eee83826 100644
--- a/chrome/browser/resources/local_ntp/most_visited_util.js
+++ b/chrome/browser/resources/local_ntp/most_visited_util.js
@@ -62,6 +62,10 @@ function createMostVisitedLink(params, href, title, text) {
link.tabIndex = '-1';
if (text)
link.textContent = text;
+ link.addEventListener('mouseover', function() {
+ var ntpApiHandle = chrome.embeddedSearch.newTabPage;
+ ntpApiHandle.logEvent('NewTabPage.NumberOfMouseOvers');
+ });
return link;
}
« no previous file with comments | « no previous file | chrome/browser/ui/search/instant_controller.cc » ('j') | chrome/browser/ui/search/instant_ntp.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698