diff options
author | Kweku Adams <kwekua@google.com> | 2020-05-14 08:02:45 -0700 |
---|---|---|
committer | Kweku Adams <kwekua@google.com> | 2020-05-14 18:03:58 +0000 |
commit | 107be82a9c723fb0db1460d54540a8eb8cef1686 (patch) | |
tree | 50e0ee333e727faf0b7e9d9f06aba4b240fe79c9 | |
parent | 0a1a911a5e2162f8d083508da0fd029cc40880c1 (diff) |
Reduce widget exemption status.
Apps with active widgets should be fine operating in the ACTIVE bucket
instead of being fully exempted from app standby. They were previously
put in the EXEMPTED bucket because we previously didn't have a concept
of exemption levels.
Bug: 156236665
Test: atest FrameworksServicesTests:AppIdleHistoryTests
Test: atest FrameworksServicesTests:AppStandbyControllerTests
Change-Id: Iea30f00ca8199f487bcc3281e4ae3f98b41d369f
-rw-r--r-- | apex/jobscheduler/service/java/com/android/server/usage/AppStandbyController.java | 3 | ||||
-rw-r--r-- | services/tests/servicestests/src/com/android/server/usage/AppStandbyControllerTests.java | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/apex/jobscheduler/service/java/com/android/server/usage/AppStandbyController.java b/apex/jobscheduler/service/java/com/android/server/usage/AppStandbyController.java index 2834ab14f28d..94e5d0b2591b 100644 --- a/apex/jobscheduler/service/java/com/android/server/usage/AppStandbyController.java +++ b/apex/jobscheduler/service/java/com/android/server/usage/AppStandbyController.java @@ -1098,8 +1098,7 @@ public class AppStandbyController implements AppStandbyInternal { if (mAppWidgetManager != null && mInjector.isBoundWidgetPackage(mAppWidgetManager, packageName, userId)) { - // TODO: consider lowering to ACTIVE - return STANDBY_BUCKET_EXEMPTED; + return STANDBY_BUCKET_ACTIVE; } if (isDeviceProvisioningPackage(packageName)) { diff --git a/services/tests/servicestests/src/com/android/server/usage/AppStandbyControllerTests.java b/services/tests/servicestests/src/com/android/server/usage/AppStandbyControllerTests.java index e4102205ddbb..2d45f9ea40c7 100644 --- a/services/tests/servicestests/src/com/android/server/usage/AppStandbyControllerTests.java +++ b/services/tests/servicestests/src/com/android/server/usage/AppStandbyControllerTests.java @@ -387,7 +387,7 @@ public class AppStandbyControllerTests { @Test public void testBoundWidgetPackageExempt() throws Exception { assumeTrue(mInjector.getContext().getSystemService(AppWidgetManager.class) != null); - assertEquals(STANDBY_BUCKET_EXEMPTED, + assertEquals(STANDBY_BUCKET_ACTIVE, mController.getAppStandbyBucket(PACKAGE_EXEMPTED_1, USER_ID, mInjector.mElapsedRealtime, false)); } |