From 7c69ec7cfebfe2ef3e7dcf09c1365dc11b5fd683 Mon Sep 17 00:00:00 2001 From: Mehdi Alizadeh Date: Tue, 15 May 2018 17:58:44 -0700 Subject: [PATCH] Moves Swipe Up overlay from Settings app to android.internal Bug: 79778025 Test: Robo tests Change-Id: I0cfe5876c5d15552bd4a8e6a96931056846c880d --- res/values/config.xml | 3 --- .../settings/gestures/SwipeUpPreferenceController.java | 6 +++--- .../gestures/SwipeUpPreferenceControllerTest.java | 8 +++----- 3 files changed, 6 insertions(+), 11 deletions(-) diff --git a/res/values/config.xml b/res/values/config.xml index f8a7acc9c61..e3ec74f2636 100755 --- a/res/values/config.xml +++ b/res/values/config.xml @@ -131,7 +131,4 @@ - - false - diff --git a/src/com/android/settings/gestures/SwipeUpPreferenceController.java b/src/com/android/settings/gestures/SwipeUpPreferenceController.java index 27c724194f5..b768a2ce63c 100644 --- a/src/com/android/settings/gestures/SwipeUpPreferenceController.java +++ b/src/com/android/settings/gestures/SwipeUpPreferenceController.java @@ -24,7 +24,7 @@ import android.os.UserManager; import android.provider.Settings; import android.support.annotation.VisibleForTesting; -import com.android.settings.R; +import com.android.internal.R; public class SwipeUpPreferenceController extends GesturePreferenceController { @@ -46,7 +46,7 @@ public class SwipeUpPreferenceController extends GesturePreferenceController { } final ComponentName recentsComponentName = ComponentName.unflattenFromString( - context.getString(com.android.internal.R.string.config_recentsComponentName)); + context.getString(R.string.config_recentsComponentName)); final Intent quickStepIntent = new Intent(ACTION_QUICKSTEP) .setPackage(recentsComponentName.getPackageName()); if (context.getPackageManager().resolveService(quickStepIntent, @@ -81,7 +81,7 @@ public class SwipeUpPreferenceController extends GesturePreferenceController { @Override public boolean isChecked() { final int defaultValue = mContext.getResources() - .getBoolean(com.android.internal.R.bool.config_swipe_up_gesture_default) ? ON : OFF; + .getBoolean(R.bool.config_swipe_up_gesture_default) ? ON : OFF; final int swipeUpEnabled = Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.SWIPE_UP_TO_SWITCH_APPS_ENABLED, defaultValue); return swipeUpEnabled != OFF; diff --git a/tests/robotests/src/com/android/settings/gestures/SwipeUpPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/gestures/SwipeUpPreferenceControllerTest.java index 4107b7398a7..3585650a588 100644 --- a/tests/robotests/src/com/android/settings/gestures/SwipeUpPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/gestures/SwipeUpPreferenceControllerTest.java @@ -27,7 +27,7 @@ import android.content.pm.ResolveInfo; import android.os.UserManager; import android.provider.Settings; -import com.android.settings.R; +import com.android.internal.R; import com.android.settings.testutils.SettingsRobolectricTestRunner; import com.android.settings.testutils.shadow.SettingsShadowResources; @@ -57,8 +57,7 @@ public class SwipeUpPreferenceControllerTest { public void setUp() { SettingsShadowResources.overrideResource(R.bool.config_swipe_up_gesture_setting_available, true); - SettingsShadowResources.overrideResource( - com.android.internal.R.bool.config_swipe_up_gesture_default, true); + SettingsShadowResources.overrideResource(R.bool.config_swipe_up_gesture_default, true); mContext = RuntimeEnvironment.application; mPackageManager = Shadows.shadowOf(mContext.getPackageManager()); @@ -107,8 +106,7 @@ public class SwipeUpPreferenceControllerTest { @Test public void testIsChecked_defaultIsFalse_shouldReturnFalse() { - SettingsShadowResources.overrideResource( - com.android.internal.R.bool.config_swipe_up_gesture_default, false); + SettingsShadowResources.overrideResource(R.bool.config_swipe_up_gesture_default, false); assertThat(mController.isChecked()).isFalse(); }