Resets hover state when ToolbarActionView is dragged

chrome/browser/ui/views/toolbar/toolbar_action_view_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/sessions/session_tab_helper.h"
 #include "chrome/browser/ui/toolbar/test_toolbar_action_view_controller.h"
@@ skipping 99 matching lines @@
         CreateParams(views::Widget::InitParams::TYPE_WINDOW_FRAMELESS);
     params.bounds = gfx::Rect(0, 0, 200, 200);
     widget_->Init(params);
   }
   void TearDown() override {
     if (!widget_->IsClosed())
       widget_->Close();
     views::ViewsTestBase::TearDown();
   }
 
+ protected:
   views::Widget* widget() { return widget_; }
 
+  void SetDraggedView(views::View* dragged_view) {
+    widget()->set_dragged_view(dragged_view);
+  }
+
  private:
   // The widget managed by this test.
   views::Widget* widget_;
 
   // Web contents need a fake ui thread.
   content::TestBrowserThread ui_thread_;
 
   DISALLOW_COPY_AND_ASSIGN(ToolbarActionViewUnitTest);
 };
 
@@ skipping 53 matching lines @@
   controller.ShowPopup(false);
   EXPECT_EQ(views::Button::STATE_NORMAL, view.state());
   controller.HidePopup();
 
   // Show the popup through a user action - the button should be pressed.
   controller.ShowPopup(true);
   EXPECT_EQ(views::Button::STATE_PRESSED, view.state());
   controller.HidePopup();
   EXPECT_EQ(views::Button::STATE_NORMAL, view.state());
 
+  // Simulate dragging |view|. Once a drag is started a capture loss should
+  // reset button state.
+  view.SetHotTracked(true);
+  EXPECT_EQ(views::Button::ButtonState::STATE_HOVERED, view.state());
+  SetDraggedView(&view);
+  widget()->SetCapture(&view);
+  widget()->ReleaseCapture();
+  EXPECT_EQ(views::Button::ButtonState::STATE_NORMAL, view.state());
+  SetDraggedView(nullptr);
+
   // Ensure that the button's enabled state reflects that of the controller.
   controller.SetEnabled(false);
   EXPECT_EQ(views::Button::STATE_DISABLED, view.state());
   controller.SetEnabled(true);
   EXPECT_EQ(views::Button::STATE_NORMAL, view.state());
 
   // Ensure that clicking on an otherwise-disabled action optionally opens the
   // context menu.
   controller.SetDisabledClickOpensMenu(true);
   controller.SetEnabled(false);
@@ skipping 20 matching lines @@
   // If the view isn't visible, the overflow button should be pressed for
   // popups.
   view.SetVisible(false);
   controller.ShowPopup(true);
   EXPECT_EQ(views::Button::STATE_NORMAL, view.state());
   EXPECT_EQ(views::Button::STATE_PRESSED, overflow_button.state());
   controller.HidePopup();
   EXPECT_EQ(views::Button::STATE_NORMAL, view.state());
   EXPECT_EQ(views::Button::STATE_NORMAL, overflow_button.state());
 }