Don't call preventDefault in the FocusGrid's click event

ui/webui/resources/js/cr/ui/focus_grid.js

Index: ui/webui/resources/js/cr/ui/focus_grid.js
diff --git a/ui/webui/resources/js/cr/ui/focus_grid.js b/ui/webui/resources/js/cr/ui/focus_grid.js
index 4ff1635f06f3fce2e1bd2757a03a34204372010e..869585f2a2ff2cbf6eff9b97777af0f06a8f117e 100644
--- a/ui/webui/resources/js/cr/ui/focus_grid.js
+++ b/ui/webui/resources/js/cr/ui/focus_grid.js
@@ -65,7 +65,6 @@ cr.define('cr.ui', function() {
         row.makeRowActive(row.contains(e.target));
       });
 
-      e.preventDefault();

[Dan Beam, 2015/02/05 19:31:10]

when/why was this added?

[hcarmona, 2015/02/05 21:16:09]

This was added as part of the downloads changes. It shouldn't be there. Buttons
are the only element that doesn't work so this wasn't caught because all other
links worked fine.

This function makes sure that there are no lingering "active" rows when an item
is clicked. So it doesn't make sense to stop event handling at this point.

[Dan Beam, 2015/02/05 21:20:36]

are you sure you're not confusing preventDefault() with stopPropagation()?

       return true;
     },
   };