Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 915a4d6c authored by Piotr Wilczyński's avatar Piotr Wilczyński Committed by Automerger Merge Worker
Browse files

Merge "Make DPC listen to auto-brightness mode changes" into tm-qpr-dev am: fd4bed64 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20339748



Change-Id: I4a5213ac33cc98c1e18da1a339f1b8d8b399ac48
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 5a8d230d fd4bed64

core/java/android/hardware/display/DisplayManagerInternal.java

+0 −7
Original line number Diff line number Diff line
@@ -427,9 +427,6 @@ public abstract class DisplayManagerInternal { 
        // 1 (brighter). Set to Float.NaN if there's no override.

        public float screenAutoBrightnessAdjustmentOverride;



        // If true, enables automatic brightness control.

        public boolean useAutoBrightness;



        // If true, scales the brightness to a fraction of desired (as defined by

        // screenLowPowerBrightnessFactor).

        public boolean lowPowerMode;
@@ -459,7 +456,6 @@ public abstract class DisplayManagerInternal { 
            policy = POLICY_BRIGHT;

            useProximitySensor = false;

            screenBrightnessOverride = PowerManager.BRIGHTNESS_INVALID_FLOAT;

            useAutoBrightness = false;

            screenAutoBrightnessAdjustmentOverride = Float.NaN;

            screenLowPowerBrightnessFactor = 0.5f;

            blockScreenOn = false;
@@ -483,7 +479,6 @@ public abstract class DisplayManagerInternal { 
            policy = other.policy;

            useProximitySensor = other.useProximitySensor;

            screenBrightnessOverride = other.screenBrightnessOverride;

            useAutoBrightness = other.useAutoBrightness;

            screenAutoBrightnessAdjustmentOverride = other.screenAutoBrightnessAdjustmentOverride;

            screenLowPowerBrightnessFactor = other.screenLowPowerBrightnessFactor;

            blockScreenOn = other.blockScreenOn;
@@ -505,7 +500,6 @@ public abstract class DisplayManagerInternal { 
                    && useProximitySensor == other.useProximitySensor

                    && floatEquals(screenBrightnessOverride,

                            other.screenBrightnessOverride)

                    && useAutoBrightness == other.useAutoBrightness

                    && floatEquals(screenAutoBrightnessAdjustmentOverride,

                            other.screenAutoBrightnessAdjustmentOverride)

                    && screenLowPowerBrightnessFactor
@@ -531,7 +525,6 @@ public abstract class DisplayManagerInternal { 
            return "policy=" + policyToString(policy)

                    + ", useProximitySensor=" + useProximitySensor

                    + ", screenBrightnessOverride=" + screenBrightnessOverride

                    + ", useAutoBrightness=" + useAutoBrightness

                    + ", screenAutoBrightnessAdjustmentOverride="

                    + screenAutoBrightnessAdjustmentOverride

                    + ", screenLowPowerBrightnessFactor=" + screenLowPowerBrightnessFactor

services/core/java/com/android/server/display/DisplayPowerController.java

+28 −5
Original line number Diff line number Diff line
@@ -459,6 +459,8 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 
    // PowerManager.BRIGHTNESS_INVALID_FLOAT when there's no temporary adjustment set.

    private float mTemporaryAutoBrightnessAdjustment;



    private boolean mUseAutoBrightness;



    private boolean mIsRbcActive;



    // Whether there's a callback to tell listeners the display has changed scheduled to run. When
@@ -692,6 +694,7 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 


    public void onSwitchUser(@UserIdInt int newUserId) {

        handleSettingsChange(true /* userSwitch */);

        handleBrightnessModeChange();

        if (mBrightnessTracker != null) {

            mBrightnessTracker.onSwitchUser(newUserId);

        }
@@ -941,6 +944,10 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 
        mContext.getContentResolver().registerContentObserver(

                Settings.System.getUriFor(Settings.System.SCREEN_AUTO_BRIGHTNESS_ADJ),

                false /*notifyForDescendants*/, mSettingsObserver, UserHandle.USER_ALL);

        mContext.getContentResolver().registerContentObserver(

                Settings.System.getUriFor(Settings.System.SCREEN_BRIGHTNESS_MODE),

                false /*notifyForDescendants*/, mSettingsObserver, UserHandle.USER_ALL);

        handleBrightnessModeChange();

    }



    private void setUpAutoBrightness(Resources resources, Handler handler) {
@@ -1344,11 +1351,11 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 


        final boolean autoBrightnessEnabledInDoze =

                mAllowAutoBrightnessWhileDozingConfig && Display.isDozeState(state);

        final boolean autoBrightnessEnabled = mPowerRequest.useAutoBrightness

        final boolean autoBrightnessEnabled = mUseAutoBrightness

                    && (state == Display.STATE_ON || autoBrightnessEnabledInDoze)

                    && Float.isNaN(brightnessState)

                    && mAutomaticBrightnessController != null;

        final boolean autoBrightnessDisabledDueToDisplayOff = mPowerRequest.useAutoBrightness

        final boolean autoBrightnessDisabledDueToDisplayOff = mUseAutoBrightness

                    && !(state == Display.STATE_ON || autoBrightnessEnabledInDoze);

        final int autoBrightnessState = autoBrightnessEnabled

                ? AutomaticBrightnessController.AUTO_BRIGHTNESS_ENABLED
@@ -1700,7 +1707,7 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 
                || brightnessAdjustmentFlags != 0) {

            float lastBrightness = mLastBrightnessEvent.brightness;

            mTempBrightnessEvent.initialBrightness = lastBrightness;

            mTempBrightnessEvent.automaticBrightnessEnabled = mPowerRequest.useAutoBrightness;

            mTempBrightnessEvent.automaticBrightnessEnabled = mUseAutoBrightness;

            mLastBrightnessEvent.copyFrom(mTempBrightnessEvent);

            BrightnessEvent newEvent = new BrightnessEvent(mTempBrightnessEvent);
@@ -2372,6 +2379,18 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 
        sendUpdatePowerState();

    }



    private void handleBrightnessModeChange() {

        final int screenBrightnessModeSetting = Settings.System.getIntForUser(

                mContext.getContentResolver(),

                Settings.System.SCREEN_BRIGHTNESS_MODE,

                Settings.System.SCREEN_BRIGHTNESS_MODE_MANUAL, UserHandle.USER_CURRENT);

        mHandler.post(() -> {

            mUseAutoBrightness = screenBrightnessModeSetting

                    == Settings.System.SCREEN_BRIGHTNESS_MODE_AUTOMATIC;

            updatePowerState();

        });

    }



    private float getAutoBrightnessAdjustmentSetting() {

        final float adj = Settings.System.getFloatForUser(mContext.getContentResolver(),

                Settings.System.SCREEN_AUTO_BRIGHTNESS_ADJ, 0.0f, UserHandle.USER_CURRENT);
@@ -2461,7 +2480,7 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 
    private void notifyBrightnessTrackerChanged(float brightness, boolean userInitiated,

            boolean hadUserDataPoint) {

        final float brightnessInNits = convertToNits(brightness);

        if (mPowerRequest.useAutoBrightness && brightnessInNits >= 0.0f

        if (mUseAutoBrightness && brightnessInNits >= 0.0f

                && mAutomaticBrightnessController != null && mBrightnessTracker != null) {

            // We only want to track changes on devices that can actually map the display backlight

            // values into a physical brightness unit since the value provided by the API is in
@@ -3099,9 +3118,13 @@ final class DisplayPowerController implements AutomaticBrightnessController.Call 


        @Override

        public void onChange(boolean selfChange, Uri uri) {

            if (uri.equals(Settings.System.getUriFor(Settings.System.SCREEN_BRIGHTNESS_MODE))) {

                handleBrightnessModeChange();

            } else {

                handleSettingsChange(false /* userSwitch */);

            }

        }

    }



    private final class ScreenOnUnblocker implements WindowManagerPolicy.ScreenOnListener {

        @Override

services/core/java/com/android/server/power/PowerGroup.java

+5 −7
Original line number Diff line number Diff line
@@ -419,16 +419,14 @@ public class PowerGroup { 
        return mDisplayPowerRequest.policy;

    }



    boolean updateLocked(float screenBrightnessOverride, boolean autoBrightness,

            boolean useProximitySensor, boolean boostScreenBrightness, int dozeScreenState,

            float dozeScreenBrightness, boolean overrideDrawWakeLock,

            PowerSaveState powerSaverState, boolean quiescent, boolean dozeAfterScreenOff,

            boolean vrModeEnabled, boolean bootCompleted, boolean screenBrightnessBoostInProgress,

            boolean waitForNegativeProximity) {

    boolean updateLocked(float screenBrightnessOverride, boolean useProximitySensor,

            boolean boostScreenBrightness, int dozeScreenState, float dozeScreenBrightness,

            boolean overrideDrawWakeLock, PowerSaveState powerSaverState, boolean quiescent,

            boolean dozeAfterScreenOff, boolean vrModeEnabled, boolean bootCompleted,

            boolean screenBrightnessBoostInProgress, boolean waitForNegativeProximity) {

        mDisplayPowerRequest.policy = getDesiredScreenPolicyLocked(quiescent, dozeAfterScreenOff,

                vrModeEnabled, bootCompleted, screenBrightnessBoostInProgress);

        mDisplayPowerRequest.screenBrightnessOverride = screenBrightnessOverride;

        mDisplayPowerRequest.useAutoBrightness = autoBrightness;

        mDisplayPowerRequest.useProximitySensor = useProximitySensor;

        mDisplayPowerRequest.boostScreenBrightness = boostScreenBrightness;

services/core/java/com/android/server/power/PowerManagerService.java

+2 −20
Original line number Diff line number Diff line
@@ -584,10 +584,6 @@ public final class PowerManagerService extends SystemService 
    private boolean mIsFaceDown = false;

    private long mLastFlipTime = 0L;



    // The screen brightness mode.

    // One of the Settings.System.SCREEN_BRIGHTNESS_MODE_* constants.

    private int mScreenBrightnessModeSetting;



    // The screen brightness setting override from the window manager

    // to allow the current foreground activity to override the brightness.

    private float mScreenBrightnessOverrideFromWindowManager =
@@ -1481,10 +1477,6 @@ public final class PowerManagerService extends SystemService 
            mSystemProperties.set(SYSTEM_PROPERTY_RETAIL_DEMO_ENABLED, retailDemoValue);

        }



        mScreenBrightnessModeSetting = Settings.System.getIntForUser(resolver,

                Settings.System.SCREEN_BRIGHTNESS_MODE,

                Settings.System.SCREEN_BRIGHTNESS_MODE_MANUAL, UserHandle.USER_CURRENT);



        mDirty |= DIRTY_SETTINGS;

    }
@@ -3466,23 +3458,18 @@ public final class PowerManagerService extends SystemService 
                final PowerGroup powerGroup = mPowerGroups.valueAt(idx);

                final int groupId = powerGroup.getGroupId();



                // Determine appropriate screen brightness and auto-brightness adjustments.

                final boolean autoBrightness;

                // Determine appropriate screen brightness.

                final float screenBrightnessOverride;

                if (!mBootCompleted) {

                    // Keep the brightness steady during boot. This requires the

                    // bootloader brightness and the default brightness to be identical.

                    autoBrightness = false;

                    screenBrightnessOverride = mScreenBrightnessDefault;

                } else if (isValidBrightness(mScreenBrightnessOverrideFromWindowManager)) {

                    autoBrightness = false;

                    screenBrightnessOverride = mScreenBrightnessOverrideFromWindowManager;

                } else {

                    autoBrightness = (mScreenBrightnessModeSetting

                            == Settings.System.SCREEN_BRIGHTNESS_MODE_AUTOMATIC);

                    screenBrightnessOverride = PowerManager.BRIGHTNESS_INVALID_FLOAT;

                }

                boolean ready = powerGroup.updateLocked(screenBrightnessOverride, autoBrightness,

                boolean ready = powerGroup.updateLocked(screenBrightnessOverride,

                        shouldUseProximitySensorLocked(), shouldBoostScreenBrightness(),

                        mDozeScreenStateOverrideFromDreamManager,

                        mDozeScreenBrightnessOverrideFromDreamManagerFloat,
@@ -3503,7 +3490,6 @@ public final class PowerManagerService extends SystemService 
                            powerGroup.getUserActivitySummaryLocked())

                            + ", mBootCompleted=" + mBootCompleted

                            + ", screenBrightnessOverride=" + screenBrightnessOverride

                            + ", useAutoBrightness=" + autoBrightness

                            + ", mScreenBrightnessBoostInProgress="

                            + mScreenBrightnessBoostInProgress

                            + ", mIsVrModeEnabled= " + mIsVrModeEnabled
@@ -4525,7 +4511,6 @@ public final class PowerManagerService extends SystemService 
                    + mMaximumScreenOffTimeoutFromDeviceAdmin + " (enforced="

                    + isMaximumScreenOffTimeoutFromDeviceAdminEnforcedLocked() + ")");

            pw.println("  mStayOnWhilePluggedInSetting=" + mStayOnWhilePluggedInSetting);

            pw.println("  mScreenBrightnessModeSetting=" + mScreenBrightnessModeSetting);

            pw.println("  mScreenBrightnessOverrideFromWindowManager="

                    + mScreenBrightnessOverrideFromWindowManager);

            pw.println("  mUserActivityTimeoutOverrideFromWindowManager="
@@ -4903,9 +4888,6 @@ public final class PowerManagerService extends SystemService 
                            != 0));

            proto.end(stayOnWhilePluggedInToken);



            proto.write(

                    PowerServiceSettingsAndConfigurationDumpProto.SCREEN_BRIGHTNESS_MODE_SETTING,

                    mScreenBrightnessModeSetting);

            proto.write(

                    PowerServiceSettingsAndConfigurationDumpProto

                            .SCREEN_BRIGHTNESS_OVERRIDE_FROM_WINDOW_MANAGER,

services/tests/servicestests/src/com/android/server/power/PowerGroupTest.java

+0 −22
Original line number Diff line number Diff line
@@ -258,7 +258,6 @@ public class PowerGroupTest { 
                .build();



        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ false,

                /* useProximitySensor= */ false,

                /* boostScreenBrightness= */ false,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -275,7 +274,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_DIM);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(false);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(false);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(false);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -299,7 +297,6 @@ public class PowerGroupTest { 
        mPowerGroup.setWakeLockSummaryLocked(WAKE_LOCK_DOZE);



        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -316,7 +313,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_DOZE);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_ON);
@@ -339,7 +335,6 @@ public class PowerGroupTest { 
        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_DOZING);



        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -356,7 +351,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_OFF);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -378,7 +372,6 @@ public class PowerGroupTest { 
        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_AWAKE);



        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -395,7 +388,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_OFF);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -417,7 +409,6 @@ public class PowerGroupTest { 
        mPowerGroup.sleepLocked(TIMESTAMP1, UID, GO_TO_SLEEP_REASON_TIMEOUT);

        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_ASLEEP);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -434,7 +425,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_OFF);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -455,7 +445,6 @@ public class PowerGroupTest { 
                .build();

        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_AWAKE);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -472,7 +461,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_VR);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -494,7 +482,6 @@ public class PowerGroupTest { 
        mPowerGroup.sleepLocked(TIMESTAMP1, UID, GO_TO_SLEEP_REASON_TIMEOUT);

        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_ASLEEP);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -511,7 +498,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_OFF);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -534,7 +520,6 @@ public class PowerGroupTest { 
        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_DREAMING);

        mPowerGroup.setWakeLockSummaryLocked(WAKE_LOCK_SCREEN_BRIGHT);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -551,7 +536,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_BRIGHT);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -572,7 +556,6 @@ public class PowerGroupTest { 
                .build();

        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_AWAKE);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -589,7 +572,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_BRIGHT);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -611,7 +593,6 @@ public class PowerGroupTest { 
        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_AWAKE);

        mPowerGroup.setUserActivitySummaryLocked(USER_ACTIVITY_SCREEN_BRIGHT);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -628,7 +609,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_BRIGHT);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);
@@ -649,7 +629,6 @@ public class PowerGroupTest { 
                .build();

        assertThat(mPowerGroup.getWakefulnessLocked()).isEqualTo(WAKEFULNESS_AWAKE);

        mPowerGroup.updateLocked(/* screenBrightnessOverride= */ BRIGHTNESS,

                /* autoBrightness = */ true,

                /* useProximitySensor= */ true,

                /* boostScreenBrightness= */ true,

                /* dozeScreenStateOverride= */ Display.STATE_ON,
@@ -666,7 +645,6 @@ public class PowerGroupTest { 
                mPowerGroup.mDisplayPowerRequest;

        assertThat(displayPowerRequest.policy).isEqualTo(POLICY_BRIGHT);

        assertThat(displayPowerRequest.screenBrightnessOverride).isWithin(PRECISION).of(BRIGHTNESS);

        assertThat(displayPowerRequest.useAutoBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.useProximitySensor).isEqualTo(true);

        assertThat(displayPowerRequest.boostScreenBrightness).isEqualTo(true);

        assertThat(displayPowerRequest.dozeScreenState).isEqualTo(Display.STATE_UNKNOWN);