summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjoenchen <joenchen@google.com>2022-09-06 18:03:57 +0000
committerjoenchen <joenchen@google.com>2022-09-14 17:43:52 +0000
commit4292810fbe008693390696ae0b9e91e9ffded231 (patch)
tree2f2ee9c6003ea8b55af1450405f7f6ce016be531
parentfad66b2c0b0dba8e6df0e023b5f199dbf0598968 (diff)
downloadnative-4292810fbe008693390696ae0b9e91e9ffded231.tar.gz
Set mBrightness when needsComposite is zero
When needsComposite is zero in persistBrightness(), the mBrightness needs to be updated to avoid unexpected skipping. Bug: 244268674 Test: adb shell cmd device_state state 0 or 1 repeatly Change-Id: Iccf925c1407e4be489da2dba521b93eee1b1c4b5
-rw-r--r--services/surfaceflinger/DisplayDevice.cpp6
1 files changed, 4 insertions, 2 deletions
diff --git a/services/surfaceflinger/DisplayDevice.cpp b/services/surfaceflinger/DisplayDevice.cpp
index 86809007b4..269a1fa17b 100644
--- a/services/surfaceflinger/DisplayDevice.cpp
+++ b/services/surfaceflinger/DisplayDevice.cpp
@@ -325,8 +325,10 @@ void DisplayDevice::stageBrightness(float brightness) {
}
void DisplayDevice::persistBrightness(bool needsComposite) {
- if (needsComposite && mStagedBrightness && mBrightness != *mStagedBrightness) {
- getCompositionDisplay()->setNextBrightness(*mStagedBrightness);
+ if (mStagedBrightness && mBrightness != *mStagedBrightness) {
+ if (needsComposite) {
+ getCompositionDisplay()->setNextBrightness(*mStagedBrightness);
+ }
mBrightness = *mStagedBrightness;
}
mStagedBrightness = std::nullopt;