Enable Android support for Chromoting PINless (paired) authentication

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.app.ActionBar;
 import android.app.Activity;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
@@ skipping 391 matching lines @@
         public void onLongPress(MotionEvent e) {
             if (e.getPointerCount() > 1) {
                 return;
             }
 
             float[] coordinates = new float[] {e.getRawX(), e.getY()};
 
             Log.i("mouse", "Finger held down");
             if (mMousePressed) {
                 Log.i("mouse", "\tReleasing the currently-pressed button");
-                handleMouseMovement(coordinates, mMouseButton, false);
+                handleMouseMovement(new float[] {coordinates[0], coordinates[1]},

[garykac, 2013/08/06 20:50:32]

Why is this necessary?

[solb, 2013/08/07 04:30:46]

Because in Java, arrays are objects and objects are passed by reference, so this
method would actually mutate the coordinates array. As you point out (and as
evidenced by the fact that this bug was present), this was a poor interface. As
such, I changed the handleMouseMovement() signature to instead accept primitive
ints.

+                        mMouseButton, false);
             }
 
             Log.i("mouse", "\tStarting right click");
             mMouseButton = BUTTON_RIGHT;
             mMousePressed = true;
             handleMouseMovement(coordinates, mMouseButton, mMousePressed);
         }
     }
 }