commit e0bc94b8d9652506aa49359370fdc24f566d05d7
author Dominik Laskowski <domlaskowski@google.com> Sun Aug 04 15:39:25 2024 -0400
committer Dominik Laskowski <domlaskowski@google.com> Wed Aug 14 13:09:00 2024 -0400
tree 6b67befac8198d6dbe96e7e9ffdf19e3a466651a
parent f5048752aebaa179b992e01b7900b1417240561e

SF: Remove Scheduler::resetPhaseConfiguration

Merge the only call into setPacesetterDisplay.

This does not change behavior.

Bug: 255635821
Flag: EXEMPT refactor
Test: presubmit
Change-Id: I2719e35231093273203df24272d61e998d7f0b20

services/surfaceflinger/Scheduler/Scheduler.cpp

diff --git a/services/surfaceflinger/Scheduler/Scheduler.cpp b/services/surfaceflinger/Scheduler/Scheduler.cpp
index 6b31135..a3884bc 100644
--- a/services/surfaceflinger/Scheduler/Scheduler.cpp
+++ b/services/surfaceflinger/Scheduler/Scheduler.cpp
@@ -122,6 +122,12 @@
 
     demotePacesetterDisplay(kPromotionParams);
     promotePacesetterDisplay(pacesetterId, kPromotionParams);
+
+    // Cancel the pending refresh rate change, if any, before updating the phase configuration.
+    mVsyncModulator->cancelRefreshRateChange();
+
+    mVsyncConfiguration->reset();
+    updatePhaseConfiguration(pacesetterSelectorPtr()->getActiveMode().fps);
 }
 
 void Scheduler::registerDisplay(PhysicalDisplayId displayId, RefreshRateSelectorPtr selectorPtr,
@@ -488,14 +494,6 @@
                    refreshRate.getPeriod());
 }
 
-void Scheduler::resetPhaseConfiguration(Fps refreshRate) {
-    // Cancel the pending refresh rate change, if any, before updating the phase configuration.
-    mVsyncModulator->cancelRefreshRateChange();
-
-    mVsyncConfiguration->reset();
-    updatePhaseConfiguration(refreshRate);
-}
-
 void Scheduler::setActiveDisplayPowerModeForRefreshRateStats(hal::PowerMode powerMode) {
     mRefreshRateStats->setPowerMode(powerMode);
 }