MenuButtons enter Hover state upon Tap Down

ui/views/controls/button/menu_button.cc

Index: ui/views/controls/button/menu_button.cc
diff --git a/ui/views/controls/button/menu_button.cc b/ui/views/controls/button/menu_button.cc
index f2b3852bb644d60444b92396f80bf70ba20207b5..cace30436f4dac9d42ec2d057cbeb5e9b3c130e1 100644
--- a/ui/views/controls/button/menu_button.cc
+++ b/ui/views/controls/button/menu_button.cc
@@ -9,6 +9,7 @@
 #include "ui/base/dragdrop/drag_drop_types.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/base/ui_base_switches_util.h"
 #include "ui/events/event.h"
 #include "ui/events/event_constants.h"
 #include "ui/gfx/canvas.h"
@@ -205,12 +206,23 @@ void MenuButton::OnMouseMoved(const ui::MouseEvent& event) {
 }
 
 void MenuButton::OnGestureEvent(ui::GestureEvent* event) {
-  if (state() != STATE_DISABLED && ShouldEnterPushedState(*event) &&
-      !Activate()) {
-    // When |Activate()| returns |false|, it means that a menu is shown and
-    // has handled the gesture event. So, there is no need to further process
-    // the gesture event here.
-    return;
+  if (state() != STATE_DISABLED) {
+    if (switches::IsTouchFeedbackEnabled()) {

[flackr, 2014/10/27 17:51:27]

Should this be done after line 220 so that the menu gets first crack at handling
the event?

[jonross, 2014/10/27 18:51:39]

Good call, that way any classes extending get the first shot.

Done.

+      if (event->type() == ui::ET_GESTURE_TAP_DOWN) {
+        event->SetHandled();

[flackr, 2014/10/27 17:51:27]

This doesn't prevent context menu generation, does it?

[jonross, 2014/10/27 18:51:39]

No it does not, it just keeps this view as the event target for future events,
including ones that go out of bounds. Allows it to receive the cancel/end on
moves out of the view. Show Press still causes the context menu to execute.

+        SetState(Button::STATE_HOVERED);
+      } else if (state() == Button::STATE_HOVERED &&
+                 (event->type() == ui::ET_GESTURE_TAP_CANCEL ||
+                  event->type() == ui::ET_GESTURE_END)) {
+        SetState(Button::STATE_NORMAL);
+      }
+    }
+    if (ShouldEnterPushedState(*event) && !Activate()) {
+      // When |Activate()| returns |false|, it means that a menu is shown and
+      // has handled the gesture event. So, there is no need to further process
+      // the gesture event here.
+      return;
+    }
   }
   LabelButton::OnGestureEvent(event);
 }