From 2324d8578aeddf2cab4cb14e4cf9946f7cf3b107 Mon Sep 17 00:00:00 2001 From: Adrian Roos Date: Thu, 27 Apr 2017 15:50:14 -0700 Subject: [PATCH] AOD: Don't use DOZE screen state on AOD2 Fixes an issue where AOD2 had low color gamut. Instead, we now use the high-power screen state there. Bug: 36781008 Test: Receive notification on AOD2, observe that color is not distorted Change-Id: I370a74c0f1d5cfd49194b53b9e20bf1f0dc5d4d0 --- .../SystemUI/src/com/android/systemui/doze/DozeMachine.java | 2 +- .../SystemUI/src/com/android/systemui/doze/DozeService.java | 2 ++ .../tests/src/com/android/systemui/doze/DozeMachineTest.java | 2 +- 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/doze/DozeMachine.java b/packages/SystemUI/src/com/android/systemui/doze/DozeMachine.java index 90eb1fc8aa3e3..34d621f7b624d 100644 --- a/packages/SystemUI/src/com/android/systemui/doze/DozeMachine.java +++ b/packages/SystemUI/src/com/android/systemui/doze/DozeMachine.java @@ -91,7 +91,7 @@ public class DozeMachine { case DOZE_AOD_PAUSED: return Display.STATE_OFF; case DOZE_PULSING: - return Display.STATE_DOZE; + return Display.STATE_ON; case DOZE_AOD: return Display.STATE_DOZE_SUSPEND; default: diff --git a/packages/SystemUI/src/com/android/systemui/doze/DozeService.java b/packages/SystemUI/src/com/android/systemui/doze/DozeService.java index e55a597223853..8181c4e8428d1 100644 --- a/packages/SystemUI/src/com/android/systemui/doze/DozeService.java +++ b/packages/SystemUI/src/com/android/systemui/doze/DozeService.java @@ -60,6 +60,8 @@ public class DozeService extends DreamService implements DozeMachine.Service { super.onDreamingStarted(); mDozeMachine.requestState(DozeMachine.State.INITIALIZED); startDozing(); + setDozeScreenBrightness(getResources().getInteger( + com.android.internal.R.integer.config_screenBrightnessDoze)); } @Override diff --git a/packages/SystemUI/tests/src/com/android/systemui/doze/DozeMachineTest.java b/packages/SystemUI/tests/src/com/android/systemui/doze/DozeMachineTest.java index 5771b2819f37e..24d8b924ee85f 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/doze/DozeMachineTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/doze/DozeMachineTest.java @@ -238,7 +238,7 @@ public class DozeMachineTest { mMachine.requestState(DOZE_REQUEST_PULSE); mMachine.requestState(DOZE_PULSING); - assertEquals(Display.STATE_DOZE, mServiceFake.screenState); + assertEquals(Display.STATE_ON, mServiceFake.screenState); } @Test