Resets hover state when ToolbarActionView is dragged

ui/views/widget/widget.h

Index: ui/views/widget/widget.h
diff --git a/ui/views/widget/widget.h b/ui/views/widget/widget.h
index 2d5b68e42837afab1a6a0c7093793b5049fe572c..31548860da7c89b26c3547e1cf803847a7c881be 100644
--- a/ui/views/widget/widget.h
+++ b/ui/views/widget/widget.h
@@ -596,6 +596,9 @@ class VIEWS_EXPORT Widget : public internal::NativeWidgetDelegate,
   }
   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);