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

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

 // Copyright 2013 The Chromium Authors. All rights reserved.

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

I think you should just delete the non-OpenGl files.  This codepath isn't being
tested and I don't think keeping them around is worth the maintenance cost.

An example is the getIntegerCursorPosition() method.  Perhaps you use it later
and I haven't seen it yet, but keeping this old code around requires more effort
on your part and the part of the reviewer. 

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

Yes. I'm planning to delete them in a separate CL.
I used it somewhere else. It's removed now.

 // 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.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.Paint;
 import android.graphics.Point;
+import android.graphics.PointF;
 import android.os.Looper;
 import android.os.SystemClock;
 import android.view.SurfaceHolder;
 
 import org.chromium.base.Log;
 import org.chromium.chromoting.jni.Client;
 import org.chromium.chromoting.jni.Display;
 
 /**
  * The user interface for viewing and interacting with a specific remote host.
@@ skipping 103 matching lines @@
             // destroying the Surface until it is unlocked.
             if (!mSurfaceCreated) {
                 return;
             }
             canvas = getHolder().lockCanvas();
             if (canvas == null) {
                 return;
             }
             canvas.setMatrix(mRenderData.transform);
             drawCursor = mRenderData.drawCursor;
-            cursorPosition = mRenderData.getCursorPosition();
+            cursorPosition = mRenderData.getIntegerCursorPosition();
         }
 
         canvas.drawColor(Color.BLACK);
         canvas.drawBitmap(image, 0, 0, new Paint());
 
         float scaleFactor;
         synchronized (mRenderData) {
             scaleFactor = mRenderData.transform.mapRadius(1);
         }
         mOnPaint.raise(new PaintEventParameter(cursorPosition, canvas, scaleFactor));
@@ skipping 71 matching lines @@
      * will not be blank if the user later switches back to our window.
      */
     @Override
     public void surfaceDestroyed(SurfaceHolder holder) {
         synchronized (mRenderData) {
             mSurfaceCreated = false;
         }
     }
 
     @Override
-    public void showInputFeedback(InputFeedbackType feedbackToShow, Point pos) {
+    public void showInputFeedback(InputFeedbackType feedbackToShow, PointF pos) {
         float radius = getFeedbackRadius(feedbackToShow);
         if (radius <= 0.0f) {
             return;
         }
-        FeedbackAnimator.startAnimation(this, pos, radius);
+        FeedbackAnimator.startAnimation(this, new Point((int) pos.x, (int) pos.y), radius);
         requestRepaint();
     }
 
     @Override
     public void transformationChanged() {
         requestRepaint();
     }
 
     @Override
     public void cursorMoved() {
@@ skipping 10 matching lines @@
     @Override
     public void setAnimationEnabled(boolean enabled) {
         synchronized (mAnimationLock) {
             if (enabled && !mInputAnimationRunning) {
                 requestRepaint();
             }
             mInputAnimationRunning = enabled;
         }
     }
 }