| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright 2019 The Android Open Source Project | 
|  | 3 | * | 
|  | 4 | * Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 5 | * you may not use this file except in compliance with the License. | 
|  | 6 | * You may obtain a copy of the License at | 
|  | 7 | * | 
|  | 8 | *      http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 9 | * | 
|  | 10 | * Unless required by applicable law or agreed to in writing, software | 
|  | 11 | * distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 13 | * See the License for the specific language governing permissions and | 
|  | 14 | * limitations under the License. | 
|  | 15 | */ | 
|  | 16 |  | 
| Marin Shalamanov | bed7fd3 | 2020-12-21 20:02:20 +0100 | [diff] [blame] | 17 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 18 | #pragma clang diagnostic push | 
|  | 19 | #pragma clang diagnostic ignored "-Wextra" | 
|  | 20 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 21 | #undef LOG_TAG | 
|  | 22 | #define LOG_TAG "SchedulerUnittests" | 
|  | 23 |  | 
|  | 24 | #include <gmock/gmock.h> | 
|  | 25 | #include <log/log.h> | 
|  | 26 | #include <thread> | 
|  | 27 |  | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 28 | #include <ui/Size.h> | 
|  | 29 |  | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 30 | #include "../../Scheduler/RefreshRateConfigs.h" | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 31 | #include "DisplayHardware/HWC2.h" | 
|  | 32 | #include "Scheduler/RefreshRateConfigs.h" | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 33 |  | 
|  | 34 | using namespace std::chrono_literals; | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 35 |  | 
|  | 36 | namespace android { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 37 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 38 | namespace scheduler { | 
|  | 39 |  | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 40 | namespace hal = android::hardware::graphics::composer::hal; | 
|  | 41 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 42 | using RefreshRate = RefreshRateConfigs::RefreshRate; | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 43 | using LayerVoteType = RefreshRateConfigs::LayerVoteType; | 
|  | 44 | using LayerRequirement = RefreshRateConfigs::LayerRequirement; | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 45 |  | 
|  | 46 | class RefreshRateConfigsTest : public testing::Test { | 
|  | 47 | protected: | 
| Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 48 | using GetBestRefreshRateInvocation = RefreshRateConfigs::GetBestRefreshRateInvocation; | 
|  | 49 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 50 | RefreshRateConfigsTest(); | 
|  | 51 | ~RefreshRateConfigsTest(); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 52 |  | 
| Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 53 | RefreshRate createRefreshRate(DisplayModePtr displayMode) { | 
| Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 54 | return {displayMode, RefreshRate::ConstructorTag(0)}; | 
| Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 55 | } | 
|  | 56 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 57 | Fps findClosestKnownFrameRate(const RefreshRateConfigs& refreshRateConfigs, Fps frameRate) { | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 58 | return refreshRateConfigs.findClosestKnownFrameRate(frameRate); | 
|  | 59 | } | 
|  | 60 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 61 | std::vector<Fps> getKnownFrameRate(const RefreshRateConfigs& refreshRateConfigs) { | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 62 | return refreshRateConfigs.mKnownFrameRates; | 
|  | 63 | } | 
|  | 64 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 65 | RefreshRate getMinRefreshRateByPolicy(const RefreshRateConfigs& refreshRateConfigs) { | 
|  | 66 | std::lock_guard lock(refreshRateConfigs.mLock); | 
|  | 67 | return refreshRateConfigs.getMinRefreshRateByPolicyLocked(); | 
|  | 68 | } | 
|  | 69 |  | 
|  | 70 | RefreshRate getMinSupportedRefreshRate(const RefreshRateConfigs& refreshRateConfigs) { | 
|  | 71 | std::lock_guard lock(refreshRateConfigs.mLock); | 
|  | 72 | return *refreshRateConfigs.mMinSupportedRefreshRate; | 
|  | 73 | } | 
|  | 74 |  | 
|  | 75 | RefreshRate getMaxSupportedRefreshRate(const RefreshRateConfigs& refreshRateConfigs) { | 
|  | 76 | std::lock_guard lock(refreshRateConfigs.mLock); | 
|  | 77 | return *refreshRateConfigs.mMaxSupportedRefreshRate; | 
|  | 78 | } | 
|  | 79 |  | 
| Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 80 | void setLastBestRefreshRateInvocation(RefreshRateConfigs& refreshRateConfigs, | 
|  | 81 | const GetBestRefreshRateInvocation& invocation) { | 
|  | 82 | std::lock_guard lock(refreshRateConfigs.mLock); | 
|  | 83 | refreshRateConfigs.lastBestRefreshRateInvocation.emplace( | 
|  | 84 | GetBestRefreshRateInvocation(invocation)); | 
|  | 85 | } | 
|  | 86 |  | 
|  | 87 | std::optional<GetBestRefreshRateInvocation> getLastBestRefreshRateInvocation( | 
|  | 88 | const RefreshRateConfigs& refreshRateConfigs) { | 
|  | 89 | std::lock_guard lock(refreshRateConfigs.mLock); | 
|  | 90 | return refreshRateConfigs.lastBestRefreshRateInvocation; | 
|  | 91 | } | 
|  | 92 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 93 | // Test config IDs | 
| Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 94 | static inline const DisplayModeId HWC_CONFIG_ID_60 = DisplayModeId(0); | 
|  | 95 | static inline const DisplayModeId HWC_CONFIG_ID_90 = DisplayModeId(1); | 
|  | 96 | static inline const DisplayModeId HWC_CONFIG_ID_72 = DisplayModeId(2); | 
|  | 97 | static inline const DisplayModeId HWC_CONFIG_ID_120 = DisplayModeId(3); | 
|  | 98 | static inline const DisplayModeId HWC_CONFIG_ID_30 = DisplayModeId(4); | 
|  | 99 | static inline const DisplayModeId HWC_CONFIG_ID_25 = DisplayModeId(5); | 
|  | 100 | static inline const DisplayModeId HWC_CONFIG_ID_50 = DisplayModeId(6); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 101 |  | 
|  | 102 | // Test configs | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 103 | DisplayModePtr mConfig60 = createDisplayMode(HWC_CONFIG_ID_60, 0, Fps(60.0f).getPeriodNsecs()); | 
|  | 104 | DisplayModePtr mConfig90 = createDisplayMode(HWC_CONFIG_ID_90, 0, Fps(90.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 105 | DisplayModePtr mConfig90DifferentGroup = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 106 | createDisplayMode(HWC_CONFIG_ID_90, 1, Fps(90.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 107 | DisplayModePtr mConfig90DifferentResolution = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 108 | createDisplayMode(HWC_CONFIG_ID_90, 0, Fps(90.0f).getPeriodNsecs(), ui::Size(111, 222)); | 
|  | 109 | DisplayModePtr mConfig72 = createDisplayMode(HWC_CONFIG_ID_72, 0, Fps(72.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 110 | DisplayModePtr mConfig72DifferentGroup = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 111 | createDisplayMode(HWC_CONFIG_ID_72, 1, Fps(72.0f).getPeriodNsecs()); | 
|  | 112 | DisplayModePtr mConfig120 = | 
|  | 113 | createDisplayMode(HWC_CONFIG_ID_120, 0, Fps(120.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 114 | DisplayModePtr mConfig120DifferentGroup = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 115 | createDisplayMode(HWC_CONFIG_ID_120, 1, Fps(120.0f).getPeriodNsecs()); | 
|  | 116 | DisplayModePtr mConfig30 = createDisplayMode(HWC_CONFIG_ID_30, 0, Fps(30.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 117 | DisplayModePtr mConfig30DifferentGroup = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 118 | createDisplayMode(HWC_CONFIG_ID_30, 1, Fps(30.0f).getPeriodNsecs()); | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 119 | DisplayModePtr mConfig25DifferentGroup = | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 120 | createDisplayMode(HWC_CONFIG_ID_25, 1, Fps(25.0f).getPeriodNsecs()); | 
|  | 121 | DisplayModePtr mConfig50 = createDisplayMode(HWC_CONFIG_ID_50, 0, Fps(50.0f).getPeriodNsecs()); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 122 |  | 
|  | 123 | // Test device configurations | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 124 | // The positions of the configs in the arrays below MUST match their IDs. For example, | 
|  | 125 | // the first config should always be 60Hz, the second 90Hz etc. | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 126 | DisplayModes m60OnlyConfigDevice = {mConfig60}; | 
|  | 127 | DisplayModes m60_90Device = {mConfig60, mConfig90}; | 
|  | 128 | DisplayModes m60_90DeviceWithDifferentGroups = {mConfig60, mConfig90DifferentGroup}; | 
|  | 129 | DisplayModes m60_90DeviceWithDifferentResolutions = {mConfig60, mConfig90DifferentResolution}; | 
|  | 130 | DisplayModes m60_72_90Device = {mConfig60, mConfig90, mConfig72}; | 
|  | 131 | DisplayModes m60_90_72_120Device = {mConfig60, mConfig90, mConfig72, mConfig120}; | 
|  | 132 | DisplayModes m30_60_72_90_120Device = {mConfig60, mConfig90, mConfig72, mConfig120, mConfig30}; | 
|  | 133 | DisplayModes m30_60Device = {mConfig60, mConfig90DifferentGroup, mConfig72DifferentGroup, | 
|  | 134 | mConfig120DifferentGroup, mConfig30}; | 
|  | 135 | DisplayModes m30_60_72_90Device = {mConfig60, mConfig90, mConfig72, mConfig120DifferentGroup, | 
|  | 136 | mConfig30}; | 
|  | 137 | DisplayModes m30_60_90Device = {mConfig60, mConfig90, mConfig72DifferentGroup, | 
|  | 138 | mConfig120DifferentGroup, mConfig30}; | 
|  | 139 | DisplayModes m25_30_50_60Device = {mConfig60, | 
|  | 140 | mConfig90, | 
|  | 141 | mConfig72DifferentGroup, | 
|  | 142 | mConfig120DifferentGroup, | 
|  | 143 | mConfig30DifferentGroup, | 
|  | 144 | mConfig25DifferentGroup, | 
|  | 145 | mConfig50}; | 
| Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 146 | DisplayModes m60_120Device = {mConfig60, mConfig120}; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 147 |  | 
|  | 148 | // Expected RefreshRate objects | 
| Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 149 | RefreshRate mExpected60Config = {mConfig60, RefreshRate::ConstructorTag(0)}; | 
|  | 150 | RefreshRate mExpectedAlmost60Config = {createDisplayMode(HWC_CONFIG_ID_60, 0, 16666665), | 
|  | 151 | RefreshRate::ConstructorTag(0)}; | 
|  | 152 | RefreshRate mExpected90Config = {mConfig90, RefreshRate::ConstructorTag(0)}; | 
|  | 153 | RefreshRate mExpected90DifferentGroupConfig = {mConfig90DifferentGroup, | 
|  | 154 | RefreshRate::ConstructorTag(0)}; | 
|  | 155 | RefreshRate mExpected90DifferentResolutionConfig = {mConfig90DifferentResolution, | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 156 | RefreshRate::ConstructorTag(0)}; | 
| Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 157 | RefreshRate mExpected72Config = {mConfig72, RefreshRate::ConstructorTag(0)}; | 
|  | 158 | RefreshRate mExpected30Config = {mConfig30, RefreshRate::ConstructorTag(0)}; | 
|  | 159 | RefreshRate mExpected120Config = {mConfig120, RefreshRate::ConstructorTag(0)}; | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 160 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 161 | private: | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 162 | DisplayModePtr createDisplayMode(DisplayModeId modeId, int32_t group, int64_t vsyncPeriod, | 
|  | 163 | ui::Size resolution = ui::Size()); | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 164 | }; | 
|  | 165 |  | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 166 | using Builder = DisplayMode::Builder; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 167 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 168 | RefreshRateConfigsTest::RefreshRateConfigsTest() { | 
|  | 169 | const ::testing::TestInfo* const test_info = | 
|  | 170 | ::testing::UnitTest::GetInstance()->current_test_info(); | 
|  | 171 | ALOGD("**** Setting up for %s.%s\n", test_info->test_case_name(), test_info->name()); | 
|  | 172 | } | 
|  | 173 |  | 
|  | 174 | RefreshRateConfigsTest::~RefreshRateConfigsTest() { | 
|  | 175 | const ::testing::TestInfo* const test_info = | 
|  | 176 | ::testing::UnitTest::GetInstance()->current_test_info(); | 
|  | 177 | ALOGD("**** Tearing down after %s.%s\n", test_info->test_case_name(), test_info->name()); | 
|  | 178 | } | 
|  | 179 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 180 | DisplayModePtr RefreshRateConfigsTest::createDisplayMode(DisplayModeId modeId, int32_t group, | 
|  | 181 | int64_t vsyncPeriod, ui::Size resolution) { | 
|  | 182 | return DisplayMode::Builder(hal::HWConfigId(modeId.value())) | 
|  | 183 | .setId(modeId) | 
| Ady Abraham | f3e81d2 | 2021-06-23 17:43:41 -0700 | [diff] [blame] | 184 | .setPhysicalDisplayId(PhysicalDisplayId(0)) | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 185 | .setVsyncPeriod(int32_t(vsyncPeriod)) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 186 | .setGroup(group) | 
| Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 187 | .setHeight(resolution.height) | 
|  | 188 | .setWidth(resolution.width) | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 189 | .build(); | 
|  | 190 | } | 
|  | 191 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 192 | namespace { | 
|  | 193 | /* ------------------------------------------------------------------------ | 
|  | 194 | * Test cases | 
|  | 195 | */ | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 196 | TEST_F(RefreshRateConfigsTest, oneDeviceConfig_SwitchingSupported) { | 
| Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 197 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 198 | std::make_unique<RefreshRateConfigs>(m60OnlyConfigDevice, | 
|  | 199 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 200 | } | 
|  | 201 |  | 
| Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 202 | TEST_F(RefreshRateConfigsTest, invalidPolicy) { | 
| Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 203 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 204 | std::make_unique<RefreshRateConfigs>(m60OnlyConfigDevice, | 
|  | 205 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 206 | ASSERT_LT(refreshRateConfigs->setDisplayManagerPolicy({DisplayModeId(10), {Fps(60), Fps(60)}}), | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 207 | 0); | 
|  | 208 | ASSERT_LT(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(20), Fps(40)}}), | 
|  | 209 | 0); | 
| Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 210 | } | 
|  | 211 |  | 
| Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 212 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap) { | 
| Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 213 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 214 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 215 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 216 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 217 | const auto& minRate = getMinSupportedRefreshRate(*refreshRateConfigs); | 
|  | 218 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 219 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 220 | ASSERT_EQ(mExpected60Config, minRate); | 
|  | 221 | ASSERT_EQ(mExpected90Config, performanceRate); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 222 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 223 | const auto& minRateByPolicy = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
| Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 224 | const auto& performanceRateByPolicy = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 225 | ASSERT_EQ(minRateByPolicy, minRate); | 
|  | 226 | ASSERT_EQ(performanceRateByPolicy, performanceRate); | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 227 | } | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 228 |  | 
|  | 229 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap_differentGroups) { | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 230 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 231 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 232 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 233 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 234 | const auto& minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
|  | 235 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); | 
|  | 236 | const auto& minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
| Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 237 | const auto& performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 238 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 239 | ASSERT_EQ(mExpected60Config, minRate); | 
|  | 240 | ASSERT_EQ(mExpected60Config, minRate60); | 
|  | 241 | ASSERT_EQ(mExpected60Config, performanceRate60); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 242 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 243 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {Fps(60), Fps(90)}}), | 
|  | 244 | 0); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 245 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 246 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 247 | const auto& minRate90 = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
| Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 248 | const auto& performanceRate90 = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 249 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 250 | ASSERT_EQ(mExpected90DifferentGroupConfig, performanceRate); | 
|  | 251 | ASSERT_EQ(mExpected90DifferentGroupConfig, minRate90); | 
|  | 252 | ASSERT_EQ(mExpected90DifferentGroupConfig, performanceRate90); | 
|  | 253 | } | 
|  | 254 |  | 
|  | 255 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap_differentResolutions) { | 
|  | 256 | auto refreshRateConfigs = | 
|  | 257 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentResolutions, | 
|  | 258 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 259 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 260 | const auto& minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
|  | 261 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); | 
|  | 262 | const auto& minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 263 | const auto& performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
|  | 264 |  | 
|  | 265 | ASSERT_EQ(mExpected60Config, minRate); | 
|  | 266 | ASSERT_EQ(mExpected60Config, minRate60); | 
|  | 267 | ASSERT_EQ(mExpected60Config, performanceRate60); | 
|  | 268 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 269 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {Fps(60), Fps(90)}}), | 
|  | 270 | 0); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 271 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 272 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 273 | const auto& minRate90 = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 274 | const auto& performanceRate90 = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
|  | 275 |  | 
|  | 276 | ASSERT_EQ(mExpected90DifferentResolutionConfig, performanceRate); | 
|  | 277 | ASSERT_EQ(mExpected90DifferentResolutionConfig, minRate90); | 
|  | 278 | ASSERT_EQ(mExpected90DifferentResolutionConfig, performanceRate90); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 279 | } | 
|  | 280 |  | 
|  | 281 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_policyChange) { | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 282 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 283 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 284 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ana Krulec | 3f6a206 | 2020-01-23 15:48:01 -0800 | [diff] [blame] | 285 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 286 | auto minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
|  | 287 | auto performanceRate = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 288 |  | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 289 | ASSERT_EQ(mExpected60Config, minRate); | 
|  | 290 | ASSERT_EQ(mExpected90Config, performanceRate); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 291 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 292 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(60)}}), | 
|  | 293 | 0); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 294 |  | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 295 | auto minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); | 
|  | 296 | auto performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 297 | ASSERT_EQ(mExpected60Config, minRate60); | 
|  | 298 | ASSERT_EQ(mExpected60Config, performanceRate60); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 299 | } | 
|  | 300 |  | 
|  | 301 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_getCurrentRefreshRate) { | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 302 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 303 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 304 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 305 | { | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 306 | auto current = refreshRateConfigs->getCurrentRefreshRate(); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 307 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_60); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 308 | } | 
|  | 309 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 310 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 311 | { | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 312 | auto current = refreshRateConfigs->getCurrentRefreshRate(); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 313 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_90); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 314 | } | 
|  | 315 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 316 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {Fps(90), Fps(90)}}), | 
|  | 317 | 0); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 318 | { | 
| Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 319 | auto current = refreshRateConfigs->getCurrentRefreshRate(); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 320 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_90); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 321 | } | 
|  | 322 | } | 
|  | 323 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 324 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_noLayers) { | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 325 | auto refreshRateConfigs = | 
|  | 326 | std::make_unique<RefreshRateConfigs>(m60_72_90Device, /*currentConfigId=*/ | 
|  | 327 | HWC_CONFIG_ID_72); | 
| Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 328 |  | 
| Steven Thomas | debafed | 2020-05-18 17:30:35 -0700 | [diff] [blame] | 329 | // If there are no layers we select the default frame rate, which is the max of the primary | 
|  | 330 | // range. | 
| Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 331 | auto layers = std::vector<LayerRequirement>{}; | 
| Steven Thomas | debafed | 2020-05-18 17:30:35 -0700 | [diff] [blame] | 332 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 333 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 334 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 335 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(60)}}), | 
|  | 336 | 0); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 337 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 338 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 339 | } | 
|  | 340 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 341 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_60_90) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 342 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 343 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 344 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 345 |  | 
|  | 346 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 347 | auto& lr = layers[0]; | 
|  | 348 |  | 
|  | 349 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 350 | lr.name = "Min"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 351 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 352 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 353 |  | 
|  | 354 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 355 | lr.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 356 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 357 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 358 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 359 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 360 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 361 | lr.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 362 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 363 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 364 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 365 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 366 | lr.name = "60Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 367 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 368 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 369 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 370 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 371 | lr.name = "45Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 372 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 373 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 374 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 375 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 376 | lr.name = "30Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 377 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 378 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 379 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 380 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 381 | lr.name = "24Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 382 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 383 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 384 |  | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 385 | lr.name = ""; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 386 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(60)}}), | 
|  | 387 | 0); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 388 |  | 
|  | 389 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 390 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 391 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 392 |  | 
|  | 393 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 394 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 395 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 396 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 397 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 398 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 399 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 400 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 401 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 402 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 403 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 404 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 405 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 406 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 407 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 408 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 409 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 410 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 411 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 412 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 413 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 414 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 415 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 416 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 417 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 418 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
|  | 419 | {HWC_CONFIG_ID_90, {Fps(90.0f), Fps(90.0f)}}), | 
|  | 420 | 0); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 421 |  | 
|  | 422 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 423 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 424 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 425 |  | 
|  | 426 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 427 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 428 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 429 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 430 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 431 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 432 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 433 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 434 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 435 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 436 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 437 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 438 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 439 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 440 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 441 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 442 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 443 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 444 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 445 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 446 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 447 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 448 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 449 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 450 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 451 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
|  | 452 | {HWC_CONFIG_ID_60, {Fps(0.0f), Fps(120.0f)}}), | 
|  | 453 | 0); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 454 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 455 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 456 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 457 |  | 
|  | 458 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 459 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 460 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 461 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 462 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 463 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 464 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 465 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 466 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 467 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 468 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 469 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 470 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 471 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 472 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 473 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 474 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 475 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 476 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 477 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 478 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 479 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 480 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 481 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 482 | } | 
|  | 483 |  | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 484 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_multipleThreshold_60_90) { | 
|  | 485 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 90}; | 
|  | 486 | auto refreshRateConfigs = | 
|  | 487 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 488 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); | 
|  | 489 |  | 
|  | 490 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 491 | auto& lr = layers[0]; | 
|  | 492 |  | 
|  | 493 | lr.vote = LayerVoteType::Min; | 
|  | 494 | lr.name = "Min"; | 
|  | 495 | EXPECT_EQ(mExpected60Config, | 
|  | 496 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 497 |  | 
|  | 498 | lr.vote = LayerVoteType::Max; | 
|  | 499 | lr.name = "Max"; | 
|  | 500 | EXPECT_EQ(mExpected90Config, | 
|  | 501 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 502 |  | 
|  | 503 | lr.desiredRefreshRate = Fps(90.0f); | 
|  | 504 | lr.vote = LayerVoteType::Heuristic; | 
|  | 505 | lr.name = "90Hz Heuristic"; | 
|  | 506 | EXPECT_EQ(mExpected90Config, | 
|  | 507 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 508 |  | 
|  | 509 | lr.desiredRefreshRate = Fps(60.0f); | 
|  | 510 | lr.name = "60Hz Heuristic"; | 
|  | 511 | EXPECT_EQ(mExpected60Config, | 
|  | 512 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 513 |  | 
|  | 514 | lr.desiredRefreshRate = Fps(45.0f); | 
|  | 515 | lr.name = "45Hz Heuristic"; | 
|  | 516 | EXPECT_EQ(mExpected90Config, | 
|  | 517 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 518 |  | 
|  | 519 | lr.desiredRefreshRate = Fps(30.0f); | 
|  | 520 | lr.name = "30Hz Heuristic"; | 
|  | 521 | EXPECT_EQ(mExpected60Config, | 
|  | 522 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 523 |  | 
|  | 524 | lr.desiredRefreshRate = Fps(24.0f); | 
|  | 525 | lr.name = "24Hz Heuristic"; | 
|  | 526 | EXPECT_EQ(mExpected60Config, | 
|  | 527 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 528 | } | 
|  | 529 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 530 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_60_72_90) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 531 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 532 | std::make_unique<RefreshRateConfigs>(m60_72_90Device, | 
|  | 533 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 534 |  | 
|  | 535 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 536 | auto& lr = layers[0]; | 
|  | 537 |  | 
|  | 538 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 539 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 540 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 541 |  | 
|  | 542 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 543 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 544 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 545 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 546 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 547 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 548 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 549 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 550 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 551 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 552 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 553 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 554 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 555 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 556 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 557 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 558 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 559 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 560 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 561 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 562 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 563 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 564 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 565 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 566 | } | 
|  | 567 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 568 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_72_90_120) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 569 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 570 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 571 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 572 |  | 
|  | 573 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 574 | LayerRequirement{.weight = 1.0f}}; | 
|  | 575 | auto& lr1 = layers[0]; | 
|  | 576 | auto& lr2 = layers[1]; | 
|  | 577 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 578 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 579 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 580 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 581 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 582 | EXPECT_EQ(mExpected120Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 583 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 584 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 585 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 586 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 587 | lr2.desiredRefreshRate = Fps(48.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 588 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 589 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 590 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 591 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 592 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 593 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 594 | lr2.desiredRefreshRate = Fps(48.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 595 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 596 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 597 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 598 | } | 
|  | 599 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 600 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_90_120_DifferentTypes) { | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 601 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 602 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 603 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 604 |  | 
|  | 605 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 606 | LayerRequirement{.weight = 1.0f}}; | 
|  | 607 | auto& lr1 = layers[0]; | 
|  | 608 | auto& lr2 = layers[1]; | 
|  | 609 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 610 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 611 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 612 | lr1.name = "24Hz ExplicitDefault"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 613 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 614 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 615 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 616 | EXPECT_EQ(mExpected120Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 617 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 618 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 619 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 620 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 621 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 622 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 623 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 624 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 625 | EXPECT_EQ(mExpected120Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 626 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 627 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 628 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 629 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 630 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 631 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 632 | lr2.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 633 | lr2.name = "60Hz ExplicitDefault"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 634 | EXPECT_EQ(mExpected120Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 635 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 636 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 637 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 638 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 639 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 640 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 641 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 642 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 643 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 644 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 645 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 646 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 647 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 648 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 649 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 650 | lr2.vote = LayerVoteType::ExplicitDefault; | 
|  | 651 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 652 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 653 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 654 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 655 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 656 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 657 | lr1.name = "24Hz ExplicitDefault"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 658 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 659 | lr2.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 660 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 661 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 662 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 663 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 664 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 665 | lr1.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 666 | lr1.name = "24Hz Heuristic"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 667 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 668 | lr2.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 669 | lr2.name = "90Hz ExplicitDefault"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 670 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 671 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 672 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 673 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 674 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 675 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 676 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 677 | lr2.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 678 | lr2.name = "90Hz ExplicitDefault"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 679 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 680 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 681 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 682 | lr1.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 683 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 684 | lr1.name = "24Hz ExplicitDefault"; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 685 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 686 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 687 | lr2.name = "90Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 688 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 689 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 690 | } | 
|  | 691 |  | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 692 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_90_120_DifferentTypes_multipleThreshold) { | 
|  | 693 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 120}; | 
|  | 694 | auto refreshRateConfigs = | 
|  | 695 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 696 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); | 
|  | 697 |  | 
|  | 698 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 699 | LayerRequirement{.weight = 1.0f}}; | 
|  | 700 | auto& lr1 = layers[0]; | 
|  | 701 | auto& lr2 = layers[1]; | 
|  | 702 |  | 
|  | 703 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 704 | lr1.vote = LayerVoteType::ExplicitDefault; | 
|  | 705 | lr1.name = "24Hz ExplicitDefault"; | 
|  | 706 | lr2.desiredRefreshRate = Fps(60.0f); | 
|  | 707 | lr2.vote = LayerVoteType::Heuristic; | 
|  | 708 | lr2.name = "60Hz Heuristic"; | 
|  | 709 | EXPECT_EQ(mExpected120Config, | 
|  | 710 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 711 |  | 
|  | 712 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 713 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 714 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
|  | 715 | lr2.desiredRefreshRate = Fps(60.0f); | 
|  | 716 | lr2.vote = LayerVoteType::Heuristic; | 
|  | 717 | lr2.name = "60Hz Heuristic"; | 
|  | 718 | EXPECT_EQ(mExpected60Config, | 
|  | 719 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 720 |  | 
|  | 721 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 722 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 723 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
|  | 724 | lr2.desiredRefreshRate = Fps(60.0f); | 
|  | 725 | lr2.vote = LayerVoteType::ExplicitDefault; | 
|  | 726 | lr2.name = "60Hz ExplicitDefault"; | 
|  | 727 | EXPECT_EQ(mExpected72Config, | 
|  | 728 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 729 |  | 
|  | 730 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 731 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 732 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
|  | 733 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 734 | lr2.vote = LayerVoteType::Heuristic; | 
|  | 735 | lr2.name = "90Hz Heuristic"; | 
|  | 736 | EXPECT_EQ(mExpected90Config, | 
|  | 737 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 738 |  | 
|  | 739 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 740 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 741 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
|  | 742 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 743 | lr2.vote = LayerVoteType::ExplicitDefault; | 
|  | 744 | lr2.name = "90Hz Heuristic"; | 
|  | 745 | EXPECT_EQ(mExpected72Config, | 
|  | 746 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 747 |  | 
|  | 748 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 749 | lr1.vote = LayerVoteType::ExplicitDefault; | 
|  | 750 | lr1.name = "24Hz ExplicitDefault"; | 
|  | 751 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 752 | lr2.vote = LayerVoteType::Heuristic; | 
|  | 753 | lr2.name = "90Hz Heuristic"; | 
|  | 754 | EXPECT_EQ(mExpected90Config, | 
|  | 755 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 756 |  | 
|  | 757 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 758 | lr1.vote = LayerVoteType::Heuristic; | 
|  | 759 | lr1.name = "24Hz Heuristic"; | 
|  | 760 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 761 | lr2.vote = LayerVoteType::ExplicitDefault; | 
|  | 762 | lr2.name = "90Hz ExplicitDefault"; | 
|  | 763 | EXPECT_EQ(mExpected72Config, | 
|  | 764 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 765 |  | 
|  | 766 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 767 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 768 | lr1.name = "24Hz ExplicitExactOrMultiple"; | 
|  | 769 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 770 | lr2.vote = LayerVoteType::ExplicitDefault; | 
|  | 771 | lr2.name = "90Hz ExplicitDefault"; | 
|  | 772 | EXPECT_EQ(mExpected72Config, | 
|  | 773 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 774 |  | 
|  | 775 | lr1.desiredRefreshRate = Fps(24.0f); | 
|  | 776 | lr1.vote = LayerVoteType::ExplicitDefault; | 
|  | 777 | lr1.name = "24Hz ExplicitDefault"; | 
|  | 778 | lr2.desiredRefreshRate = Fps(90.0f); | 
|  | 779 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 780 | lr2.name = "90Hz ExplicitExactOrMultiple"; | 
|  | 781 | EXPECT_EQ(mExpected90Config, | 
|  | 782 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 783 | } | 
|  | 784 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 785 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 786 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 787 | std::make_unique<RefreshRateConfigs>(m30_60Device, | 
|  | 788 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 789 |  | 
|  | 790 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 791 | auto& lr = layers[0]; | 
|  | 792 |  | 
|  | 793 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 794 | EXPECT_EQ(mExpected30Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 795 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 796 |  | 
|  | 797 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 798 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 799 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 800 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 801 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 802 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 803 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 804 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 805 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 806 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 807 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 808 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 809 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 810 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 811 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 812 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 813 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 814 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 815 | EXPECT_EQ(mExpected30Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 816 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 817 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 818 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 819 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 820 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 821 | } | 
|  | 822 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 823 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_72_90) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 824 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 825 | std::make_unique<RefreshRateConfigs>(m30_60_72_90Device, | 
|  | 826 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 827 |  | 
|  | 828 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 829 | auto& lr = layers[0]; | 
|  | 830 |  | 
|  | 831 | lr.vote = LayerVoteType::Min; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 832 | lr.name = "Min"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 833 | EXPECT_EQ(mExpected30Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 834 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 835 |  | 
|  | 836 | lr.vote = LayerVoteType::Max; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 837 | lr.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 838 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 839 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 840 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 841 | lr.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 842 | lr.vote = LayerVoteType::Heuristic; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 843 | lr.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 844 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 845 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 846 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 847 | lr.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 848 | lr.name = "60Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 849 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 850 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 851 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 852 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 853 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 854 | lr.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 855 | lr.name = "45Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 856 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 857 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 858 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 859 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 860 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 861 | lr.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 862 | lr.name = "30Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 863 | EXPECT_EQ(mExpected30Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 864 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 865 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 866 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 867 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 868 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 869 | lr.name = "24Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 870 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 871 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 872 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 873 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 874 |  | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 875 | lr.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 876 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 877 | lr.name = "24Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 878 | EXPECT_EQ(mExpected72Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 879 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 880 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 881 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 882 | } | 
|  | 883 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 884 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_PriorityTest) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 885 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 886 | std::make_unique<RefreshRateConfigs>(m30_60_90Device, | 
|  | 887 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 888 |  | 
|  | 889 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 890 | LayerRequirement{.weight = 1.0f}}; | 
|  | 891 | auto& lr1 = layers[0]; | 
|  | 892 | auto& lr2 = layers[1]; | 
|  | 893 |  | 
|  | 894 | lr1.vote = LayerVoteType::Min; | 
|  | 895 | lr2.vote = LayerVoteType::Max; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 896 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 897 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 898 |  | 
|  | 899 | lr1.vote = LayerVoteType::Min; | 
|  | 900 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 901 | lr2.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 902 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 903 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 904 |  | 
|  | 905 | lr1.vote = LayerVoteType::Min; | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 906 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 907 | lr2.desiredRefreshRate = Fps(24.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 908 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 909 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 910 |  | 
|  | 911 | lr1.vote = LayerVoteType::Max; | 
|  | 912 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 913 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 914 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 915 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 916 |  | 
|  | 917 | lr1.vote = LayerVoteType::Max; | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 918 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 919 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 920 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 921 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 922 |  | 
|  | 923 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 924 | lr1.desiredRefreshRate = Fps(15.0f); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 925 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 926 | lr2.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 927 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 928 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 929 |  | 
|  | 930 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 931 | lr1.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 932 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 933 | lr2.desiredRefreshRate = Fps(45.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 934 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 935 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 936 | } | 
|  | 937 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 938 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_24FpsVideo) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 939 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 940 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 941 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 942 |  | 
|  | 943 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 944 | auto& lr = layers[0]; | 
|  | 945 |  | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 946 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 947 | for (float fps = 23.0f; fps < 25.0f; fps += 0.1f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 948 | lr.desiredRefreshRate = Fps(fps); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 949 | const auto& refreshRate = | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 950 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 951 | EXPECT_EQ(mExpected60Config, refreshRate) << fps << "Hz chooses " << refreshRate.getName(); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 952 | } | 
|  | 953 | } | 
|  | 954 |  | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 955 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_24FpsVideo_multipleThreshold_60_120) { | 
|  | 956 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 120}; | 
|  | 957 | auto refreshRateConfigs = | 
|  | 958 | std::make_unique<RefreshRateConfigs>(m60_120Device, | 
|  | 959 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); | 
|  | 960 |  | 
|  | 961 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 962 | auto& lr = layers[0]; | 
|  | 963 |  | 
|  | 964 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 965 | for (float fps = 23.0f; fps < 25.0f; fps += 0.1f) { | 
|  | 966 | lr.desiredRefreshRate = Fps(fps); | 
|  | 967 | const auto& refreshRate = | 
|  | 968 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}); | 
|  | 969 | EXPECT_EQ(mExpected60Config, refreshRate) << fps << "Hz chooses " << refreshRate.getName(); | 
|  | 970 | } | 
|  | 971 | } | 
|  | 972 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 973 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_getBestRefreshRate_Explicit) { | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 974 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 975 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 976 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 977 |  | 
|  | 978 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 979 | LayerRequirement{.weight = 1.0f}}; | 
|  | 980 | auto& lr1 = layers[0]; | 
|  | 981 | auto& lr2 = layers[1]; | 
|  | 982 |  | 
|  | 983 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 984 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 985 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 986 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 987 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 988 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 989 |  | 
|  | 990 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 991 | lr1.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 992 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 993 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 994 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 995 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 996 |  | 
|  | 997 | lr1.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 998 | lr1.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 999 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1000 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1001 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1002 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 1003 | } | 
|  | 1004 |  | 
| Ana Krulec | 72f0d6e | 2020-01-06 15:24:47 -0800 | [diff] [blame] | 1005 | TEST_F(RefreshRateConfigsTest, testInPolicy) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1006 | ASSERT_TRUE(mExpectedAlmost60Config.inPolicy(Fps(60.000004f), Fps(60.000004f))); | 
|  | 1007 | ASSERT_TRUE(mExpectedAlmost60Config.inPolicy(Fps(59.0f), Fps(60.1f))); | 
|  | 1008 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(Fps(75.0f), Fps(90.0f))); | 
|  | 1009 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(Fps(60.0011f), Fps(90.0f))); | 
|  | 1010 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(Fps(50.0f), Fps(59.998f))); | 
| Ana Krulec | 72f0d6e | 2020-01-06 15:24:47 -0800 | [diff] [blame] | 1011 | } | 
|  | 1012 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1013 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_75HzContent) { | 
| Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 1014 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1015 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1016 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 1017 |  | 
|  | 1018 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1019 | auto& lr = layers[0]; | 
|  | 1020 |  | 
| Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 1021 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 1022 | for (float fps = 75.0f; fps < 100.0f; fps += 0.1f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1023 | lr.desiredRefreshRate = Fps(fps); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1024 | const auto& refreshRate = | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1025 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1026 | EXPECT_EQ(mExpected90Config, refreshRate) << fps << "Hz chooses " << refreshRate.getName(); | 
| Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 1027 | } | 
|  | 1028 | } | 
|  | 1029 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1030 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_Multiples) { | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1031 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1032 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1033 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1034 |  | 
|  | 1035 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1036 | LayerRequirement{.weight = 1.0f}}; | 
|  | 1037 | auto& lr1 = layers[0]; | 
|  | 1038 | auto& lr2 = layers[1]; | 
|  | 1039 |  | 
|  | 1040 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1041 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1042 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1043 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1044 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1045 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1046 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1047 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1048 |  | 
|  | 1049 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1050 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1051 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1052 | lr2.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1053 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1054 | lr2.name = "90Hz ExplicitDefault"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1055 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1056 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1057 |  | 
|  | 1058 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1059 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1060 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1061 | lr2.vote = LayerVoteType::Max; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1062 | lr2.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1063 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1064 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1065 |  | 
|  | 1066 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1067 | lr1.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1068 | lr1.name = "30Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1069 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1070 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1071 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1072 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1073 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1074 |  | 
|  | 1075 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1076 | lr1.desiredRefreshRate = Fps(30.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1077 | lr1.name = "30Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1078 | lr2.vote = LayerVoteType::Max; | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1079 | lr2.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1080 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1081 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1082 | } | 
|  | 1083 |  | 
|  | 1084 | TEST_F(RefreshRateConfigsTest, scrollWhileWatching60fps_60_90) { | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1085 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1086 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1087 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1088 |  | 
|  | 1089 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1090 | LayerRequirement{.weight = 1.0f}}; | 
|  | 1091 | auto& lr1 = layers[0]; | 
|  | 1092 | auto& lr2 = layers[1]; | 
|  | 1093 |  | 
|  | 1094 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1095 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1096 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1097 | lr2.vote = LayerVoteType::NoVote; | 
|  | 1098 | lr2.name = "NoVote"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1099 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1100 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1101 |  | 
|  | 1102 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1103 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1104 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1105 | lr2.vote = LayerVoteType::NoVote; | 
|  | 1106 | lr2.name = "NoVote"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1107 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1108 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1109 |  | 
|  | 1110 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1111 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1112 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1113 | lr2.vote = LayerVoteType::Max; | 
|  | 1114 | lr2.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1115 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1116 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1117 |  | 
|  | 1118 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1119 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1120 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1121 | lr2.vote = LayerVoteType::Max; | 
|  | 1122 | lr2.name = "Max"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1123 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1124 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1125 |  | 
|  | 1126 | // The other layer starts to provide buffers | 
|  | 1127 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1128 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1129 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1130 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1131 | lr2.desiredRefreshRate = Fps(90.0f); | 
| Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1132 | lr2.name = "90Hz Heuristic"; | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1133 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1134 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1135 | } | 
|  | 1136 |  | 
|  | 1137 | TEST_F(RefreshRateConfigsTest, touchConsidered) { | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1138 | RefreshRateConfigs::GlobalSignals consideredSignals; | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1139 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1140 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1141 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1142 |  | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1143 | refreshRateConfigs->getBestRefreshRate({}, {.touch = false, .idle = false}, &consideredSignals); | 
|  | 1144 | EXPECT_EQ(false, consideredSignals.touch); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1145 |  | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1146 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true, .idle = false}, &consideredSignals); | 
|  | 1147 | EXPECT_EQ(true, consideredSignals.touch); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1148 |  | 
|  | 1149 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1150 | LayerRequirement{.weight = 1.0f}}; | 
|  | 1151 | auto& lr1 = layers[0]; | 
|  | 1152 | auto& lr2 = layers[1]; | 
|  | 1153 |  | 
|  | 1154 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1155 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1156 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1157 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1158 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1159 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1160 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false}, | 
|  | 1161 | &consideredSignals); | 
|  | 1162 | EXPECT_EQ(true, consideredSignals.touch); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1163 |  | 
|  | 1164 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1165 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1166 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1167 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1168 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1169 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1170 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false}, | 
|  | 1171 | &consideredSignals); | 
|  | 1172 | EXPECT_EQ(false, consideredSignals.touch); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1173 |  | 
|  | 1174 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1175 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1176 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1177 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1178 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1179 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1180 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false}, | 
|  | 1181 | &consideredSignals); | 
|  | 1182 | EXPECT_EQ(true, consideredSignals.touch); | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1183 |  | 
|  | 1184 | lr1.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1185 | lr1.desiredRefreshRate = Fps(60.0f); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1186 | lr1.name = "60Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1187 | lr2.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1188 | lr2.desiredRefreshRate = Fps(60.0f); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1189 | lr2.name = "60Hz Heuristic"; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1190 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false}, | 
|  | 1191 | &consideredSignals); | 
|  | 1192 | EXPECT_EQ(false, consideredSignals.touch); | 
| Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1193 | } | 
|  | 1194 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1195 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitDefault) { | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1196 | auto refreshRateConfigs = | 
|  | 1197 | std::make_unique<RefreshRateConfigs>(m60_90_72_120Device, /*currentConfigId=*/ | 
|  | 1198 | HWC_CONFIG_ID_60); | 
| Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1199 |  | 
|  | 1200 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1201 | auto& lr = layers[0]; | 
|  | 1202 |  | 
|  | 1203 | // Prepare a table with the vote and the expected refresh rate | 
|  | 1204 | const std::vector<std::pair<float, float>> testCases = { | 
|  | 1205 | {130, 120}, {120, 120}, {119, 120}, {110, 120}, | 
|  | 1206 |  | 
|  | 1207 | {100, 90},  {90, 90},   {89, 90}, | 
|  | 1208 |  | 
|  | 1209 | {80, 72},   {73, 72},   {72, 72},   {71, 72},   {70, 72}, | 
|  | 1210 |  | 
|  | 1211 | {65, 60},   {60, 60},   {59, 60},   {58, 60}, | 
|  | 1212 |  | 
|  | 1213 | {55, 90},   {50, 90},   {45, 90}, | 
|  | 1214 |  | 
|  | 1215 | {42, 120},  {40, 120},  {39, 120}, | 
|  | 1216 |  | 
|  | 1217 | {37, 72},   {36, 72},   {35, 72}, | 
|  | 1218 |  | 
|  | 1219 | {30, 60}, | 
|  | 1220 | }; | 
|  | 1221 |  | 
|  | 1222 | for (const auto& test : testCases) { | 
|  | 1223 | lr.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1224 | lr.desiredRefreshRate = Fps(test.first); | 
| Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1225 |  | 
|  | 1226 | std::stringstream ss; | 
|  | 1227 | ss << "ExplicitDefault " << test.first << " fps"; | 
|  | 1228 | lr.name = ss.str(); | 
|  | 1229 |  | 
|  | 1230 | const auto& refreshRate = | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1231 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}); | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1232 | EXPECT_TRUE(refreshRate.getFps().equalsWithMargin(Fps(test.second))) | 
| Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1233 | << "Expecting " << test.first << "fps => " << test.second << "Hz"; | 
|  | 1234 | } | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 1235 | } | 
|  | 1236 |  | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1237 | TEST_F(RefreshRateConfigsTest, | 
|  | 1238 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_ignoresTouchFlag) { | 
|  | 1239 | auto refreshRateConfigs = | 
|  | 1240 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1241 | /*currentConfigId=*/HWC_CONFIG_ID_90); | 
|  | 1242 |  | 
|  | 1243 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1244 | {HWC_CONFIG_ID_90, {Fps(90.f), Fps(90.f)}, {Fps(60.f), Fps(90.f)}}), | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1245 | 0); | 
|  | 1246 |  | 
|  | 1247 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1248 | auto& lr = layers[0]; | 
|  | 1249 |  | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1250 | RefreshRateConfigs::GlobalSignals consideredSignals; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1251 | lr.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1252 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1253 | lr.name = "60Hz ExplicitDefault"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1254 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1255 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1256 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = true}, | 
|  | 1257 | &consideredSignals)); | 
|  | 1258 | EXPECT_EQ(false, consideredSignals.touch); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1259 | } | 
|  | 1260 |  | 
|  | 1261 | TEST_F(RefreshRateConfigsTest, | 
|  | 1262 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_ignoresIdleFlag) { | 
|  | 1263 | auto refreshRateConfigs = | 
|  | 1264 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1265 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1266 |  | 
|  | 1267 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1268 | {HWC_CONFIG_ID_60, {Fps(60.f), Fps(60.f)}, {Fps(60.f), Fps(90.f)}}), | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1269 | 0); | 
|  | 1270 |  | 
|  | 1271 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1272 | auto& lr = layers[0]; | 
|  | 1273 |  | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1274 | lr.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1275 | lr.desiredRefreshRate = Fps(90.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1276 | lr.name = "90Hz ExplicitDefault"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1277 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1278 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1279 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = true})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1280 | } | 
|  | 1281 |  | 
|  | 1282 | TEST_F(RefreshRateConfigsTest, | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1283 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_onlySwitchesRatesForExplicitFocusedLayers) { | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1284 | auto refreshRateConfigs = | 
|  | 1285 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1286 | /*currentConfigId=*/HWC_CONFIG_ID_90); | 
|  | 1287 |  | 
|  | 1288 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1289 | {HWC_CONFIG_ID_90, {Fps(90.f), Fps(90.f)}, {Fps(60.f), Fps(90.f)}}), | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1290 | 0); | 
|  | 1291 |  | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1292 | RefreshRateConfigs::GlobalSignals consideredSignals; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1293 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1294 | refreshRateConfigs->getBestRefreshRate({}, {.touch = false, .idle = false}, | 
|  | 1295 | &consideredSignals)); | 
|  | 1296 | EXPECT_EQ(false, consideredSignals.touch); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1297 |  | 
|  | 1298 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1299 | auto& lr = layers[0]; | 
|  | 1300 |  | 
|  | 1301 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1302 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1303 | lr.name = "60Hz ExplicitExactOrMultiple"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1304 | lr.focused = false; | 
|  | 1305 | EXPECT_EQ(mExpected90Config, | 
|  | 1306 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1307 |  | 
|  | 1308 | lr.focused = true; | 
| Ady Abraham | 20c029c | 2020-07-06 12:58:05 -0700 | [diff] [blame] | 1309 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1310 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1311 |  | 
|  | 1312 | lr.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1313 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1314 | lr.name = "60Hz ExplicitDefault"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1315 | lr.focused = false; | 
|  | 1316 | EXPECT_EQ(mExpected90Config, | 
|  | 1317 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1318 |  | 
|  | 1319 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1320 | EXPECT_EQ(mExpected60Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1321 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1322 |  | 
|  | 1323 | lr.vote = LayerVoteType::Heuristic; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1324 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1325 | lr.name = "60Hz Heuristic"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1326 | lr.focused = false; | 
|  | 1327 | EXPECT_EQ(mExpected90Config, | 
|  | 1328 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1329 |  | 
|  | 1330 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1331 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1332 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1333 |  | 
|  | 1334 | lr.vote = LayerVoteType::Max; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1335 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1336 | lr.name = "60Hz Max"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1337 | lr.focused = false; | 
|  | 1338 | EXPECT_EQ(mExpected90Config, | 
|  | 1339 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1340 |  | 
|  | 1341 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1342 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1343 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1344 |  | 
|  | 1345 | lr.vote = LayerVoteType::Min; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1346 | lr.desiredRefreshRate = Fps(60.0f); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1347 | lr.name = "60Hz Min"; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1348 | lr.focused = false; | 
|  | 1349 | EXPECT_EQ(mExpected90Config, | 
|  | 1350 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1351 |  | 
|  | 1352 | lr.focused = true; | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1353 | EXPECT_EQ(mExpected90Config, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1354 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1355 | } | 
|  | 1356 |  | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1357 | TEST_F(RefreshRateConfigsTest, groupSwitchingNotAllowed) { | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1358 | auto refreshRateConfigs = | 
| Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1359 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1360 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1361 |  | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1362 | // The default policy doesn't allow group switching. Verify that no | 
|  | 1363 | // group switches are performed. | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1364 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1365 | auto& layer = layers[0]; | 
|  | 1366 | layer.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1367 | layer.desiredRefreshRate = Fps(90.0f); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1368 | layer.seamlessness = Seamlessness::SeamedAndSeamless; | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1369 | layer.name = "90Hz ExplicitDefault"; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1370 | layer.focused = true; | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1371 |  | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1372 | ASSERT_EQ(HWC_CONFIG_ID_60, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1373 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1374 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1375 | } | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1376 |  | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1377 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayer) { | 
|  | 1378 | auto refreshRateConfigs = | 
|  | 1379 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1380 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1381 | RefreshRateConfigs::Policy policy; | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1382 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1383 | policy.allowGroupSwitching = true; | 
|  | 1384 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1385 |  | 
|  | 1386 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1387 | auto& layer = layers[0]; | 
|  | 1388 | layer.vote = LayerVoteType::ExplicitDefault; | 
|  | 1389 | layer.desiredRefreshRate = Fps(90.0f); | 
|  | 1390 | layer.seamlessness = Seamlessness::SeamedAndSeamless; | 
|  | 1391 | layer.name = "90Hz ExplicitDefault"; | 
|  | 1392 | layer.focused = true; | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1393 | ASSERT_EQ(HWC_CONFIG_ID_90, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1394 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1395 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1396 | } | 
|  | 1397 |  | 
|  | 1398 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerOnlySeamless) { | 
|  | 1399 | auto refreshRateConfigs = | 
|  | 1400 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1401 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1402 | RefreshRateConfigs::Policy policy; | 
|  | 1403 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1404 | policy.allowGroupSwitching = true; | 
|  | 1405 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1406 |  | 
|  | 1407 | // Verify that we won't change the group if seamless switch is required. | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1408 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1409 | auto& layer = layers[0]; | 
|  | 1410 | layer.vote = LayerVoteType::ExplicitDefault; | 
|  | 1411 | layer.desiredRefreshRate = Fps(90.0f); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1412 | layer.seamlessness = Seamlessness::OnlySeamless; | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1413 | layer.name = "90Hz ExplicitDefault"; | 
|  | 1414 | layer.focused = true; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1415 | ASSERT_EQ(HWC_CONFIG_ID_60, | 
|  | 1416 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1417 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1418 | } | 
|  | 1419 |  | 
|  | 1420 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerOnlySeamlessDefaultFps) { | 
|  | 1421 | auto refreshRateConfigs = | 
|  | 1422 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1423 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1424 | RefreshRateConfigs::Policy policy; | 
|  | 1425 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1426 | policy.allowGroupSwitching = true; | 
|  | 1427 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1428 |  | 
|  | 1429 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1430 |  | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1431 | // Verify that we won't do a seamless switch if we request the same mode as the default | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1432 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1433 | auto& layer = layers[0]; | 
|  | 1434 | layer.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1435 | layer.desiredRefreshRate = Fps(60.0f); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1436 | layer.seamlessness = Seamlessness::OnlySeamless; | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1437 | layer.name = "60Hz ExplicitDefault"; | 
|  | 1438 | layer.focused = true; | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1439 | ASSERT_EQ(HWC_CONFIG_ID_90, | 
|  | 1440 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1441 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1442 | } | 
|  | 1443 |  | 
|  | 1444 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerDefaultSeamlessness) { | 
|  | 1445 | auto refreshRateConfigs = | 
|  | 1446 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1447 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1448 | RefreshRateConfigs::Policy policy; | 
|  | 1449 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1450 | policy.allowGroupSwitching = true; | 
|  | 1451 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1452 |  | 
|  | 1453 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1454 |  | 
|  | 1455 | // Verify that if the current config is in another group and there are no layers with | 
|  | 1456 | // seamlessness=SeamedAndSeamless we'll go back to the default group. | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1457 |  | 
|  | 1458 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1459 | auto& layer = layers[0]; | 
|  | 1460 | layer.vote = LayerVoteType::ExplicitDefault; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1461 | layer.desiredRefreshRate = Fps(60.0f); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1462 | layer.seamlessness = Seamlessness::Default; | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1463 | layer.name = "60Hz ExplicitDefault"; | 
|  | 1464 | layer.focused = true; | 
|  | 1465 |  | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1466 | ASSERT_EQ(HWC_CONFIG_ID_60, | 
|  | 1467 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1468 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1469 | } | 
|  | 1470 |  | 
|  | 1471 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersOnlySeamlessAndSeamed) { | 
|  | 1472 | auto refreshRateConfigs = | 
|  | 1473 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1474 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1475 | RefreshRateConfigs::Policy policy; | 
|  | 1476 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1477 | policy.allowGroupSwitching = true; | 
|  | 1478 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1479 |  | 
|  | 1480 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1481 |  | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1482 | // If there's a layer with seamlessness=SeamedAndSeamless, another layer with | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1483 | // seamlessness=OnlySeamless can't change the mode group. | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1484 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1485 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
|  | 1486 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 1487 | layers[0].seamlessness = Seamlessness::OnlySeamless; | 
|  | 1488 | layers[0].name = "60Hz ExplicitDefault"; | 
|  | 1489 | layers[0].focused = true; | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1490 |  | 
|  | 1491 | layers.push_back(LayerRequirement{.weight = 0.5f}); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1492 | layers[1].vote = LayerVoteType::ExplicitDefault; | 
|  | 1493 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; | 
|  | 1494 | layers[1].desiredRefreshRate = Fps(90.0f); | 
|  | 1495 | layers[1].name = "90Hz ExplicitDefault"; | 
|  | 1496 | layers[1].focused = false; | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1497 |  | 
|  | 1498 | ASSERT_EQ(HWC_CONFIG_ID_90, | 
|  | 1499 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1500 | .getModeId()); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1501 | } | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1502 |  | 
| Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1503 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersDefaultFocusedAndSeamed) { | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1504 | auto refreshRateConfigs = | 
|  | 1505 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1506 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1507 | RefreshRateConfigs::Policy policy; | 
|  | 1508 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1509 | policy.allowGroupSwitching = true; | 
|  | 1510 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1511 |  | 
|  | 1512 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1513 |  | 
| Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1514 | // If there's a focused layer with seamlessness=SeamedAndSeamless, another layer with | 
|  | 1515 | // seamlessness=Default can't change the mode group back to the group of the default | 
|  | 1516 | // mode. | 
|  | 1517 | // For example, this may happen when a video playback requests and gets a seamed switch, | 
|  | 1518 | // but another layer (with default seamlessness) starts animating. The animating layer | 
|  | 1519 | // should not cause a seamed switch. | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1520 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1521 | layers[0].seamlessness = Seamlessness::Default; | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1522 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 1523 | layers[0].focused = true; | 
|  | 1524 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
|  | 1525 | layers[0].name = "60Hz ExplicitDefault"; | 
|  | 1526 |  | 
| Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1527 | layers.push_back(LayerRequirement{.weight = 0.1f}); | 
|  | 1528 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; | 
|  | 1529 | layers[1].desiredRefreshRate = Fps(90.0f); | 
|  | 1530 | layers[1].focused = true; | 
|  | 1531 | layers[1].vote = LayerVoteType::ExplicitDefault; | 
|  | 1532 | layers[1].name = "90Hz ExplicitDefault"; | 
|  | 1533 |  | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1534 | ASSERT_EQ(HWC_CONFIG_ID_90, | 
|  | 1535 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1536 | .getModeId()); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1537 | } | 
|  | 1538 |  | 
| Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1539 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersDefaultNotFocusedAndSeamed) { | 
|  | 1540 | auto refreshRateConfigs = | 
|  | 1541 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1542 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1543 | RefreshRateConfigs::Policy policy; | 
|  | 1544 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1545 | policy.allowGroupSwitching = true; | 
|  | 1546 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1547 |  | 
|  | 1548 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
|  | 1549 |  | 
|  | 1550 | // Layer with seamlessness=Default can change the mode group if there's a not | 
|  | 1551 | // focused layer with seamlessness=SeamedAndSeamless. This happens for example, | 
|  | 1552 | // when in split screen mode the user switches between the two visible applications. | 
|  | 1553 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1554 | layers[0].seamlessness = Seamlessness::Default; | 
|  | 1555 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 1556 | layers[0].focused = true; | 
|  | 1557 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
|  | 1558 | layers[0].name = "60Hz ExplicitDefault"; | 
|  | 1559 |  | 
|  | 1560 | layers.push_back(LayerRequirement{.weight = 0.7f}); | 
| Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1561 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; | 
|  | 1562 | layers[1].desiredRefreshRate = Fps(90.0f); | 
|  | 1563 | layers[1].focused = false; | 
|  | 1564 | layers[1].vote = LayerVoteType::ExplicitDefault; | 
|  | 1565 | layers[1].name = "90Hz ExplicitDefault"; | 
|  | 1566 |  | 
| Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1567 | ASSERT_EQ(HWC_CONFIG_ID_60, | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1568 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
|  | 1569 | .getModeId()); | 
|  | 1570 | } | 
|  | 1571 |  | 
|  | 1572 | TEST_F(RefreshRateConfigsTest, nonSeamlessVotePrefersSeamlessSwitches) { | 
|  | 1573 | auto refreshRateConfigs = | 
|  | 1574 | std::make_unique<RefreshRateConfigs>(m30_60Device, | 
|  | 1575 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1576 |  | 
|  | 1577 | // Allow group switching. | 
|  | 1578 | RefreshRateConfigs::Policy policy; | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1579 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1580 | policy.allowGroupSwitching = true; | 
|  | 1581 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1582 |  | 
|  | 1583 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1584 | auto& layer = layers[0]; | 
|  | 1585 | layer.vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1586 | layer.desiredRefreshRate = Fps(60.0f); | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1587 | layer.seamlessness = Seamlessness::SeamedAndSeamless; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1588 | layer.name = "60Hz ExplicitExactOrMultiple"; | 
|  | 1589 | layer.focused = true; | 
|  | 1590 |  | 
|  | 1591 | ASSERT_EQ(HWC_CONFIG_ID_60, | 
|  | 1592 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1593 | .getModeId()); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1594 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1595 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_120); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1596 | ASSERT_EQ(HWC_CONFIG_ID_120, | 
|  | 1597 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1598 | .getModeId()); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1599 | } | 
|  | 1600 |  | 
|  | 1601 | TEST_F(RefreshRateConfigsTest, nonSeamlessExactAndSeamlessMultipleLayers) { | 
|  | 1602 | auto refreshRateConfigs = | 
|  | 1603 | std::make_unique<RefreshRateConfigs>(m25_30_50_60Device, | 
|  | 1604 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1605 |  | 
|  | 1606 | // Allow group switching. | 
|  | 1607 | RefreshRateConfigs::Policy policy; | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1608 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1609 | policy.allowGroupSwitching = true; | 
|  | 1610 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1611 |  | 
|  | 1612 | auto layers = std::vector< | 
|  | 1613 | LayerRequirement>{LayerRequirement{.name = "60Hz ExplicitDefault", | 
|  | 1614 | .vote = LayerVoteType::ExplicitDefault, | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1615 | .desiredRefreshRate = Fps(60.0f), | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1616 | .seamlessness = Seamlessness::SeamedAndSeamless, | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1617 | .weight = 0.5f, | 
|  | 1618 | .focused = false}, | 
|  | 1619 | LayerRequirement{.name = "25Hz ExplicitExactOrMultiple", | 
|  | 1620 | .vote = LayerVoteType::ExplicitExactOrMultiple, | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1621 | .desiredRefreshRate = Fps(25.0f), | 
| Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1622 | .seamlessness = Seamlessness::OnlySeamless, | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1623 | .weight = 1.0f, | 
|  | 1624 | .focused = true}}; | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1625 |  | 
|  | 1626 | ASSERT_EQ(HWC_CONFIG_ID_50, | 
|  | 1627 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1628 | .getModeId()); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1629 |  | 
| Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1630 | auto& seamedLayer = layers[0]; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1631 | seamedLayer.name = "30Hz ExplicitDefault", seamedLayer.desiredRefreshRate = Fps(30.0f); | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1632 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_30); | 
| Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1633 |  | 
|  | 1634 | ASSERT_EQ(HWC_CONFIG_ID_25, | 
|  | 1635 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1636 | .getModeId()); | 
| Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1637 | } | 
|  | 1638 |  | 
| Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1639 | TEST_F(RefreshRateConfigsTest, minLayersDontTrigerSeamedSwitch) { | 
|  | 1640 | auto refreshRateConfigs = | 
|  | 1641 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, | 
|  | 1642 | /*currentConfigId=*/HWC_CONFIG_ID_90); | 
|  | 1643 |  | 
|  | 1644 | // Allow group switching. | 
|  | 1645 | RefreshRateConfigs::Policy policy; | 
|  | 1646 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; | 
|  | 1647 | policy.allowGroupSwitching = true; | 
|  | 1648 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); | 
|  | 1649 |  | 
|  | 1650 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.name = "Min", | 
|  | 1651 | .vote = LayerVoteType::Min, | 
|  | 1652 | .weight = 1.f, | 
|  | 1653 | .focused = true}}; | 
|  | 1654 |  | 
|  | 1655 | ASSERT_EQ(HWC_CONFIG_ID_90, | 
|  | 1656 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}) | 
|  | 1657 | .getModeId()); | 
|  | 1658 | } | 
|  | 1659 |  | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1660 | TEST_F(RefreshRateConfigsTest, primaryVsAppRequestPolicy) { | 
|  | 1661 | auto refreshRateConfigs = | 
| Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1662 | std::make_unique<RefreshRateConfigs>(m30_60_90Device, | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1663 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1664 |  | 
|  | 1665 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1666 | layers[0].name = "Test layer"; | 
|  | 1667 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1668 | // Return the config ID from calling getBestRefreshRate() for a single layer with the | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1669 | // given voteType and fps. | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1670 | auto getFrameRate = [&](LayerVoteType voteType, Fps fps, bool touchActive = false, | 
| Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 1671 | bool focused = true) -> DisplayModeId { | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1672 | layers[0].vote = voteType; | 
|  | 1673 | layers[0].desiredRefreshRate = fps; | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1674 | layers[0].focused = focused; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1675 | return refreshRateConfigs->getBestRefreshRate(layers, {.touch = touchActive, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1676 | .getModeId(); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1677 | }; | 
|  | 1678 |  | 
|  | 1679 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1680 | {HWC_CONFIG_ID_60, {Fps(30.f), Fps(60.f)}, {Fps(30.f), Fps(90.f)}}), | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1681 | 0); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1682 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1683 | refreshRateConfigs->getBestRefreshRate({}, {.touch = false, .idle = false}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1684 | .getModeId()); | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1685 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::NoVote, Fps(90.f))); | 
|  | 1686 | EXPECT_EQ(HWC_CONFIG_ID_30, getFrameRate(LayerVoteType::Min, Fps(90.f))); | 
|  | 1687 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, Fps(90.f))); | 
|  | 1688 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Heuristic, Fps(90.f))); | 
|  | 1689 | EXPECT_EQ(HWC_CONFIG_ID_90, getFrameRate(LayerVoteType::ExplicitDefault, Fps(90.f))); | 
|  | 1690 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitExactOrMultiple, Fps(90.f))); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1691 |  | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1692 | // Layers not focused are not allowed to override primary config | 
|  | 1693 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1694 | getFrameRate(LayerVoteType::ExplicitDefault, Fps(90.f), /*touch=*/false, | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1695 | /*focused=*/false)); | 
|  | 1696 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1697 | getFrameRate(LayerVoteType::ExplicitExactOrMultiple, Fps(90.f), /*touch=*/false, | 
| Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1698 | /*focused=*/false)); | 
|  | 1699 |  | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1700 | // Touch boost should be restricted to the primary range. | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1701 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, Fps(90.f), /*touch=*/true)); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1702 | // When we're higher than the primary range max due to a layer frame rate setting, touch boost | 
|  | 1703 | // shouldn't drag us back down to the primary range max. | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1704 | EXPECT_EQ(HWC_CONFIG_ID_90, | 
|  | 1705 | getFrameRate(LayerVoteType::ExplicitDefault, Fps(90.f), /*touch=*/true)); | 
| Ady Abraham | 20c029c | 2020-07-06 12:58:05 -0700 | [diff] [blame] | 1706 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1707 | getFrameRate(LayerVoteType::ExplicitExactOrMultiple, Fps(90.f), /*touch=*/true)); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1708 |  | 
|  | 1709 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1710 | {HWC_CONFIG_ID_60, {Fps(60.f), Fps(60.f)}, {Fps(60.f), Fps(60.f)}}), | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1711 | 0); | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1712 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::NoVote, Fps(90.f))); | 
|  | 1713 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Min, Fps(90.f))); | 
|  | 1714 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, Fps(90.f))); | 
|  | 1715 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Heuristic, Fps(90.f))); | 
|  | 1716 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitDefault, Fps(90.f))); | 
|  | 1717 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitExactOrMultiple, Fps(90.f))); | 
| Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1718 | } | 
|  | 1719 |  | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1720 | TEST_F(RefreshRateConfigsTest, idle) { | 
|  | 1721 | auto refreshRateConfigs = | 
|  | 1722 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1723 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1724 |  | 
|  | 1725 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1726 | layers[0].name = "Test layer"; | 
|  | 1727 |  | 
| Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 1728 | const auto getIdleFrameRate = [&](LayerVoteType voteType, bool touchActive) -> DisplayModeId { | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1729 | layers[0].vote = voteType; | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1730 | layers[0].desiredRefreshRate = Fps(90.f); | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1731 | RefreshRateConfigs::GlobalSignals consideredSignals; | 
|  | 1732 | const auto configId = | 
|  | 1733 | refreshRateConfigs | 
|  | 1734 | ->getBestRefreshRate(layers, {.touch = touchActive, .idle = true}, | 
|  | 1735 | &consideredSignals) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1736 | .getModeId(); | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1737 | // Refresh rate will be chosen by either touch state or idle state | 
|  | 1738 | EXPECT_EQ(!touchActive, consideredSignals.idle); | 
|  | 1739 | return configId; | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1740 | }; | 
|  | 1741 |  | 
|  | 1742 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1743 | {HWC_CONFIG_ID_60, {Fps(60.f), Fps(90.f)}, {Fps(60.f), Fps(90.f)}}), | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1744 | 0); | 
|  | 1745 |  | 
|  | 1746 | // Idle should be lower priority than touch boost. | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1747 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::NoVote, /*touchActive=*/true)); | 
|  | 1748 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Min, /*touchActive=*/true)); | 
|  | 1749 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Max, /*touchActive=*/true)); | 
|  | 1750 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Heuristic, /*touchActive=*/true)); | 
|  | 1751 | EXPECT_EQ(HWC_CONFIG_ID_90, | 
|  | 1752 | getIdleFrameRate(LayerVoteType::ExplicitDefault, /*touchActive=*/true)); | 
|  | 1753 | EXPECT_EQ(HWC_CONFIG_ID_90, | 
|  | 1754 | getIdleFrameRate(LayerVoteType::ExplicitExactOrMultiple, /*touchActive=*/true)); | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1755 |  | 
|  | 1756 | // With no layers, idle should still be lower priority than touch boost. | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1757 | EXPECT_EQ(HWC_CONFIG_ID_90, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1758 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true, .idle = true}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1759 | .getModeId()); | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1760 |  | 
|  | 1761 | // Idle should be higher precedence than other layer frame rate considerations. | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1762 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1763 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::NoVote, /*touchActive=*/false)); | 
|  | 1764 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Min, /*touchActive=*/false)); | 
|  | 1765 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Max, /*touchActive=*/false)); | 
|  | 1766 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Heuristic, /*touchActive=*/false)); | 
|  | 1767 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
|  | 1768 | getIdleFrameRate(LayerVoteType::ExplicitDefault, /*touchActive=*/false)); | 
|  | 1769 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
|  | 1770 | getIdleFrameRate(LayerVoteType::ExplicitExactOrMultiple, /*touchActive=*/false)); | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1771 |  | 
|  | 1772 | // Idle should be applied rather than the current config when there are no layers. | 
|  | 1773 | EXPECT_EQ(HWC_CONFIG_ID_60, | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1774 | refreshRateConfigs->getBestRefreshRate({}, {.touch = false, .idle = true}) | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1775 | .getModeId()); | 
| Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1776 | } | 
|  | 1777 |  | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1778 | TEST_F(RefreshRateConfigsTest, findClosestKnownFrameRate) { | 
|  | 1779 | auto refreshRateConfigs = | 
|  | 1780 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1781 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1782 |  | 
|  | 1783 | for (float fps = 1.0f; fps <= 120.0f; fps += 0.1f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1784 | const auto knownFrameRate = findClosestKnownFrameRate(*refreshRateConfigs, Fps(fps)); | 
|  | 1785 | Fps expectedFrameRate; | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1786 | if (fps < 26.91f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1787 | expectedFrameRate = Fps(24.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1788 | } else if (fps < 37.51f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1789 | expectedFrameRate = Fps(30.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1790 | } else if (fps < 52.51f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1791 | expectedFrameRate = Fps(45.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1792 | } else if (fps < 66.01f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1793 | expectedFrameRate = Fps(60.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1794 | } else if (fps < 81.01f) { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1795 | expectedFrameRate = Fps(72.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1796 | } else { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1797 | expectedFrameRate = Fps(90.0f); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1798 | } | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1799 | EXPECT_TRUE(expectedFrameRate.equalsWithMargin(knownFrameRate)) | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1800 | << "findClosestKnownFrameRate(" << fps << ") = " << knownFrameRate; | 
|  | 1801 | } | 
|  | 1802 | } | 
|  | 1803 |  | 
|  | 1804 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_KnownFrameRate) { | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1805 | auto refreshRateConfigs = | 
|  | 1806 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 1807 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1808 |  | 
|  | 1809 | struct ExpectedRate { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1810 | Fps rate; | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1811 | const RefreshRate& expected; | 
|  | 1812 | }; | 
|  | 1813 |  | 
|  | 1814 | /* clang-format off */ | 
|  | 1815 | std::vector<ExpectedRate> knownFrameRatesExpectations = { | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1816 | {Fps(24.0f), mExpected60Config}, | 
|  | 1817 | {Fps(30.0f), mExpected60Config}, | 
|  | 1818 | {Fps(45.0f), mExpected90Config}, | 
|  | 1819 | {Fps(60.0f), mExpected60Config}, | 
|  | 1820 | {Fps(72.0f), mExpected90Config}, | 
|  | 1821 | {Fps(90.0f), mExpected90Config}, | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1822 | }; | 
|  | 1823 | /* clang-format on */ | 
|  | 1824 |  | 
|  | 1825 | // Make sure the test tests all the known frame rate | 
|  | 1826 | const auto knownFrameRateList = getKnownFrameRate(*refreshRateConfigs); | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1827 | const auto equal = | 
|  | 1828 | std::equal(knownFrameRateList.begin(), knownFrameRateList.end(), | 
|  | 1829 | knownFrameRatesExpectations.begin(), | 
|  | 1830 | [](Fps a, const ExpectedRate& b) { return a.equalsWithMargin(b.rate); }); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1831 | EXPECT_TRUE(equal); | 
|  | 1832 |  | 
|  | 1833 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 1834 | auto& layer = layers[0]; | 
|  | 1835 | layer.vote = LayerVoteType::Heuristic; | 
|  | 1836 | for (const auto& expectedRate : knownFrameRatesExpectations) { | 
|  | 1837 | layer.desiredRefreshRate = expectedRate.rate; | 
| Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1838 | const auto& refreshRate = | 
|  | 1839 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false}); | 
| Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1840 | EXPECT_EQ(expectedRate.expected, refreshRate); | 
|  | 1841 | } | 
|  | 1842 | } | 
|  | 1843 |  | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1844 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExact) { | 
|  | 1845 | auto refreshRateConfigs = | 
|  | 1846 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 1847 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1848 |  | 
|  | 1849 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1850 | LayerRequirement{.weight = 0.5f}}; | 
|  | 1851 | auto& explicitExactLayer = layers[0]; | 
|  | 1852 | auto& explicitExactOrMultipleLayer = layers[1]; | 
|  | 1853 |  | 
|  | 1854 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 1855 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; | 
|  | 1856 | explicitExactOrMultipleLayer.desiredRefreshRate = Fps(60); | 
|  | 1857 |  | 
|  | 1858 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; | 
|  | 1859 | explicitExactLayer.name = "ExplicitExact"; | 
|  | 1860 | explicitExactLayer.desiredRefreshRate = Fps(30); | 
|  | 1861 |  | 
|  | 1862 | EXPECT_EQ(mExpected30Config, | 
|  | 1863 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1864 | EXPECT_EQ(mExpected30Config, | 
|  | 1865 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
|  | 1866 |  | 
|  | 1867 | explicitExactOrMultipleLayer.desiredRefreshRate = Fps(120); | 
|  | 1868 | explicitExactLayer.desiredRefreshRate = Fps(60); | 
|  | 1869 | EXPECT_EQ(mExpected60Config, | 
|  | 1870 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1871 |  | 
|  | 1872 | explicitExactLayer.desiredRefreshRate = Fps(72); | 
|  | 1873 | EXPECT_EQ(mExpected72Config, | 
|  | 1874 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1875 |  | 
|  | 1876 | explicitExactLayer.desiredRefreshRate = Fps(90); | 
|  | 1877 | EXPECT_EQ(mExpected90Config, | 
|  | 1878 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1879 |  | 
|  | 1880 | explicitExactLayer.desiredRefreshRate = Fps(120); | 
|  | 1881 | EXPECT_EQ(mExpected120Config, | 
|  | 1882 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1883 | } | 
|  | 1884 |  | 
|  | 1885 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExactEnableFrameRateOverride) { | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1886 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1887 | auto refreshRateConfigs = | 
|  | 1888 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1889 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1890 |  | 
|  | 1891 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1892 | LayerRequirement{.weight = 0.5f}}; | 
|  | 1893 | auto& explicitExactLayer = layers[0]; | 
|  | 1894 | auto& explicitExactOrMultipleLayer = layers[1]; | 
|  | 1895 |  | 
|  | 1896 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 1897 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; | 
|  | 1898 | explicitExactOrMultipleLayer.desiredRefreshRate = Fps(60); | 
|  | 1899 |  | 
|  | 1900 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; | 
|  | 1901 | explicitExactLayer.name = "ExplicitExact"; | 
|  | 1902 | explicitExactLayer.desiredRefreshRate = Fps(30); | 
|  | 1903 |  | 
|  | 1904 | EXPECT_EQ(mExpected60Config, | 
|  | 1905 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1906 | EXPECT_EQ(mExpected120Config, | 
|  | 1907 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
|  | 1908 |  | 
|  | 1909 | explicitExactOrMultipleLayer.desiredRefreshRate = Fps(120); | 
|  | 1910 | explicitExactLayer.desiredRefreshRate = Fps(60); | 
|  | 1911 | EXPECT_EQ(mExpected120Config, | 
|  | 1912 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1913 |  | 
|  | 1914 | explicitExactLayer.desiredRefreshRate = Fps(72); | 
|  | 1915 | EXPECT_EQ(mExpected72Config, | 
|  | 1916 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1917 |  | 
|  | 1918 | explicitExactLayer.desiredRefreshRate = Fps(90); | 
|  | 1919 | EXPECT_EQ(mExpected90Config, | 
|  | 1920 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1921 |  | 
|  | 1922 | explicitExactLayer.desiredRefreshRate = Fps(120); | 
|  | 1923 | EXPECT_EQ(mExpected120Config, | 
|  | 1924 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 1925 | } | 
|  | 1926 |  | 
| Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1927 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ReadsCached) { | 
|  | 1928 | using GlobalSignals = RefreshRateConfigs::GlobalSignals; | 
|  | 1929 |  | 
|  | 1930 | auto refreshRateConfigs = | 
|  | 1931 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 1932 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1933 |  | 
|  | 1934 | setLastBestRefreshRateInvocation(*refreshRateConfigs, | 
|  | 1935 | GetBestRefreshRateInvocation{.layerRequirements = std::vector< | 
|  | 1936 | LayerRequirement>(), | 
|  | 1937 | .globalSignals = {.touch = true, | 
|  | 1938 | .idle = true}, | 
|  | 1939 | .outSignalsConsidered = | 
|  | 1940 | {.touch = true, | 
|  | 1941 | .idle = false}, | 
|  | 1942 | .resultingBestRefreshRate = | 
|  | 1943 | createRefreshRate( | 
|  | 1944 | mConfig90)}); | 
|  | 1945 |  | 
|  | 1946 | EXPECT_EQ(createRefreshRate(mConfig90), | 
|  | 1947 | refreshRateConfigs->getBestRefreshRate(std::vector<LayerRequirement>(), | 
|  | 1948 | {.touch = true, .idle = true})); | 
|  | 1949 |  | 
|  | 1950 | const GlobalSignals cachedSignalsConsidered{.touch = true, .idle = false}; | 
|  | 1951 | setLastBestRefreshRateInvocation(*refreshRateConfigs, | 
|  | 1952 | GetBestRefreshRateInvocation{.layerRequirements = std::vector< | 
|  | 1953 | LayerRequirement>(), | 
|  | 1954 | .globalSignals = {.touch = true, | 
|  | 1955 | .idle = true}, | 
|  | 1956 | .outSignalsConsidered = | 
|  | 1957 | cachedSignalsConsidered, | 
|  | 1958 | .resultingBestRefreshRate = | 
|  | 1959 | createRefreshRate( | 
|  | 1960 | mConfig30)}); | 
|  | 1961 |  | 
|  | 1962 | GlobalSignals signalsConsidered; | 
|  | 1963 | EXPECT_EQ(createRefreshRate(mConfig30), | 
|  | 1964 | refreshRateConfigs->getBestRefreshRate(std::vector<LayerRequirement>(), | 
|  | 1965 | {.touch = true, .idle = true}, | 
|  | 1966 | &signalsConsidered)); | 
|  | 1967 |  | 
|  | 1968 | EXPECT_EQ(cachedSignalsConsidered, signalsConsidered); | 
|  | 1969 | } | 
|  | 1970 |  | 
|  | 1971 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_WritesCache) { | 
|  | 1972 | using GlobalSignals = RefreshRateConfigs::GlobalSignals; | 
|  | 1973 |  | 
|  | 1974 | auto refreshRateConfigs = | 
|  | 1975 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 1976 | /*currentConfigId=*/HWC_CONFIG_ID_60); | 
|  | 1977 | ASSERT_FALSE(getLastBestRefreshRateInvocation(*refreshRateConfigs).has_value()); | 
|  | 1978 |  | 
|  | 1979 | GlobalSignals globalSignals{.touch = true, .idle = true}; | 
|  | 1980 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 1981 | LayerRequirement{.weight = 0.5f}}; | 
|  | 1982 | const auto lastResult = | 
|  | 1983 | refreshRateConfigs->getBestRefreshRate(layers, globalSignals, | 
|  | 1984 | /* outSignalsConsidered */ nullptr); | 
|  | 1985 |  | 
|  | 1986 | const auto lastInvocation = getLastBestRefreshRateInvocation(*refreshRateConfigs); | 
|  | 1987 |  | 
|  | 1988 | ASSERT_TRUE(lastInvocation.has_value()); | 
|  | 1989 | ASSERT_EQ(layers, lastInvocation->layerRequirements); | 
|  | 1990 | ASSERT_EQ(globalSignals, lastInvocation->globalSignals); | 
|  | 1991 | ASSERT_EQ(lastResult, lastInvocation->resultingBestRefreshRate); | 
|  | 1992 |  | 
|  | 1993 | // outSignalsConsidered needs to be populated even tho earlier we gave nullptr | 
|  | 1994 | // to getBestRefreshRate() | 
|  | 1995 | GlobalSignals detaultSignals; | 
|  | 1996 | ASSERT_FALSE(detaultSignals == lastInvocation->outSignalsConsidered); | 
|  | 1997 | } | 
|  | 1998 |  | 
| Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1999 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExactTouchBoost) { | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2000 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; | 
| Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 2001 | auto refreshRateConfigs = | 
|  | 2002 | std::make_unique<RefreshRateConfigs>(m60_120Device, | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2003 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); | 
| Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 2004 |  | 
|  | 2005 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}, | 
|  | 2006 | LayerRequirement{.weight = 0.5f}}; | 
|  | 2007 | auto& explicitExactLayer = layers[0]; | 
|  | 2008 | auto& explicitExactOrMultipleLayer = layers[1]; | 
|  | 2009 |  | 
|  | 2010 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 2011 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; | 
|  | 2012 | explicitExactOrMultipleLayer.desiredRefreshRate = Fps(60); | 
|  | 2013 |  | 
|  | 2014 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; | 
|  | 2015 | explicitExactLayer.name = "ExplicitExact"; | 
|  | 2016 | explicitExactLayer.desiredRefreshRate = Fps(30); | 
|  | 2017 |  | 
|  | 2018 | EXPECT_EQ(mExpected60Config, | 
|  | 2019 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 2020 | EXPECT_EQ(mExpected120Config, | 
|  | 2021 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
|  | 2022 |  | 
|  | 2023 | explicitExactOrMultipleLayer.vote = LayerVoteType::NoVote; | 
|  | 2024 |  | 
|  | 2025 | EXPECT_EQ(mExpected60Config, | 
|  | 2026 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = false, .idle = false})); | 
|  | 2027 | EXPECT_EQ(mExpected60Config, | 
|  | 2028 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = false})); | 
|  | 2029 | } | 
|  | 2030 |  | 
| Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2031 | TEST_F(RefreshRateConfigsTest, testComparisonOperator) { | 
|  | 2032 | EXPECT_TRUE(mExpected60Config < mExpected90Config); | 
|  | 2033 | EXPECT_FALSE(mExpected60Config < mExpected60Config); | 
|  | 2034 | EXPECT_FALSE(mExpected90Config < mExpected90Config); | 
|  | 2035 | } | 
|  | 2036 |  | 
|  | 2037 | TEST_F(RefreshRateConfigsTest, testKernelIdleTimerAction) { | 
|  | 2038 | using KernelIdleTimerAction = scheduler::RefreshRateConfigs::KernelIdleTimerAction; | 
|  | 2039 |  | 
|  | 2040 | auto refreshRateConfigs = | 
|  | 2041 | std::make_unique<RefreshRateConfigs>(m60_90Device, | 
|  | 2042 | /*currentConfigId=*/HWC_CONFIG_ID_90); | 
|  | 2043 | // SetPolicy(60, 90), current 90Hz => TurnOn. | 
|  | 2044 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2045 |  | 
|  | 2046 | // SetPolicy(60, 90), current 60Hz => TurnOn. | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 2047 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(90)}}), | 
|  | 2048 | 0); | 
| Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2049 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2050 |  | 
| Ady Abraham | a91605e | 2021-06-18 11:41:47 -0700 | [diff] [blame] | 2051 | // SetPolicy(60, 60), current 60Hz => TurnOff | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 2052 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(60)}}), | 
|  | 2053 | 0); | 
| Ady Abraham | a91605e | 2021-06-18 11:41:47 -0700 | [diff] [blame] | 2054 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); | 
| Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2055 |  | 
|  | 2056 | // SetPolicy(90, 90), current 90Hz => TurnOff. | 
| Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 2057 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {Fps(90), Fps(90)}}), | 
|  | 2058 | 0); | 
| Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2059 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2060 | } | 
|  | 2061 |  | 
| TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2062 | TEST_F(RefreshRateConfigsTest, testKernelIdleTimerActionFor120Hz) { | 
|  | 2063 | using KernelIdleTimerAction = scheduler::RefreshRateConfigs::KernelIdleTimerAction; | 
|  | 2064 |  | 
|  | 2065 | // Tests with 120Hz | 
|  | 2066 | auto refreshRateConfigs = | 
|  | 2067 | std::make_unique<RefreshRateConfigs>(m60_120Device, | 
|  | 2068 | /*currentConfigId=*/HWC_CONFIG_ID_120); | 
|  | 2069 | // SetPolicy(0, 60), current 60Hz => TurnOn. | 
|  | 2070 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(0), Fps(60)}}), | 
|  | 2071 | 0); | 
|  | 2072 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2073 |  | 
|  | 2074 | // SetPolicy(60, 60), current 60Hz => TurnOff. | 
|  | 2075 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(60)}}), | 
|  | 2076 | 0); | 
|  | 2077 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2078 |  | 
|  | 2079 | // SetPolicy(60, 120), current 60Hz => TurnOn. | 
|  | 2080 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {Fps(60), Fps(120)}}), | 
|  | 2081 | 0); | 
|  | 2082 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2083 |  | 
|  | 2084 | // SetPolicy(120, 120), current 120Hz => TurnOff. | 
|  | 2085 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( | 
|  | 2086 | {HWC_CONFIG_ID_120, {Fps(120), Fps(120)}}), | 
|  | 2087 | 0); | 
|  | 2088 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); | 
|  | 2089 | } | 
|  | 2090 |  | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2091 | TEST_F(RefreshRateConfigsTest, getFrameRateDivider) { | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2092 | auto refreshRateConfigs = | 
|  | 2093 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, | 
|  | 2094 | /*currentConfigId=*/HWC_CONFIG_ID_30); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2095 |  | 
|  | 2096 | const auto frameRate = Fps(30.f); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2097 | Fps displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2098 | EXPECT_EQ(1, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2099 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2100 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_60); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2101 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2102 | EXPECT_EQ(2, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2103 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2104 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_72); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2105 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2106 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2107 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2108 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2109 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2110 | EXPECT_EQ(3, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2111 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2112 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_120); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2113 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2114 | EXPECT_EQ(4, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); | 
| Ady Abraham | 62f216c | 2020-10-13 19:07:23 -0700 | [diff] [blame] | 2115 |  | 
| Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2116 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); | 
| Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2117 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); | 
|  | 2118 | EXPECT_EQ(4, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, Fps(22.5f))); | 
|  | 2119 | EXPECT_EQ(4, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, Fps(22.6f))); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2120 | } | 
|  | 2121 |  | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2122 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_noLayers) { | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2123 | auto refreshRateConfigs = | 
|  | 2124 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ | 
|  | 2125 | HWC_CONFIG_ID_120); | 
|  | 2126 |  | 
|  | 2127 | auto layers = std::vector<LayerRequirement>{}; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2128 | ASSERT_TRUE(refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false) | 
|  | 2129 | .empty()); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2130 | } | 
|  | 2131 |  | 
|  | 2132 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_60on120) { | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2133 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2134 | auto refreshRateConfigs = | 
|  | 2135 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2136 | HWC_CONFIG_ID_120, config); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2137 |  | 
|  | 2138 | auto layers = std::vector<LayerRequirement>{LayerRequirement{.weight = 1.0f}}; | 
|  | 2139 | layers[0].name = "Test layer"; | 
|  | 2140 | layers[0].ownerUid = 1234; | 
|  | 2141 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 2142 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2143 | auto frameRateOverrides = | 
|  | 2144 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2145 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2146 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2147 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2148 |  | 
|  | 2149 | layers[0].vote = LayerVoteType::ExplicitExactOrMultiple; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2150 | frameRateOverrides = | 
|  | 2151 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2152 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2153 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2154 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2155 |  | 
|  | 2156 | layers[0].vote = LayerVoteType::NoVote; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2157 | frameRateOverrides = | 
|  | 2158 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2159 | ASSERT_TRUE(frameRateOverrides.empty()); | 
|  | 2160 |  | 
|  | 2161 | layers[0].vote = LayerVoteType::Min; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2162 | frameRateOverrides = | 
|  | 2163 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2164 | ASSERT_TRUE(frameRateOverrides.empty()); | 
|  | 2165 |  | 
|  | 2166 | layers[0].vote = LayerVoteType::Max; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2167 | frameRateOverrides = | 
|  | 2168 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2169 | ASSERT_TRUE(frameRateOverrides.empty()); | 
|  | 2170 |  | 
|  | 2171 | layers[0].vote = LayerVoteType::Heuristic; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2172 | frameRateOverrides = | 
|  | 2173 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2174 | ASSERT_TRUE(frameRateOverrides.empty()); | 
|  | 2175 | } | 
|  | 2176 |  | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2177 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_twoUids) { | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2178 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2179 | auto refreshRateConfigs = | 
|  | 2180 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2181 | HWC_CONFIG_ID_120, config); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2182 |  | 
|  | 2183 | auto layers = std::vector<LayerRequirement>{ | 
|  | 2184 | LayerRequirement{.ownerUid = 1234, .weight = 1.0f}, | 
|  | 2185 | LayerRequirement{.ownerUid = 5678, .weight = 1.0f}, | 
|  | 2186 | }; | 
|  | 2187 |  | 
|  | 2188 | layers[0].name = "Test layer 1234"; | 
|  | 2189 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 2190 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
|  | 2191 |  | 
|  | 2192 | layers[1].name = "Test layer 5678"; | 
|  | 2193 | layers[1].desiredRefreshRate = Fps(30.0f); | 
|  | 2194 | layers[1].vote = LayerVoteType::ExplicitDefault; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2195 | auto frameRateOverrides = | 
|  | 2196 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2197 |  | 
|  | 2198 | ASSERT_EQ(2, frameRateOverrides.size()); | 
|  | 2199 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2200 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2201 | ASSERT_EQ(1, frameRateOverrides.count(5678)); | 
|  | 2202 | ASSERT_EQ(30.0f, frameRateOverrides.at(5678).getValue()); | 
|  | 2203 |  | 
|  | 2204 | layers[1].vote = LayerVoteType::Heuristic; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2205 | frameRateOverrides = | 
|  | 2206 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2207 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2208 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2209 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2210 |  | 
|  | 2211 | layers[1].ownerUid = 1234; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2212 | frameRateOverrides = | 
|  | 2213 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
|  | 2214 | ASSERT_TRUE(frameRateOverrides.empty()); | 
|  | 2215 | } | 
|  | 2216 |  | 
|  | 2217 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_touch) { | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2218 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2219 | auto refreshRateConfigs = | 
|  | 2220 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ | 
| rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2221 | HWC_CONFIG_ID_120, config); | 
| Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2222 |  | 
|  | 2223 | auto layers = std::vector<LayerRequirement>{ | 
|  | 2224 | LayerRequirement{.ownerUid = 1234, .weight = 1.0f}, | 
|  | 2225 | }; | 
|  | 2226 |  | 
|  | 2227 | layers[0].name = "Test layer"; | 
|  | 2228 | layers[0].desiredRefreshRate = Fps(60.0f); | 
|  | 2229 | layers[0].vote = LayerVoteType::ExplicitDefault; | 
|  | 2230 |  | 
|  | 2231 | auto frameRateOverrides = | 
|  | 2232 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
|  | 2233 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2234 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2235 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2236 |  | 
|  | 2237 | frameRateOverrides = | 
|  | 2238 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/true); | 
|  | 2239 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2240 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2241 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2242 |  | 
|  | 2243 | layers[0].vote = LayerVoteType::ExplicitExact; | 
|  | 2244 | frameRateOverrides = | 
|  | 2245 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
|  | 2246 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2247 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2248 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2249 |  | 
|  | 2250 | frameRateOverrides = | 
|  | 2251 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/true); | 
|  | 2252 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2253 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2254 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2255 |  | 
|  | 2256 | layers[0].vote = LayerVoteType::ExplicitExactOrMultiple; | 
|  | 2257 | frameRateOverrides = | 
|  | 2258 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/false); | 
|  | 2259 | ASSERT_EQ(1, frameRateOverrides.size()); | 
|  | 2260 | ASSERT_EQ(1, frameRateOverrides.count(1234)); | 
|  | 2261 | ASSERT_EQ(60.0f, frameRateOverrides.at(1234).getValue()); | 
|  | 2262 |  | 
|  | 2263 | frameRateOverrides = | 
|  | 2264 | refreshRateConfigs->getFrameRateOverrides(layers, Fps(120.0f), /*touch=*/true); | 
| Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2265 | ASSERT_TRUE(frameRateOverrides.empty()); | 
| Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2266 | } | 
|  | 2267 |  | 
| Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 2268 | } // namespace | 
|  | 2269 | } // namespace scheduler | 
|  | 2270 | } // namespace android | 
| Marin Shalamanov | bed7fd3 | 2020-12-21 20:02:20 +0100 | [diff] [blame] | 2271 |  | 
|  | 2272 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 2273 | #pragma clang diagnostic pop // ignored "-Wextra" |