commit de8d9a1d97fd68924be14da78d6d0e76407385f3
author Leon Scroggins III <scroggo@google.com> Tue Jan 23 12:05:49 2024 -0500
committer Leon Scroggins III <scroggo@google.com> Tue Jan 23 12:08:49 2024 -0500
tree 7603c4744cd008058f3fc04471e52f6672781ebf
parent a9c519873255af50508ba47d483b951b272c88e9

Remove PowerMode from RefreshRateStats

This is always called with PowerMode::OFF, so simplify the constructor.
No change in logical behavior.

Bug: 316575264
Test: mmma -j frameworks/native/services/surfaceflinger
Change-Id: Id2ed54b896d41888aa4c4526fc7000243b764d1b

services/surfaceflinger/Scheduler/Scheduler.cpp

diff --git a/services/surfaceflinger/Scheduler/Scheduler.cpp b/services/surfaceflinger/Scheduler/Scheduler.cpp
index 27ca17f..45a651a 100644
--- a/services/surfaceflinger/Scheduler/Scheduler.cpp
+++ b/services/surfaceflinger/Scheduler/Scheduler.cpp
@@ -77,8 +77,7 @@
         mFeatures(features),
         mVsyncConfiguration(factory.createVsyncConfiguration(activeRefreshRate)),
         mVsyncModulator(sp<VsyncModulator>::make(mVsyncConfiguration->getCurrentConfigs())),
-        mRefreshRateStats(std::make_unique<RefreshRateStats>(timeStats, activeRefreshRate,
-                                                             hal::PowerMode::OFF)),
+        mRefreshRateStats(std::make_unique<RefreshRateStats>(timeStats, activeRefreshRate)),
         mSchedulerCallback(callback),
         mVsyncTrackerCallback(vsyncTrackerCallback) {}