Move VsyncConfiguration from SF to Scheduler
This is part of the long term goal, tracked in b/241285191, of moving scheduling logic/state from SF to Scheduler. It also paves the way for a cleaner version of Ib927935de6ba2b7b8d5037b42eb635ae92019634. Bring RefreshRateStats along to Scheduler, since it is updated along with work that belongs in Scheduler. Mostly this is intended to not change behavior, but there are some exceptions: - the ordering of dumpsys SurfaceFlinger has changed, as some components are now internal to Scheduler, so they are dumped along with it. - two extra calls to update the RefreshRateStats along with updatePhaseConfiguration. The missing calls appear to be an oversight, and it's more natural to include the update in its new location. Move construction of VsyncModulator into Scheduler, since it is constructed from the VsyncConfiguration, now constructed in Scheduler. Add TestableScheduler.cpp, to prevent adding a circular dependency, now that the class's constructor references TestableSurfaceFlinger. Add Dumper::out for simplicity in updating dump calls. Bug: 241285191 Bug: 255601557 Bug: 256196556 Bug: 259132483 Test: libscheduler_test Test: libsurfaceflinger_unittest Change-Id: I01c93cfc3d2342cfbaaf8fe47dc2cd57d0e1fc43
Loading
Please register or sign in to comment