commit b26872ada5781414b4efb2810bd7baff94c744e9
author Ady Abraham <adyabr@google.com> Wed Mar 29 17:51:20 2023 -0700
committer Ady Abraham <adyabr@google.com> Thu Mar 30 23:00:02 2023 +0000
tree e84a826afa9dccb3195c4ce9822f75e5be549e9a
parent db913040c5b7f0cf8b57c8ec9e2a45a6a68e64cc

SF: use render rates when checking if allowed by policy

This fixed a bug where SF might pick a render rate outside
the policy.

Bug: 276355056
Test: SF unit tests
Change-Id: I9d54b4fd62859d99b2c50cbc43320eb84c2b6cdf

services/surfaceflinger/Scheduler/RefreshRateSelector.cpp

diff --git a/services/surfaceflinger/Scheduler/RefreshRateSelector.cpp b/services/surfaceflinger/Scheduler/RefreshRateSelector.cpp
index eec7c08..f136e9f 100644
--- a/services/surfaceflinger/Scheduler/RefreshRateSelector.cpp
+++ b/services/surfaceflinger/Scheduler/RefreshRateSelector.cpp
@@ -564,7 +564,7 @@
                 continue;
             }
 
-            const bool inPrimaryRange = policy->primaryRanges.physical.includes(modePtr->getFps());
+            const bool inPrimaryRange = policy->primaryRanges.render.includes(fps);
             if ((primaryRangeIsSingleRate || !inPrimaryRange) &&
                 !(layer.focused &&
                   (layer.vote == LayerVoteType::ExplicitDefault ||