Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(370)

Unified Diff: remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java

Issue 2272483002: [Remoting Android] Remove Synchronizations on RenderData (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reviewer's Feedback / Merge ToT Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java
diff --git a/remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java b/remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java
index 87480a24bf0d3f6e22effea96233dad787f2a941..e009e74540ece14e300eabb236e2e10bce5e3ea7 100644
--- a/remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java
+++ b/remoting/android/java/src/org/chromium/chromoting/TouchInputStrategy.java
@@ -56,9 +56,7 @@ public class TouchInputStrategy implements InputStrategyInterface {
mInjector = injector;
mQueuedEvents = new LinkedList<MotionEvent>();
- synchronized (mRenderData) {
- mRenderData.drawCursor = false;
- }
+ mRenderData.drawCursor = false;
}
@Override
@@ -207,13 +205,12 @@ public class TouchInputStrategy implements InputStrategyInterface {
// Use a copy of the original event so the original event can be passed to other
// detectors/handlers in an unmodified state.
event = MotionEvent.obtain(event);
- synchronized (mRenderData) {
- // Transform the event coordinates so they represent the remote screen coordinates
- // instead of the local touch display.
- Matrix inverted = new Matrix();
- mRenderData.transform.invert(inverted);
- event.transform(inverted);
- }
+
+ // Transform the event coordinates so they represent the remote screen coordinates
+ // instead of the local touch display.
+ Matrix inverted = new Matrix();
+ mRenderData.transform.invert(inverted);
+ event.transform(inverted);
return event;
}

Powered by Google App Engine
This is Rietveld 408576698