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