Basic keyboard access for recently_closed menu on NTP (re-work).

chrome/browser/resources/ntp4/recently_closed.js

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 a50306dbb6cc79647d4990040e92d69f45d25b42..8399ca12dad06b260a159baa50a002311f7a9daa 100644
--- a/chrome/browser/resources/ntp4/recently_closed.js
+++ b/chrome/browser/resources/ntp4/recently_closed.js
@@ -90,19 +90,27 @@ cr.define('ntp', function() {
         a.textContent = data.title;
       }
 
-      function onClick(e) {
+      function onActivated(e) {
         ntp.logTimeToClick('RecentlyClosed');
         chrome.send('recordAppLaunchByURL',
                     [encodeURIComponent(data.url),
                      ntp.APP_LAUNCH.NTP_RECENTLY_CLOSED]);
         var index = Array.prototype.indexOf.call(a.parentNode.children, a);
-        chrome.send('reopenTab', [data.sessionId, index,
-            e.button, e.altKey, e.ctrlKey, e.metaKey, e.shiftKey]);
-        // We are likely deleted by this point!
+        var orig = e.originalEvent;
+        var params = [data.sessionId,
+                      index,
+                      orig.type == 'click' ? orig.button : 0,
+                      orig.altKey,
+                      orig.ctrlKey,
+                      orig.metaKey,
+                      orig.shiftKey];
+        chrome.send('reopenTab', params);
 
+        // We are likely deleted by this point!
+        e.stopPropagation();

[Dan Beam, 2013/03/14 21:22:01]

aboxhall@: ^ why did you add this?

         e.preventDefault();
       }
-      a.addEventListener('click', onClick);
+      a.addEventListener('activate', onActivated);
 
       this.menu.appendChild(a);
       cr.ui.decorate(a, MenuItem);