Chrome requires WebTouchPoint to store WebFloatPoint, instead of WebPoint.

content/browser/renderer_host/input/touch_event_queue.cc

 // 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.
 
 #include "content/browser/renderer_host/input/touch_event_queue.h"
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/stl_util.h"
@@ skipping 151 matching lines @@
   TouchEventWithLatencyInfo timeout_event_;
 
   // Provides timeout-based callback behavior.
   TimeoutMonitor timeout_monitor_;
 };
 
 // Provides touchmove slop suppression for a single touch that remains within
 // a given slop region, unless the touchstart is preventDefault'ed.
 class TouchEventQueue::TouchMoveSlopSuppressor {
  public:
-  // TODO(jdduke): Remove int cast on suppression length, crbug.com/336807.
   TouchMoveSlopSuppressor(double slop_suppression_length_dips)
-      : slop_suppression_length_dips_squared_(
-            static_cast<int>(slop_suppression_length_dips) *
-            static_cast<int>(slop_suppression_length_dips)),
+      : slop_suppression_length_dips_squared_(slop_suppression_length_dips *
+                                              slop_suppression_length_dips),
         suppressing_touch_moves_(false) {}
 
   bool FilterEvent(const WebTouchEvent& event) {
     if (WebTouchEventTraits::IsTouchSequenceStart(event)) {
       touch_sequence_start_position_ =
-          gfx::PointF(event.touches[0].position.x, event.touches[0].position.y);
+          gfx::PointF(event.touches[0].position);
       suppressing_touch_moves_ = slop_suppression_length_dips_squared_ != 0;
     }
 
     if (event.type != WebInputEvent::TouchMove)
       return false;
 
     if (suppressing_touch_moves_) {
       // Movement with a secondary pointer should terminate suppression.
       if (event.touchesLength > 1) {
         suppressing_touch_moves_ = false;
       } else if (event.touchesLength == 1) {
         // Movement outside of the slop region should terminate suppression.
-        // TODO(jdduke): Use strict inequality, crbug.com/336807.
-        gfx::PointF position(event.touches[0].position.x,
-                             event.touches[0].position.y);
-        if ((position - touch_sequence_start_position_).LengthSquared() >=
-                slop_suppression_length_dips_squared_)
+        gfx::PointF position(event.touches[0].position);
+        if ((position - touch_sequence_start_position_).LengthSquared() >
+            slop_suppression_length_dips_squared_)
           suppressing_touch_moves_ = false;
       }
     }
     return suppressing_touch_moves_;
   }
 
   void ConfirmTouchEvent(InputEventAckState ack_result) {
     if (ack_result == INPUT_EVENT_ACK_STATE_CONSUMED)
       suppressing_touch_moves_ = false;
   }
@@ skipping 422 matching lines @@
   } else if (event.type == WebInputEvent::TouchStart) {
     for (unsigned i = 0; i < event.touchesLength; ++i) {
       const WebTouchPoint& point = event.touches[i];
       if (point.state == WebTouchPoint::StatePressed)
         touch_ack_states_[point.id] = ack_result;
     }
   }
 }
 
 }  // namespace content