Ash: Double-clicking on a shelf item now opens the item

Previously it would open the item then immediately minimize it. This caused
user confusion because they tried a double-click and didn't see anything
open on the screen.

BUG=404060
TEST=ash_unittests ShelfViewTest.ClickingTwiceActivatesOnce

Review URL: https://codereview.chromium.org/492963008

Cr-Commit-Position: refs/heads/master@{#291695}

2 files changed, 16 insertions, 0 deletions

diff --git a/ash/test/shelf_view_test_api.cc b/ash/test/shelf_view_test_api.cc
index d190f95..b55690f 100644
--- a/ash/test/shelf_view_test_api.cc
+++ b/ash/test/shelf_view_test_api.cc
@@ -114,6 +114,11 @@ int ShelfViewTestAPI::GetButtonSpacing() {
   return kShelfButtonSpacing;
 }
 
+void ShelfViewTestAPI::ButtonPressed(views::Button* sender,
+                                     const ui::Event& event) {
+  return shelf_view_->ButtonPressed(sender, event);
+}
+
 bool ShelfViewTestAPI::SameDragType(ShelfItemType typea,
                                     ShelfItemType typeb) const {
   return shelf_view_->SameDragType(typea, typeb);
diff --git a/ash/test/shelf_view_test_api.h b/ash/test/shelf_view_test_api.h
index 71853fd..7916ea7 100644
--- a/ash/test/shelf_view_test_api.h
+++ b/ash/test/shelf_view_test_api.h
@@ -13,6 +13,14 @@ class Rect;
 class Size;
 }
 
+namespace ui {
+class Event;
+}
+
+namespace views {
+class Button;
+}
+
 namespace ash {
 class OverflowBubble;
 class ShelfButton;
@@ -70,6 +78,9 @@ class ShelfViewTestAPI {
   // Returns the button space size.
   int GetButtonSpacing();
 
+  // Wrapper for ShelfView::ButtonPressed.
+  void ButtonPressed(views::Button* sender, const ui::Event& event);
+
   // Wrapper for ShelfView::SameDragType.
   bool SameDragType(ShelfItemType typea, ShelfItemType typeb) const;