Merge "Use InputSettings in Settings app"

This commit is contained in:
Josep del Río
2023-02-14 17:18:40 +00:00
committed by Android (Google) Code Review
6 changed files with 31 additions and 45 deletions

View File

@@ -24,6 +24,7 @@ import android.content.ContentResolver;
import android.content.Context; import android.content.Context;
import android.database.ContentObserver; import android.database.ContentObserver;
import android.hardware.input.InputManager; import android.hardware.input.InputManager;
import android.hardware.input.InputSettings;
import android.os.Handler; import android.os.Handler;
import android.os.Parcel; import android.os.Parcel;
import android.os.Parcelable; import android.os.Parcelable;
@@ -74,16 +75,16 @@ public class PointerSpeedPreference extends SeekBarDialogPreference implements
super.onBindDialogView(view); super.onBindDialogView(view);
mSeekBar = getSeekBar(view); mSeekBar = getSeekBar(view);
mSeekBar.setMax(InputManager.MAX_POINTER_SPEED - InputManager.MIN_POINTER_SPEED); mSeekBar.setMax(InputSettings.MAX_POINTER_SPEED - InputSettings.MIN_POINTER_SPEED);
mOldSpeed = mIm.getPointerSpeed(getContext()); mOldSpeed = InputSettings.getPointerSpeed(getContext());
mSeekBar.setProgress(mOldSpeed - InputManager.MIN_POINTER_SPEED); mSeekBar.setProgress(mOldSpeed - InputSettings.MIN_POINTER_SPEED);
mSeekBar.setOnSeekBarChangeListener(this); mSeekBar.setOnSeekBarChangeListener(this);
mSeekBar.setContentDescription(getTitle()); mSeekBar.setContentDescription(getTitle());
} }
public void onProgressChanged(SeekBar seekBar, int progress, boolean fromTouch) { public void onProgressChanged(SeekBar seekBar, int progress, boolean fromTouch) {
if (!mTouchInProgress) { if (!mTouchInProgress) {
mIm.tryPointerSpeed(progress + InputManager.MIN_POINTER_SPEED); mIm.tryPointerSpeed(progress + InputSettings.MIN_POINTER_SPEED);
} }
if (progress != mLastProgress) { if (progress != mLastProgress) {
seekBar.performHapticFeedback(CLOCK_TICK); seekBar.performHapticFeedback(CLOCK_TICK);
@@ -100,13 +101,13 @@ public class PointerSpeedPreference extends SeekBarDialogPreference implements
public void onStopTrackingTouch(SeekBar seekBar) { public void onStopTrackingTouch(SeekBar seekBar) {
mTouchInProgress = false; mTouchInProgress = false;
mIm.tryPointerSpeed(seekBar.getProgress() + InputManager.MIN_POINTER_SPEED); mIm.tryPointerSpeed(seekBar.getProgress() + InputSettings.MIN_POINTER_SPEED);
mJankMonitor.end(CUJ_SETTINGS_SLIDER); mJankMonitor.end(CUJ_SETTINGS_SLIDER);
} }
private void onSpeedChanged() { private void onSpeedChanged() {
int speed = mIm.getPointerSpeed(getContext()); int speed = InputSettings.getPointerSpeed(getContext());
mSeekBar.setProgress(speed - InputManager.MIN_POINTER_SPEED); mSeekBar.setProgress(speed - InputSettings.MIN_POINTER_SPEED);
} }
@Override @Override
@@ -116,8 +117,8 @@ public class PointerSpeedPreference extends SeekBarDialogPreference implements
final ContentResolver resolver = getContext().getContentResolver(); final ContentResolver resolver = getContext().getContentResolver();
if (positiveResult) { if (positiveResult) {
mIm.setPointerSpeed(getContext(), InputSettings.setPointerSpeed(getContext(),
mSeekBar.getProgress() + InputManager.MIN_POINTER_SPEED); mSeekBar.getProgress() + InputSettings.MIN_POINTER_SPEED);
} else { } else {
restoreOldState(); restoreOldState();
} }
@@ -158,7 +159,7 @@ public class PointerSpeedPreference extends SeekBarDialogPreference implements
SavedState myState = (SavedState) state; SavedState myState = (SavedState) state;
super.onRestoreInstanceState(myState.getSuperState()); super.onRestoreInstanceState(myState.getSuperState());
mOldSpeed = myState.oldSpeed; mOldSpeed = myState.oldSpeed;
mIm.tryPointerSpeed(myState.progress + InputManager.MIN_POINTER_SPEED); mIm.tryPointerSpeed(myState.progress + InputSettings.MIN_POINTER_SPEED);
} }
private static class SavedState extends BaseSavedState { private static class SavedState extends BaseSavedState {

View File

@@ -17,29 +17,25 @@
package com.android.settings.inputmethod; package com.android.settings.inputmethod;
import android.content.Context; import android.content.Context;
import android.hardware.input.InputManager; import android.hardware.input.InputSettings;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.core.TogglePreferenceController; import com.android.settings.core.TogglePreferenceController;
public class TrackpadBottomPreferenceController extends TogglePreferenceController { public class TrackpadBottomPreferenceController extends TogglePreferenceController {
private InputManager mIm;
public TrackpadBottomPreferenceController(Context context, String key) { public TrackpadBottomPreferenceController(Context context, String key) {
super(context, key); super(context, key);
mIm = context.getSystemService(InputManager.class);
} }
@Override @Override
public boolean isChecked() { public boolean isChecked() {
return mIm.useTouchpadRightClickZone(mContext); return InputSettings.useTouchpadRightClickZone(mContext);
} }
@Override @Override
public boolean setChecked(boolean isChecked) { public boolean setChecked(boolean isChecked) {
mIm.setTouchpadRightClickZone(mContext, isChecked); InputSettings.setTouchpadRightClickZone(mContext, isChecked);
return true; return true;
} }

View File

@@ -17,7 +17,7 @@
package com.android.settings.inputmethod; package com.android.settings.inputmethod;
import android.content.Context; import android.content.Context;
import android.hardware.input.InputManager; import android.hardware.input.InputSettings;
import androidx.preference.PreferenceScreen; import androidx.preference.PreferenceScreen;
@@ -26,13 +26,10 @@ import com.android.settings.widget.SeekBarPreference;
public class TrackpadPointerSpeedPreferenceController extends SliderPreferenceController { public class TrackpadPointerSpeedPreferenceController extends SliderPreferenceController {
private InputManager mIm;
private SeekBarPreference mPreference; private SeekBarPreference mPreference;
public TrackpadPointerSpeedPreferenceController(Context context, String key) { public TrackpadPointerSpeedPreferenceController(Context context, String key) {
super(context, key); super(context, key);
mIm = context.getSystemService(InputManager.class);
} }
@Override @Override
@@ -55,22 +52,22 @@ public class TrackpadPointerSpeedPreferenceController extends SliderPreferenceCo
if (position < getMin() || position > getMax()) { if (position < getMin() || position > getMax()) {
return false; return false;
} }
mIm.setTouchpadPointerSpeed(mContext, position); InputSettings.setTouchpadPointerSpeed(mContext, position);
return true; return true;
} }
@Override @Override
public int getSliderPosition() { public int getSliderPosition() {
return mIm.getTouchpadPointerSpeed(mContext); return InputSettings.getTouchpadPointerSpeed(mContext);
} }
@Override @Override
public int getMin() { public int getMin() {
return InputManager.MIN_POINTER_SPEED; return InputSettings.MIN_POINTER_SPEED;
} }
@Override @Override
public int getMax() { public int getMax() {
return InputManager.MAX_POINTER_SPEED; return InputSettings.MAX_POINTER_SPEED;
} }
} }

View File

@@ -17,29 +17,25 @@
package com.android.settings.inputmethod; package com.android.settings.inputmethod;
import android.content.Context; import android.content.Context;
import android.hardware.input.InputManager; import android.hardware.input.InputSettings;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.core.TogglePreferenceController; import com.android.settings.core.TogglePreferenceController;
public class TrackpadReverseScrollingPreferenceController extends TogglePreferenceController { public class TrackpadReverseScrollingPreferenceController extends TogglePreferenceController {
private InputManager mIm;
public TrackpadReverseScrollingPreferenceController(Context context, String key) { public TrackpadReverseScrollingPreferenceController(Context context, String key) {
super(context, key); super(context, key);
mIm = context.getSystemService(InputManager.class);
} }
@Override @Override
public boolean isChecked() { public boolean isChecked() {
return mIm.useTouchpadNaturalScrolling(mContext); return InputSettings.useTouchpadNaturalScrolling(mContext);
} }
@Override @Override
public boolean setChecked(boolean isChecked) { public boolean setChecked(boolean isChecked) {
mIm.setTouchpadNaturalScrolling(mContext, isChecked); InputSettings.setTouchpadNaturalScrolling(mContext, isChecked);
return true; return true;
} }

View File

@@ -17,29 +17,25 @@
package com.android.settings.inputmethod; package com.android.settings.inputmethod;
import android.content.Context; import android.content.Context;
import android.hardware.input.InputManager; import android.hardware.input.InputSettings;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.core.TogglePreferenceController; import com.android.settings.core.TogglePreferenceController;
public class TrackpadTapToClickPreferenceController extends TogglePreferenceController { public class TrackpadTapToClickPreferenceController extends TogglePreferenceController {
private InputManager mIm;
public TrackpadTapToClickPreferenceController(Context context, String key) { public TrackpadTapToClickPreferenceController(Context context, String key) {
super(context, key); super(context, key);
mIm = context.getSystemService(InputManager.class);
} }
@Override @Override
public boolean isChecked() { public boolean isChecked() {
return mIm.useTouchpadTapToClick(mContext); return InputSettings.useTouchpadTapToClick(mContext);
} }
@Override @Override
public boolean setChecked(boolean isChecked) { public boolean setChecked(boolean isChecked) {
mIm.setTouchpadTapToClick(mContext, isChecked); InputSettings.setTouchpadTapToClick(mContext, isChecked);
return true; return true;
} }

View File

@@ -19,7 +19,7 @@ package com.android.settings.inputmethod;
import static com.google.common.truth.Truth.assertThat; import static com.google.common.truth.Truth.assertThat;
import android.content.Context; import android.content.Context;
import android.hardware.input.InputManager; import android.hardware.input.InputSettings;
import android.os.UserHandle; import android.os.UserHandle;
import android.provider.Settings; import android.provider.Settings;
@@ -50,7 +50,7 @@ public class TrackpadPointerSpeedPreferenceControllerTest {
mDefaultSpeed = Settings.System.getIntForUser( mDefaultSpeed = Settings.System.getIntForUser(
mContext.getContentResolver(), mContext.getContentResolver(),
SETTING_KEY, SETTING_KEY,
InputManager.DEFAULT_POINTER_SPEED, InputSettings.DEFAULT_POINTER_SPEED,
UserHandle.USER_CURRENT); UserHandle.USER_CURRENT);
} }
@@ -62,12 +62,12 @@ public class TrackpadPointerSpeedPreferenceControllerTest {
@Test @Test
public void getMin_expected() { public void getMin_expected() {
assertThat(mController.getMin()).isEqualTo(InputManager.MIN_POINTER_SPEED); assertThat(mController.getMin()).isEqualTo(InputSettings.MIN_POINTER_SPEED);
} }
@Test @Test
public void getMax_expected() { public void getMax_expected() {
assertThat(mController.getMax()).isEqualTo(InputManager.MAX_POINTER_SPEED); assertThat(mController.getMax()).isEqualTo(InputSettings.MAX_POINTER_SPEED);
} }
@Test @Test
@@ -89,7 +89,7 @@ public class TrackpadPointerSpeedPreferenceControllerTest {
@Test @Test
public void setSliderPosition_speedValueOverMaxValue_shouldReturnFalse() { public void setSliderPosition_speedValueOverMaxValue_shouldReturnFalse() {
int inputSpeed = InputManager.MAX_POINTER_SPEED + 1; int inputSpeed = InputSettings.MAX_POINTER_SPEED + 1;
boolean result = mController.setSliderPosition(inputSpeed); boolean result = mController.setSliderPosition(inputSpeed);
@@ -99,7 +99,7 @@ public class TrackpadPointerSpeedPreferenceControllerTest {
@Test @Test
public void setSliderPosition_speedValueOverMinValue_shouldReturnFalse() { public void setSliderPosition_speedValueOverMinValue_shouldReturnFalse() {
int inputSpeed = InputManager.MIN_POINTER_SPEED - 1; int inputSpeed = InputSettings.MIN_POINTER_SPEED - 1;
boolean result = mController.setSliderPosition(inputSpeed); boolean result = mController.setSliderPosition(inputSpeed);