Support targeting for gestures in ViewTargeter

ui/views/widget/root_view.h

Index: ui/views/widget/root_view.h
diff --git a/ui/views/widget/root_view.h b/ui/views/widget/root_view.h
index 97ce1d5cb2cbbc157b093dc482b30ac941528540..533d69bd875dfe6db4d2c452539e622fef74a24d 100644
--- a/ui/views/widget/root_view.h
+++ b/ui/views/widget/root_view.h
@@ -17,9 +17,11 @@
 namespace views {
 
 namespace test {
+class ViewTargeterTest;
 class WidgetTest;
 }
 
+class ViewTargeter;
 class Widget;
 
 // This is a views-internal API and should not be used externally.
@@ -123,7 +125,9 @@ class VIEWS_EXPORT RootView : public View,
 
  private:
   friend class ::views::View;
+  friend class ::views::ViewTargeter;
   friend class ::views::Widget;
+  friend class ::views::test::ViewTargeterTest;
   friend class ::views::test::WidgetTest;
 
   // Input ---------------------------------------------------------------------
@@ -187,10 +191,13 @@ class VIEWS_EXPORT RootView : public View,
   int last_mouse_event_x_;
   int last_mouse_event_y_;
 
-  // The view currently handling gesture events. When set, this handler receives
-  // all gesture events.
+  // The View currently handling gesture events.
   View* gesture_handler_;
 
+  // If true, then gesture events received from Widget should be dispatched to
+  // |gesture_handler_|.
+  bool dispatch_to_gesture_handler_;

[sadrul, 2014/08/26 16:21:44]

Is the main purpose here to stop the bubbling of the event? Would
'allow_gesture_bubbling_' be clearer/more appropriate?

[tdanderson, 2014/08/26 20:21:15]

Yes, that's pretty much the purpose of it. Changed the name as you suggested. I
was hesitant to name it something similar in the first place because even if the
member is false, we do still allow bubbling for SCROLL_BEGIN. But I do like it
better than the current name.

+
   scoped_ptr<internal::PreEventDispatchHandler> pre_dispatch_handler_;
   scoped_ptr<internal::PostEventDispatchHandler> post_dispatch_handler_;