Resets hover state when ToolbarActionView is dragged

ui/views/widget/widget.h

 // Copyright (c) 2012 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.
 
 #ifndef UI_VIEWS_WIDGET_WIDGET_H_
 #define UI_VIEWS_WIDGET_WIDGET_H_
 
 #include <set>
 #include <stack>
 #include <vector>
@@ skipping 578 matching lines @@
                     int operation,
                     ui::DragDropTypes::DragEventSource source);
 
   // Returns the view that requested the current drag operation via
   // RunShellDrag(), or NULL if there is no such view or drag operation.
   View* dragged_view() {
     return const_cast<View*>(const_cast<const Widget*>(this)->dragged_view());
   }
   const View* dragged_view() const { return dragged_view_; }
 
+  // Unit tests may temporarily set |dragged_view| to simulate a drag.
+  void set_dragged_view(View* dragged_view) { dragged_view_ = dragged_view; }

[sky, 2016/03/01 03:58:48]

Exposing api like this, especially when it doesn't indicate it is for test only
is easy to misuse. Instead friend the test.

[varkha, 2016/03/01 04:03:34]

I've tried that (making it private and making ToolbarActionViewUnitTest a friend
of Widget) before sending it for review and the compiler didn't like that. Is
there an easy reason that you can see why it breaks in this case? Is there some
visibility constraint in component build that gets broken?

[varkha, 2016/03/01 04:58:53]

Done. Just needed to mention that the friend test class was in global namespace.

+
   // Adds the specified |rect| in client area coordinates to the rectangle to be
   // redrawn.
   virtual void SchedulePaintInRect(const gfx::Rect& rect);
 
   // Sets the currently visible cursor. If |cursor| is NULL, the cursor used
   // before the current is restored.
   void SetCursor(gfx::NativeCursor cursor);
 
   // Returns true if and only if mouse events are enabled.
   bool IsMouseEventsEnabled() const;
@@ skipping 359 matching lines @@
   bool movement_disabled_;
 
   ScopedObserver<ui::NativeTheme, ui::NativeThemeObserver> observer_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(Widget);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WIDGET_WIDGET_H_