Log NTP hovers in 1993 clients

chrome/browser/resources/local_ntp/most_visited_title.js

Index: chrome/browser/resources/local_ntp/most_visited_title.js
diff --git a/chrome/browser/resources/local_ntp/most_visited_title.js b/chrome/browser/resources/local_ntp/most_visited_title.js
index 703219c161c706805ebb3295cd3f49e7ffb6302b..702096a00984e17002666c1487447c5e03f40e54 100644
--- a/chrome/browser/resources/local_ntp/most_visited_title.js
+++ b/chrome/browser/resources/local_ntp/most_visited_title.js
@@ -11,7 +11,16 @@ window.addEventListener('DOMContentLoaded', function() {
   'use strict';
 
   fillMostVisited(window.location, function(params, data) {
-    document.body.appendChild(
-        createMostVisitedLink(params, data.url, data.title, data.title));
+    function handleMouseOver() {
+      var params = parseQueryParams(document.location);
+      if ('pos' in params && isFinite(params.pos)) {
+        var pos = parseInt(params.pos, 10);
+        var ntpApiHandle = chrome.embeddedSearch.newTabPage;
+        ntpApiHandle.logEvent('NewTabPage.NumberOfMouseOvers', 1, null, pos);
+      }
+    };

[beaudoin, 2013/06/22 01:14:53]

Seems to be duplicated code, one more reason to add all that in
createMostVisited.

[annark1, 2013/06/25 16:01:38]

Done.

+    var link = createMostVisitedLink(params, data.url, data.title, data.title);
+    link.addEventListener('mouseover', handleMouseOver);
+    document.body.appendChild(link);
   });
 });