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

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

Issue 2410353002: Moving logic for System UI state changes into DesktopCanvas class (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
diff --git a/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java b/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
index 7894bd993a5705941d710d8c9dca727538a5c181..d73ef5a121d7ea9c9502c842b2fecb60dc0298ce 100644
--- a/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
+++ b/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
@@ -250,7 +250,7 @@ public class TouchInputHandler {
new Event.ParameterRunnable<SystemUiVisibilityChangedEventParameter>() {
@Override
public void run(SystemUiVisibilityChangedEventParameter parameter) {
- handleSystemUiVisibilityChanged(parameter);
+ mDesktopCanvas.onSystemUiVisibilityChanged(parameter);
}
});
@@ -348,17 +348,6 @@ public class TouchInputHandler {
mRenderStub.setCursorVisibility(mRenderData.drawCursor);
}
- private void handleSystemUiVisibilityChanged(
- SystemUiVisibilityChangedEventParameter parameter) {
- if (parameter.systemUiVisible) {
- mDesktopCanvas.setSystemUiOffsetValues(parameter.left, parameter.top,
- mRenderData.screenWidth - parameter.right,
- mRenderData.screenHeight - parameter.bottom);
- } else {
- mDesktopCanvas.clearSystemUiOffsets();
- }
- }
-
private boolean handleTouchEvent(MotionEvent event) {
// Give the underlying input strategy a chance to observe the current motion event before
// passing it to the gesture detectors. This allows the input strategy to react to the
« no previous file with comments | « remoting/android/java/src/org/chromium/chromoting/DesktopCanvas.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698