Reland: Fix touchpad gesture routing to renderers

content/browser/renderer_host/render_widget_host_input_event_router.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_INPUT_EVENT_ROUTER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_INPUT_EVENT_ROUTER_H_
 
 #include <stdint.h>
 
 #include <deque>
@@ skipping 78 matching lines @@
                          const gfx::Point& point_in_quad_space) override;
     bool AcceptHitTarget(const cc::SurfaceDrawQuad* surface_quad,
                          const gfx::Point& point_in_quad_space) override;
 
     const std::unordered_map<cc::SurfaceId, HittestData, cc::SurfaceIdHash>&
         hittest_data_;
   };
 
   using SurfaceIdNamespaceOwnerMap =
       base::hash_map<uint32_t, RenderWidgetHostViewBase*>;
-  struct GestureTargetData {
+  struct TouchscreenGestureTargetData {
     RenderWidgetHostViewBase* target;
     const gfx::Vector2d delta;
 
-    GestureTargetData(RenderWidgetHostViewBase* target, gfx::Vector2d delta)
+    TouchscreenGestureTargetData(RenderWidgetHostViewBase* target,
+                                 gfx::Vector2d delta)
         : target(target), delta(delta) {}
   };
-  using GestureTargetQueue = std::deque<GestureTargetData>;
+  using TouchscreenGestureTargetQueue =
+      std::deque<TouchscreenGestureTargetData>;
 
   void ClearAllObserverRegistrations();
 
   RenderWidgetHostViewBase* FindEventTarget(RenderWidgetHostViewBase* root_view,
                                             const gfx::Point& point,
                                             gfx::Point* transformed_point);
 
+  void RouteTouchscreenGestureEvent(RenderWidgetHostViewBase* root_view,
+                                    blink::WebGestureEvent* event,
+                                    const ui::LatencyInfo& latency);
+  void RouteTouchpadGestureEvent(RenderWidgetHostViewBase* root_view,
+                                 blink::WebGestureEvent* event,
+                                 const ui::LatencyInfo& latency);
+
   SurfaceIdNamespaceOwnerMap owner_map_;
-  GestureTargetQueue gesture_target_queue_;
+  TouchscreenGestureTargetQueue touchscreen_gesture_target_queue_;
   RenderWidgetHostViewBase* touch_target_;
-  RenderWidgetHostViewBase* gesture_target_;
+  RenderWidgetHostViewBase* touchscreen_gesture_target_;
+  RenderWidgetHostViewBase* touchpad_gesture_target_;
   gfx::Vector2d touch_delta_;
-  gfx::Vector2d gesture_delta_;
+  gfx::Vector2d touchscreen_gesture_delta_;
+  gfx::Vector2d touchpad_gesture_delta_;
   int active_touches_;

[sadrul, 2016/06/08 15:56:30]

Would it make sense to group these variables? e.g.

  struct {
    RWHVBase* target;
    gfx::Vector2d delta;
  } touch_, touchscreen_gesture_, touchpad_gesture_;

[tdresser, 2016/06/08 17:41:12]

+1

We should perhaps rename the TouchscreenGestureTargetData struct to TargetData,
and instantiate it 3 times here?

[mohsen, 2016/06/08 23:32:26]

Cool. That makes it much cleaner. Done.

   std::unordered_map<cc::SurfaceId, HittestData, cc::SurfaceIdHash>
       hittest_data_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostInputEventRouter);
   FRIEND_TEST_ALL_PREFIXES(SitePerProcessBrowserTest,
                            InputEventRouterGestureTargetQueueTest);
+  FRIEND_TEST_ALL_PREFIXES(SitePerProcessBrowserTest,
+                           InputEventRouterTouchpadGestureTargetTest);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_INPUT_EVENT_ROUTER_H_