[Remoting Android] Define feedback animation size in dp

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

 // 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.
 
 package org.chromium.chromoting;
 
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.Paint;
@@ skipping 206 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) {
-        if (feedbackToShow != InputFeedbackType.NONE) {

[Yuwei, 2016/07/25 23:43:12]

Removed this check since NONE is not been used anywhere else...

[joedow, 2016/07/26 03:03:24]

It is set in a few places.  The check was useful a few refactorings ago, I'd
make sure it is not not needed for all input types and scenarios (IIRC the old
animator had a problem if it was called with a zero radius which is why this was
important).

[Yuwei, 2016/07/26 18:47:26]

You are right. Looks like it is set in several places. We probably still need to
check this value unless it is actually removed from the code... Just added back
the check.

-            FeedbackAnimator.startAnimation(this, pos, feedbackToShow);
-            requestRepaint();
-        }
+    public void showInputFeedback(float feedbackRadius, Point pos) {
+        FeedbackAnimator.startAnimation(this, pos, feedbackRadius);
+        requestRepaint();
     }
 
     @Override
     public void transformationChanged() {
         requestRepaint();
     }
 
     @Override
     public void cursorMoved() {
         // For current implementation, cursorMoved() is always followed by transformationChanged()
         // even if the canvas isn't really changed. For future we should improve this by not calling
         // transformationChanged() if the cursor is moved but the canvas is not changed.
     }
 
     @Override
     public void cursorVisibilityChanged() {
         requestRepaint();
     }
 
     @Override
     public void setAnimationEnabled(boolean enabled) {
         synchronized (mAnimationLock) {
             if (enabled && !mInputAnimationRunning) {
                 requestRepaint();
             }
             mInputAnimationRunning = enabled;
         }
     }
 }