Log NTP hovers in 1993 clients

chrome/browser/resources/local_ntp/most_visited_thumbnail.js

Index: chrome/browser/resources/local_ntp/most_visited_thumbnail.js
diff --git a/chrome/browser/resources/local_ntp/most_visited_thumbnail.js b/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
index 7417b04619d91cd857993580c4ee55af0c9203d0..f422f19dff7b74e3411870086ef57de3d022d3f3 100644
--- a/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
+++ b/chrome/browser/resources/local_ntp/most_visited_thumbnail.js
@@ -16,8 +16,20 @@ window.addEventListener('DOMContentLoaded', function() {
       var domain = document.createElement('div');
       domain.textContent = data.domain;
       link.appendChild(domain);
+      link.addEventListener('mouseover', handleMouseOver);
+      link.addEventListener('click', handleClick);

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

I don't see handleClick defined anywhere. Also, whi do you put it only on
DomainElements?

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

Should not have been there. Sorry, I didn't catch it from an early iteration.
On 2013/06/22 01:14:53, beaudoin wrote:

       document.body.appendChild(link);
     };
+
+    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]

Would it make sense to move this to createMostVisitedLink and add the event
listeners in there? You seem to need them both for DomainElement and regular
thumbnails.

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

Done.

+
     if (data.thumbnailUrl) {
       var image = new Image();
       image.onload = function() {
@@ -26,6 +38,7 @@ window.addEventListener('DOMContentLoaded', function() {
         var link = createMostVisitedLink(params, data.url, data.title);
         link.appendChild(shadow);
         link.appendChild(image);
+        link.addEventListener('mouseover', handleMouseOver);
         document.body.appendChild(link);
       };