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

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

Issue 2391993004: [Remoting Android] Hide soft keyboard button when physical keyboard is connected (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
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 64d0da9ec9d5d0b93bc95bb08297e34dc926c828..89995d6617a92290501075e489d7361de38cfdc2 100644
--- a/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
+++ b/remoting/android/java/src/org/chromium/chromoting/TouchInputHandler.java
@@ -26,8 +26,7 @@ public class TouchInputHandler {
private static final float EPSILON = 0.001f;
private final List<Pair<Object, Event<?>>> mAttachedEvents = new ArrayList<>();
- private final DesktopView mViewer;
- private final Context mContext;
+ private final Desktop mDesktop;
private final RenderData mRenderData;
private final DesktopCanvas mDesktopCanvas;
private final RenderStub mRenderStub;
@@ -194,8 +193,7 @@ public class TouchInputHandler {
Preconditions.notNull(renderStub);
Preconditions.notNull(injector);
- mViewer = viewer;
- mContext = desktop;
+ mDesktop = desktop;
mRenderStub = renderStub;
mRenderData = new RenderData();
mDesktopCanvas = new DesktopCanvas(renderStub, mRenderData);
@@ -233,7 +231,7 @@ public class TouchInputHandler {
}
};
- attachEvent(mViewer.onTouch(), new Event.ParameterRunnable<TouchEventParameter>() {
+ attachEvent(viewer.onTouch(), new Event.ParameterRunnable<TouchEventParameter>() {
@Override
public void run(TouchEventParameter parameter) {
parameter.handled = handleTouchEvent(parameter.event);
@@ -335,7 +333,7 @@ public class TouchInputHandler {
setInputStrategy(new TouchInputStrategy(mRenderData, injector));
} else {
setInputStrategy(
- new SimulatedTouchInputStrategy(mRenderData, injector, mContext));
+ new SimulatedTouchInputStrategy(mRenderData, injector, mDesktop));
}
break;
@@ -477,10 +475,10 @@ public class TouchInputHandler {
private boolean onSwipe() {
if (mTotalMotionY > mSwipeThreshold) {
// Swipe down occurred.
- mViewer.showActionBar();
+ mDesktop.showSystemUi();
} else if (mTotalMotionY < -mSwipeThreshold) {
// Swipe up occurred.
- mViewer.showKeyboard();
+ mDesktop.showKeyboard();
} else {
return false;
}

Powered by Google App Engine
This is Rietveld 408576698