[Remoting Android] Use floating point coords for rendering the cursor

remoting/android/java/src/org/chromium/chromoting/SimulatedTouchInputStrategy.java

 // 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.
 
 package org.chromium.chromoting;
 
 import android.content.Context;
 import android.graphics.Point;
 import android.os.SystemClock;
 import android.view.MotionEvent;
@@ skipping 135 matching lines @@
     @Override
     public DesktopView.InputFeedbackType getLongPressFeedbackType() {
         return DesktopView.InputFeedbackType.LARGE_ANIMATION;
     }
 
     @Override
     public boolean isIndirectInputMode() {
         return false;
     }
 
     private Point getCursorPosition() {

[joedow, 2016/08/17 17:08:15]

Why not use the floating point cursor position everywhere?

IIRC we need to use int when sending the cursor pos to the remote machine, but I
think being consistent internally (in the client) will reduce the likelihood of
float->int->float precision loss bugs.

[Yuwei, 2016/08/17 17:42:45]

Done. Now using float everywhere and will convert them into int in
InputEventSender.

         synchronized (mRenderData) {
-            return mRenderData.getCursorPosition();
+            return mRenderData.getIntegerCursorPosition();
         }
     }
 
     private boolean isDoubleTap(int currentX, int currentY, long tapInterval) {
         if (tapInterval > mDoubleTapDurationInMs || mLastTapPoint == null) {
             return false;
         }
 
         // Convert the image based coordinates back to screen coordinates so the user experiences
         // consistent double tap behavior regardless of zoom level.
         //
         float[] currentValues = {currentX, currentY};
         float[] previousValues = {mLastTapPoint.x, mLastTapPoint.y};
         synchronized (mRenderData) {
             mRenderData.transform.mapPoints(currentValues);
             mRenderData.transform.mapPoints(previousValues);
         }
 
         int deltaX = (int) (currentValues[0] - previousValues[0]);
         int deltaY = (int) (currentValues[1] - previousValues[1]);
         return ((deltaX * deltaX + deltaY * deltaY) <= mDoubleTapSlopSquareInPx);
     }
 }