Second try at fixing location bar icon keyboard activation.

chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc

Index: chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
diff --git a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
index 67c376a36ef63f1191553e32087da0ae13ec0315..88acf5493217afa34a172b46759624ae99a4ccaf 100644
--- a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
+++ b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
@@ -128,10 +128,16 @@ gfx::Size IconLabelBubbleView::GetPreferredSize() const {
   return GetSizeForLabelWidth(label_->GetPreferredSize().width());
 }
 
+bool IconLabelBubbleView::OnKeyPressed(const ui::KeyEvent& event) {
+  if (event.key_code() == ui::VKEY_RETURN)
+    return OnActivate();
+  return false;

[Peter Kasting, 2016/05/04 02:21:00]

BubbleIconView::OnKeyPressed() returned true for space, but this returns false. 
Seems like one of these isn't right.

[Evan Stade, 2016/05/04 02:45:52]

yea, this one isn't right; it should show a ripple and count the accelerator as
handled. I filed crbug.com/608976 for that.

+}
+
 bool IconLabelBubbleView::OnKeyReleased(const ui::KeyEvent& event) {
-  if (event.key_code() != ui::VKEY_RETURN && event.key_code() != ui::VKEY_SPACE)
-    return false;
-  return OnActivate();
+  if (event.key_code() == ui::VKEY_SPACE)
+    return OnActivate();
+  return false;
 }
 
 void IconLabelBubbleView::Layout() {