Fix the Back and Forwards button broken in 23101

Issue 165318 was accepted but caused the back and forwards
button to become unresponsive to middle-clicks.

This should fix the issue but if there is a problem, we
should revert 23101 because it causes a major regression
while only providing a cosmetic enhancement.

BUG=20323
TEST=none

Review URL: http://codereview.chromium.org/199007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25313 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 4 deletions

diff --git a/views/controls/button/button_dropdown.cc b/views/controls/button/button_dropdown.cc
index db40cee..490eccf 100644
--- a/views/controls/button/button_dropdown.cc
+++ b/views/controls/button/button_dropdown.cc
@@ -40,7 +40,7 @@ ButtonDropDown::~ButtonDropDown() {
 ////////////////////////////////////////////////////////////////////////////////
 
 bool ButtonDropDown::OnMousePressed(const MouseEvent& e) {
-  if (IsEnabled() && e.IsLeftMouseButton() && HitTest(e.location())) {
+  if (IsEnabled() && IsTriggerableEvent(e) && HitTest(e.location())) {
     // Store the y pos of the mouse coordinates so we can use them later to
     // determine if the user dragged the mouse down (which should pop up the
     // drag down menu immediately, instead of waiting for the timer)
@@ -56,15 +56,15 @@ bool ButtonDropDown::OnMousePressed(const MouseEvent& e) {
 }
 
 void ButtonDropDown::OnMouseReleased(const MouseEvent& e, bool canceled) {
-  if (e.IsLeftMouseButton() || (e.IsRightMouseButton() &&
-                                !HitTest(e.location()))) {
+  if (IsTriggerableEvent(e) ||
+      (e.IsRightMouseButton() && !HitTest(e.location()))) {
     ImageButton::OnMouseReleased(e, canceled);
   }
 
   if (canceled)
     return;
 
-  if (e.IsLeftMouseButton())
+  if (IsTriggerableEvent(e))
     show_menu_factory_.RevokeAll();
 
   if (IsEnabled() && e.IsRightMouseButton() && HitTest(e.location())) {