Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove precision loss on brightness slider." into rvc-dev am: b204ade30c

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

Change-Id: I4cfb0e6cb1fd6cb03cda157886c9d4b240f8b8ee
This commit is contained in:
Fiona Campbell
2020-06-23 10:51:27 +00:00
committed by Automerger Merge Worker
parent f771bf5301 b204ade30c
commit ba2cfaafe5
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
packages/SystemUI/src/com/android/systemui/settings/BrightnessController.java
View File

@@ -443,14 +443,10 @@ public class BrightnessController implements ToggleSlider.Listener {
max = mMaximumBacklight; max = mMaximumBacklight;
} }
// convertGammaToLinearFloat returns 0-1 // convertGammaToLinearFloat returns 0-1
if (BrightnessSynchronizer.brightnessFloatToInt(mContext, brightnessValue) if (BrightnessSynchronizer.floatEquals(brightnessValue,
== BrightnessSynchronizer.brightnessFloatToInt(mContext,
convertGammaToLinearFloat(mControl.getValue(), min, max))) { convertGammaToLinearFloat(mControl.getValue(), min, max))) {
// If we have more resolution on the slider than we do in the actual setting, then // If the value in the slider is equal to the value on the current brightness
// multiple slider positions will map to the same setting value. Thus, if we see a // then the slider does not need to animate, since the brightness will not change.
// setting value here that maps to the current slider position, we don't bother to
// calculate the new slider position since it may differ and look like a brightness
// change to the user even though it isn't one.
return; return;
} }
// Returns GAMMA_SPACE_MIN - GAMMA_SPACE_MAX // Returns GAMMA_SPACE_MIN - GAMMA_SPACE_MAX