Fix movementX/Y attributes for touch pointer events

content/browser/renderer_host/input/input_router.h

 // Copyright 2013 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_INPUT_INPUT_ROUTER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_INPUT_INPUT_ROUTER_H_
 
 #include "content/browser/renderer_host/event_with_latency_info.h"
 #include "content/common/input/input_event_ack_state.h"
 #include "content/public/browser/native_web_keyboard_event.h"
@@ skipping 17 matching lines @@
 
   // WebInputEvents
   virtual void SendMouseEvent(
       const MouseEventWithLatencyInfo& mouse_event) = 0;
   virtual void SendWheelEvent(
       const MouseWheelEventWithLatencyInfo& wheel_event) = 0;
   virtual void SendKeyboardEvent(
       const NativeWebKeyboardEventWithLatencyInfo& key_event) = 0;
   virtual void SendGestureEvent(
       const GestureEventWithLatencyInfo& gesture_event) = 0;
-  virtual void SendTouchEvent(
-      const TouchEventWithLatencyInfo& touch_event) = 0;
+  virtual void SendTouchEvent(TouchEventWithLatencyInfo touch_event) = 0;

[sadrul, 2017/02/09 03:11:34]

I suggest keep this a const-ref, and make a copy in the implementation in
InputRouterImpl::SendTouchEvent() instead.

 
   // Returns the oldest queued or in-flight keyboard event sent to the router.
   virtual const NativeWebKeyboardEvent* GetLastKeyboardEvent() const = 0;
 
   // Notify the router about whether the current page is mobile-optimized (i.e.,
   // the site has a mobile-friendly viewport).
   virtual void NotifySiteIsMobileOptimized(bool is_mobile_optimized) = 0;
 
   // Request a notification from the input router when all events have been
   // fully dispatched and there are no longer any pending events.
   // Note: This may trigger a synchronous notification if the router is empty.
   virtual void RequestNotificationWhenFlushed() = 0;
 
   // Whether there are any events pending dispatch to or ack from the renderer.
   virtual bool HasPendingEvents() const = 0;
 
   // A scale factor to scale the coordinate in WebInputEvent from DIP
   // to viewport.
   virtual void SetDeviceScaleFactor(float device_scale_factor) = 0;
 
   // Sets the frame tree node id of associated frame, used when tracing
   // input event latencies to relate events to their target frames.
   virtual void SetFrameTreeNodeId(int frameTreeNodeId) = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_INPUT_ROUTER_H_