diff options
| author | 2020-04-09 16:32:49 -0700 | |
|---|---|---|
| committer | 2020-04-20 19:21:55 +0000 | |
| commit | b20aa8b11014d5f05ab2e02dfd1482fc539eeaa6 (patch) | |
| tree | 0cddade3131f40864c62c3c6006e226234ae91c8 | |
| parent | 03f9a814a5bfac260a781780824c90a59e5d8675 (diff) | |
Don't reset adaptive_sleep setting
Always respect user setting since we can't meaningfully recover from
the loss.
Bug: 153765285
Test: manually
Change-Id: I29f3b82e2c40d80c36721882724846a7439454eb
| -rw-r--r-- | services/core/java/com/android/server/power/AttentionDetector.java | 4 | ||||
| -rw-r--r-- | services/tests/servicestests/src/com/android/server/power/AttentionDetectorTest.java | 11 |
2 files changed, 0 insertions, 15 deletions
diff --git a/services/core/java/com/android/server/power/AttentionDetector.java b/services/core/java/com/android/server/power/AttentionDetector.java index 1ab6adee2320..b69c45070487 100644 --- a/services/core/java/com/android/server/power/AttentionDetector.java +++ b/services/core/java/com/android/server/power/AttentionDetector.java @@ -17,7 +17,6 @@ package com.android.server.power; import static android.provider.DeviceConfig.NAMESPACE_ATTENTION_MANAGER_SERVICE; -import static android.provider.Settings.Secure.ADAPTIVE_SLEEP; import android.Manifest; import android.app.ActivityManager; @@ -192,9 +191,6 @@ public class AttentionDetector { } if (!isAttentionServiceSupported() || !serviceHasSufficientPermissions()) { - // Turns off adaptive sleep in settings for all users if attention service is not - // available. The setting itself should also be grayed out in this case. - Settings.Secure.putInt(mContentResolver, ADAPTIVE_SLEEP, 0); return nextScreenDimming; } diff --git a/services/tests/servicestests/src/com/android/server/power/AttentionDetectorTest.java b/services/tests/servicestests/src/com/android/server/power/AttentionDetectorTest.java index c4289efe1839..e7e8aca86364 100644 --- a/services/tests/servicestests/src/com/android/server/power/AttentionDetectorTest.java +++ b/services/tests/servicestests/src/com/android/server/power/AttentionDetectorTest.java @@ -158,17 +158,6 @@ public class AttentionDetectorTest extends AndroidTestCase { } @Test - public void testOnUserActivity_disablesSettingIfNotSufficientPermissions() { - when(mPackageManager.checkPermission(any(), any())).thenReturn( - PackageManager.PERMISSION_DENIED); - - registerAttention(); - boolean enabled = Settings.Secure.getIntForUser(getContext().getContentResolver(), - Settings.Secure.ADAPTIVE_SLEEP, 0, UserHandle.USER_CURRENT) == 1; - assertFalse(enabled); - } - - @Test public void testOnUserActivity_doesntCrashIfNoAttentionService() { mAttentionManagerInternal = null; registerAttention(); |