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

Side by Side Diff: remoting/android/java/src/org/chromium/chromoting/RenderData.java

Issue 2272483002: [Remoting Android] Remove Synchronizations on RenderData (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chromoting; 5 package org.chromium.chromoting;
6 6
7 import android.graphics.Matrix; 7 import android.graphics.Matrix;
8 import android.graphics.PointF; 8 import android.graphics.PointF;
9 9
10 /** 10 /**
11 * This class stores data that needs to be accessed on both the display thread a nd the 11 * This class stores data that needs to be accessed on the event-processing thre ad.
joedow 2016/08/23 17:37:45 I would probably update this a bit more since thre
Yuwei 2016/08/23 17:57:32 Maybe something like "This class stores UI positio
12 * event-processing thread.
13 */ 12 */
14 public class RenderData { 13 public class RenderData {
15 /** Stores pan and zoom configuration and converts image coordinates to scre en coordinates. */ 14 /** Stores pan and zoom configuration and converts image coordinates to scre en coordinates. */
16 public Matrix transform = new Matrix(); 15 public Matrix transform = new Matrix();
17 16
18 public int screenWidth = 0; 17 public int screenWidth = 0;
19 public int screenHeight = 0; 18 public int screenHeight = 0;
20 public int imageWidth = 0; 19 public int imageWidth = 0;
21 public int imageHeight = 0; 20 public int imageHeight = 0;
22 21
(...skipping 29 matching lines...) Expand all
52 cursorMoved = true; 51 cursorMoved = true;
53 } 52 }
54 if (newY != mCursorPosition.y) { 53 if (newY != mCursorPosition.y) {
55 mCursorPosition.y = newY; 54 mCursorPosition.y = newY;
56 cursorMoved = true; 55 cursorMoved = true;
57 } 56 }
58 57
59 return cursorMoved; 58 return cursorMoved;
60 } 59 }
61 } 60 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698