diff options
| author | Marin Shalamanov <shalamanov@google.com> | 2021-01-07 16:56:24 +0100 |
|---|---|---|
| committer | Marin Shalamanov <shalamanov@google.com> | 2021-02-02 16:10:30 +0100 |
| commit | 045b70051322dcce2f9519b6113f349ef50d41fa (patch) | |
| tree | 948915ac2e8e86af256b97c4793e8bcfbef817c0 /services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp | |
| parent | ca5d8908341cdd58936ba179d5e64d98c4db9b92 (diff) | |
SF: Don't cache display modes in HWComposer
The display modes should be stored only in DisplayDevice.
Having the state also in HWComposer is unnesesary and hard to
keep in sync with SF, e.g. during hotplug processing of
displays which can change their supported modes.
Any HWC calls which need to validate their parameters need
to go through display device. This additinally makes the
code more undestandable.
Bug: 159590486
Test: presubmit
Change-Id: I40b03c09a5fd6092fca0682d602deb70db022fa5
Diffstat (limited to 'services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp')
| -rw-r--r-- | services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp b/services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp index e060df2420..a6d07d00ee 100644 --- a/services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp +++ b/services/surfaceflinger/tests/unittests/SetFrameRateTest.cpp @@ -185,7 +185,7 @@ void SetFrameRateTest::setupScheduler() { EXPECT_CALL(*vsyncTracker, nextAnticipatedVSyncTimeFrom(_)).WillRepeatedly(Return(0)); EXPECT_CALL(*vsyncTracker, currentPeriod()) - .WillRepeatedly(Return(FakeHwcDisplayInjector::DEFAULT_REFRESH_RATE)); + .WillRepeatedly(Return(FakeHwcDisplayInjector::DEFAULT_VSYNC_PERIOD)); EXPECT_CALL(*vsyncTracker, nextAnticipatedVSyncTimeFrom(_)).WillRepeatedly(Return(0)); mFlinger.setupScheduler(std::move(vsyncController), std::move(vsyncTracker), std::move(eventThread), std::move(sfEventThread)); |