commit 76eb78fe5769849e09f01290614beed496d99155
author Rachel Lee <rnlee@google.com> Wed Nov 27 15:44:35 2024 -0800
committer Rachel Lee <rnlee@google.com> Tue Dec 03 16:20:35 2024 -0800
tree ebed96d496c7d914cedc2b80c1570960b3a62fbb
parent ee81a84bf46e05021c91c17b98c9b047c5e5e2c1

Add GTE compatibility logic for NoVote

setFrameRate with GTE compatibility and 0 `frameRate` will be counted as NoVote.

Test: atest libsurfaceflinger_unittest
Test: atest LayerHistoryIntegrationTest
Bug: 380949716
Flag: com.android.graphics.surfaceflinger.flags.arr_setframerate_gte_enum
Change-Id: I5ffcd4a09b1661c516c11d2f04b80303af215c3a

services/surfaceflinger/Scheduler/LayerHistory.cpp

diff --git a/services/surfaceflinger/Scheduler/LayerHistory.cpp b/services/surfaceflinger/Scheduler/LayerHistory.cpp
index 171342d..630beb0 100644
--- a/services/surfaceflinger/Scheduler/LayerHistory.cpp
+++ b/services/surfaceflinger/Scheduler/LayerHistory.cpp
@@ -280,6 +280,9 @@
                     case Layer::FrameRateCompatibility::Exact:
                         return LayerVoteType::ExplicitExact;
                     case Layer::FrameRateCompatibility::Gte:
+                        if (frameRate.isNoVote()) {
+                            return LayerVoteType::NoVote;
+                        }
                         if (isVrrDevice) {
                             return LayerVoteType::ExplicitGte;
                         } else {