[Remoting Android] Remove Synchronizations on RenderData

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.PointF;
 import android.os.SystemClock;
 import android.view.MotionEvent;
@@ skipping 59 matching lines @@
         // on their muscle memory when interacting with our UI.  With respect to the original
         // problem, getScaledDoubleTapSlop() gives a value which is optimized for an Android based
         // UI however this value is too large for interacting with remote elements in our app.
         // Our solution is to use the original value from getScaledDoubleTapSlop() (which includes
         // scaling to account for display differences between devices) and apply a fudge/scale
         // factor to make the interaction more intuitive and useful for our scenario.
         int scaledDoubleTapSlopInPx = config.getScaledDoubleTapSlop();
         scaledDoubleTapSlopInPx *= DOUBLE_TAP_SLOP_SCALE_FACTOR;
         mDoubleTapSlopSquareInPx = scaledDoubleTapSlopInPx * scaledDoubleTapSlopInPx;
 
-        synchronized (mRenderData) {
-            mRenderData.drawCursor = false;
-        }
+        mRenderData.drawCursor = false;
     }
 
     @Override
     public boolean onTap(int button) {
         PointF currentTapPoint = getCursorPosition();
         if (button == InputStub.BUTTON_LEFT) {
             // Left clicks are handled a little differently than the events for other buttons.
             // This is needed because translating touch events to mouse events has a problem with
             // location consistency for double clicks.  If you take the center location of each tap
             // and inject them as mouse clicks, the distance between those two points will often
@@ skipping 54 matching lines @@
     public DesktopView.InputFeedbackType getLongPressFeedbackType() {
         return DesktopView.InputFeedbackType.LARGE_ANIMATION;
     }
 
     @Override
     public boolean isIndirectInputMode() {
         return false;
     }
 
     private PointF getCursorPosition() {
-        synchronized (mRenderData) {
-            return mRenderData.getCursorPosition();
-        }
+        return mRenderData.getCursorPosition();
     }
 
     private boolean isDoubleTap(float currentX, float 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);
-        }
+
+        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);
     }
 }