commit fdc049c5f4e104b8005d80cb33088183963f8517
author Ady Abraham <adyabr@google.com> Fri Feb 17 14:52:05 2023 +0000
committer Ady Abraham <adyabr@google.com> Fri Feb 17 16:27:04 2023 +0000
tree 65da2686b15f7803487719894a176e9e0fb6aa8e
parent 0311fdd6d2d5eca00e84bd95b1d1a296171ca038

Revert "Revert "SF: pass a render rate to VsyncTracker instead of a divisor""

This reverts commit 3d21d3afa1056efacf702b74449f9e2e7b5701f1.

Reason for revert: https://b.corp.google.com/issues/269561042#comment14

Bug: 269561042
Change-Id: Iae8d0e62be2eafb4278b64ffcb1524ece211a00c

services/surfaceflinger/Scheduler/VSyncPredictor.h

diff --git a/services/surfaceflinger/Scheduler/VSyncPredictor.h b/services/surfaceflinger/Scheduler/VSyncPredictor.h
index d0e3098..cd5d9ef 100644
--- a/services/surfaceflinger/Scheduler/VSyncPredictor.h
+++ b/services/surfaceflinger/Scheduler/VSyncPredictor.h
@@ -67,7 +67,7 @@
 
     bool isVSyncInPhase(nsecs_t timePoint, Fps frameRate) const final EXCLUDES(mMutex);
 
-    void setDivisor(unsigned divisor) final EXCLUDES(mMutex);
+    void setRenderRate(Fps) final EXCLUDES(mMutex);
 
     void dump(std::string& result) const final EXCLUDES(mMutex);
 
@@ -106,7 +106,7 @@
     size_t mLastTimestampIndex GUARDED_BY(mMutex) = 0;
     std::vector<nsecs_t> mTimestamps GUARDED_BY(mMutex);
 
-    unsigned mDivisor GUARDED_BY(mMutex) = 1;
+    std::optional<Fps> mRenderRate GUARDED_BY(mMutex);
 
     mutable std::optional<VsyncSequence> mLastVsyncSequence GUARDED_BY(mMutex);
 };