Skip to content
Snippets Groups Projects
Commit 9d2f2865 authored by Fiona Campbell's avatar Fiona Campbell Committed by Android (Google) Code Review
Browse files

Merge "Set EvenDimmer to true by default" into main

parents ec44865a 3868a685
No related branches found
No related tags found
No related merge requests found
......@@ -285,7 +285,8 @@ public class BrightnessClamperController {
List<BrightnessStateModifier> modifiers = new ArrayList<>();
modifiers.add(new DisplayDimModifier(context));
modifiers.add(new BrightnessLowPowerModeModifier());
if (flags.isEvenDimmerEnabled() && displayDeviceConfig != null) {
if (flags.isEvenDimmerEnabled() && displayDeviceConfig != null
&& displayDeviceConfig.isEvenDimmerAvailable()) {
modifiers.add(new BrightnessLowLuxModifier(handler, listener, context,
displayDeviceConfig));
}
......
......@@ -87,9 +87,7 @@ public class BrightnessLowLuxModifier extends BrightnessModifier {
mContentResolver, Settings.Secure.EVEN_DIMMER_MIN_NITS,
/* def= */ MIN_NITS_DEFAULT, userId);
boolean isActive = Settings.Secure.getFloatForUser(mContentResolver,
Settings.Secure.EVEN_DIMMER_ACTIVATED,
/* def= */ 0, userId) == 1.0f && mAutoBrightnessEnabled;
boolean isActive = isSettingEnabled() && mAutoBrightnessEnabled;
float luxBasedNitsLowerBound = mDisplayDeviceConfig.getMinNitsFromLux(mAmbientLux);
......@@ -202,6 +200,17 @@ public class BrightnessLowLuxModifier extends BrightnessModifier {
pw.println(" mMinNitsAllowed=" + mMinNitsAllowed);
}
/**
* Defaults to true, on devices where setting is unset.
*
* @return if setting indicates feature is enabled
*/
private boolean isSettingEnabled() {
return Settings.Secure.getFloatForUser(mContentResolver,
Settings.Secure.EVEN_DIMMER_ACTIVATED,
/* def= */ 1.0f, UserHandle.USER_CURRENT) == 1.0f;
}
private float getBrightnessFromNits(float nits) {
return mDisplayDeviceConfig.getBrightnessFromBacklight(
mDisplayDeviceConfig.getBacklightFromNits(nits));
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment