Fix movementX/Y attributes for touch pointer events

content/browser/renderer_host/render_widget_host_view_base.cc

 // 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.
 
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 
 #include "base/logging.h"
 #include "build/build_config.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
 #include "content/browser/gpu/gpu_data_manager_impl.h"
@@ skipping 489 matching lines @@
 }
 
 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const {
   return false;
 }
 
 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const {
   return cc::SurfaceId();
 }
 
+void RenderWidgetHostViewBase::SetMovementXYForTouchPoints(
+    blink::WebTouchEvent* event) {
+  for (size_t i = 0; i < event->touchesLength; ++i) {
+    auto& touchPoint = event->touches[i];

[sadrul, 2017/02/07 03:48:30]

touch_point

Perhaps you could make this a pointer? (I think we don't use non-const refs in
chromium code very often)

[Navid Zolghadr, 2017/02/08 17:18:49]

Done.

+    if (touchPoint.state == blink::WebTouchPoint::StateMoved) {
+      gfx::Point& lastPosition = global_touch_position_[touchPoint.id];

[sadrul, 2017/02/07 03:48:30]

const gfx::Point& last_position

[Navid Zolghadr, 2017/02/08 17:18:49]

Done.

+      touchPoint.movementX = touchPoint.screenPosition.x - lastPosition.x();
+      touchPoint.movementY = touchPoint.screenPosition.y - lastPosition.y();
+      global_touch_position_[touchPoint.id].SetPoint(
+          touchPoint.screenPosition.x, touchPoint.screenPosition.y);
+    } else {
+      touchPoint.movementX = 0;
+      touchPoint.movementY = 0;
+      if (event->touches[i].state == blink::WebTouchPoint::StateReleased ||
+          event->touches[i].state == blink::WebTouchPoint::StateCancelled) {
+        global_touch_position_.erase(event->touches[i].id);
+      } else if (event->touches[i].state ==
+                 blink::WebTouchPoint::StatePressed) {
+        global_touch_position_[touchPoint.id] = gfx::Point(
+            touchPoint.screenPosition.x, touchPoint.screenPosition.y);
+      }
+    }
+  }
+}
+
 }  // namespace content