| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -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 |  | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 17 | #include <cmath> | 
 | 18 |  | 
| Lloyd Pique | 3d0c02e | 2018-10-19 18:38:12 -0700 | [diff] [blame] | 19 | #include <compositionengine/DisplayColorProfileCreationArgs.h> | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 20 | #include <compositionengine/DisplayCreationArgs.h> | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 21 | #include <compositionengine/DisplaySurface.h> | 
 | 22 | #include <compositionengine/RenderSurfaceCreationArgs.h> | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 23 | #include <compositionengine/impl/Display.h> | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 24 | #include <compositionengine/impl/RenderSurface.h> | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 25 | #include <compositionengine/mock/CompositionEngine.h> | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 26 | #include <compositionengine/mock/DisplayColorProfile.h> | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 27 | #include <compositionengine/mock/DisplaySurface.h> | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 28 | #include <compositionengine/mock/LayerFE.h> | 
| chaviw | 8beb414 | 2019-04-11 13:09:05 -0700 | [diff] [blame] | 29 | #include <compositionengine/mock/NativeWindow.h> | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 30 | #include <compositionengine/mock/OutputLayer.h> | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 31 | #include <compositionengine/mock/RenderSurface.h> | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 32 | #include <gtest/gtest.h> | 
| Lloyd Pique | e9eff97 | 2020-05-05 12:36:44 -0700 | [diff] [blame] | 33 | #include <renderengine/mock/RenderEngine.h> | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 34 | #include <ui/Rect.h> | 
| Marin Shalamanov | 228f46b | 2021-01-28 21:11:45 +0100 | [diff] [blame] | 35 | #include <ui/StaticDisplayInfo.h> | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 36 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 37 | #include "MockHWC2.h" | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 38 | #include "MockHWComposer.h" | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 39 | #include "MockPowerAdvisor.h" | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 40 |  | 
 | 41 | namespace android::compositionengine { | 
 | 42 | namespace { | 
 | 43 |  | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 44 | namespace hal = android::hardware::graphics::composer::hal; | 
 | 45 |  | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 46 | using testing::_; | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 47 | using testing::DoAll; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 48 | using testing::Eq; | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 49 | using testing::InSequence; | 
 | 50 | using testing::NiceMock; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 51 | using testing::Pointee; | 
 | 52 | using testing::Ref; | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 53 | using testing::Return; | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 54 | using testing::ReturnRef; | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 55 | using testing::Sequence; | 
 | 56 | using testing::SetArgPointee; | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 57 | using testing::StrictMock; | 
 | 58 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 59 | constexpr PhysicalDisplayId DEFAULT_DISPLAY_ID = PhysicalDisplayId::fromPort(123u); | 
 | 60 | constexpr HalVirtualDisplayId HAL_VIRTUAL_DISPLAY_ID{456u}; | 
 | 61 | constexpr GpuVirtualDisplayId GPU_VIRTUAL_DISPLAY_ID{789u}; | 
 | 62 |  | 
 | 63 | const ui::Size DEFAULT_RESOLUTION{1920, 1080}; | 
 | 64 | constexpr uint32_t DEFAULT_LAYER_STACK = 42; | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 65 |  | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 66 | struct Layer { | 
 | 67 |     Layer() { | 
 | 68 |         EXPECT_CALL(*outputLayer, getLayerFE()).WillRepeatedly(ReturnRef(*layerFE)); | 
 | 69 |         EXPECT_CALL(*outputLayer, getHwcLayer()).WillRepeatedly(Return(&hwc2Layer)); | 
 | 70 |     } | 
 | 71 |  | 
 | 72 |     sp<mock::LayerFE> layerFE = new StrictMock<mock::LayerFE>(); | 
 | 73 |     StrictMock<mock::OutputLayer>* outputLayer = new StrictMock<mock::OutputLayer>(); | 
 | 74 |     StrictMock<HWC2::mock::Layer> hwc2Layer; | 
 | 75 | }; | 
 | 76 |  | 
 | 77 | struct LayerNoHWC2Layer { | 
 | 78 |     LayerNoHWC2Layer() { | 
 | 79 |         EXPECT_CALL(*outputLayer, getLayerFE()).WillRepeatedly(ReturnRef(*layerFE)); | 
 | 80 |         EXPECT_CALL(*outputLayer, getHwcLayer()).WillRepeatedly(Return(nullptr)); | 
 | 81 |     } | 
 | 82 |  | 
 | 83 |     sp<mock::LayerFE> layerFE = new StrictMock<mock::LayerFE>(); | 
 | 84 |     StrictMock<mock::OutputLayer>* outputLayer = new StrictMock<mock::OutputLayer>(); | 
 | 85 | }; | 
 | 86 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 87 | struct DisplayTestCommon : public testing::Test { | 
 | 88 |     // Uses the full implementation of a display | 
 | 89 |     class FullImplDisplay : public impl::Display { | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 90 |     public: | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 91 |         using impl::Display::injectOutputLayerForTest; | 
 | 92 |         virtual void injectOutputLayerForTest(std::unique_ptr<compositionengine::OutputLayer>) = 0; | 
 | 93 |     }; | 
 | 94 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 95 |     // Uses a special implementation with key internal member functions set up | 
 | 96 |     // as mock implementations, to allow for easier testing. | 
 | 97 |     struct PartialMockDisplay : public impl::Display { | 
 | 98 |         PartialMockDisplay(const compositionengine::CompositionEngine& compositionEngine) | 
 | 99 |               : mCompositionEngine(compositionEngine) {} | 
 | 100 |  | 
 | 101 |         // compositionengine::Output overrides | 
 | 102 |         const OutputCompositionState& getState() const override { return mState; } | 
 | 103 |         OutputCompositionState& editState() override { return mState; } | 
 | 104 |  | 
 | 105 |         // compositionengine::impl::Output overrides | 
 | 106 |         const CompositionEngine& getCompositionEngine() const override { | 
 | 107 |             return mCompositionEngine; | 
 | 108 |         }; | 
 | 109 |  | 
 | 110 |         // Mock implementation overrides | 
 | 111 |         MOCK_CONST_METHOD0(getOutputLayerCount, size_t()); | 
 | 112 |         MOCK_CONST_METHOD1(getOutputLayerOrderedByZByIndex, | 
 | 113 |                            compositionengine::OutputLayer*(size_t)); | 
 | 114 |         MOCK_METHOD2(ensureOutputLayer, | 
 | 115 |                      compositionengine::OutputLayer*(std::optional<size_t>, const sp<LayerFE>&)); | 
 | 116 |         MOCK_METHOD0(finalizePendingOutputLayers, void()); | 
 | 117 |         MOCK_METHOD0(clearOutputLayers, void()); | 
 | 118 |         MOCK_CONST_METHOD1(dumpState, void(std::string&)); | 
 | 119 |         MOCK_METHOD1(injectOutputLayerForTest, compositionengine::OutputLayer*(const sp<LayerFE>&)); | 
 | 120 |         MOCK_METHOD1(injectOutputLayerForTest, void(std::unique_ptr<OutputLayer>)); | 
 | 121 |         MOCK_CONST_METHOD0(anyLayersRequireClientComposition, bool()); | 
 | 122 |         MOCK_CONST_METHOD0(allLayersRequireClientComposition, bool()); | 
 | 123 |         MOCK_METHOD1(applyChangedTypesToLayers, void(const impl::Display::ChangedTypes&)); | 
 | 124 |         MOCK_METHOD1(applyDisplayRequests, void(const impl::Display::DisplayRequests&)); | 
 | 125 |         MOCK_METHOD1(applyLayerRequestsToLayers, void(const impl::Display::LayerRequests&)); | 
 | 126 |  | 
 | 127 |         const compositionengine::CompositionEngine& mCompositionEngine; | 
 | 128 |         impl::OutputCompositionState mState; | 
 | 129 |     }; | 
 | 130 |  | 
 | 131 |     static std::string getDisplayNameFromCurrentTest() { | 
 | 132 |         const ::testing::TestInfo* const test_info = | 
 | 133 |                 ::testing::UnitTest::GetInstance()->current_test_info(); | 
 | 134 |         return std::string("display for ") + test_info->test_case_name() + "." + test_info->name(); | 
 | 135 |     } | 
 | 136 |  | 
 | 137 |     template <typename Display> | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 138 |     static std::shared_ptr<Display> createDisplay( | 
 | 139 |             const compositionengine::CompositionEngine& compositionEngine, | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 140 |             compositionengine::DisplayCreationArgs args) { | 
 | 141 |         args.name = getDisplayNameFromCurrentTest(); | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 142 |         return impl::createDisplayTemplated<Display>(compositionEngine, args); | 
 | 143 |     } | 
 | 144 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 145 |     template <typename Display> | 
 | 146 |     static std::shared_ptr<StrictMock<Display>> createPartialMockDisplay( | 
 | 147 |             const compositionengine::CompositionEngine& compositionEngine, | 
 | 148 |             compositionengine::DisplayCreationArgs args) { | 
 | 149 |         args.name = getDisplayNameFromCurrentTest(); | 
 | 150 |         auto display = std::make_shared<StrictMock<Display>>(compositionEngine); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 151 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 152 |         display->setConfiguration(args); | 
 | 153 |  | 
 | 154 |         return display; | 
 | 155 |     } | 
 | 156 |  | 
 | 157 |     DisplayTestCommon() { | 
 | 158 |         EXPECT_CALL(mCompositionEngine, getHwComposer()).WillRepeatedly(ReturnRef(mHwComposer)); | 
| Lloyd Pique | e9eff97 | 2020-05-05 12:36:44 -0700 | [diff] [blame] | 159 |         EXPECT_CALL(mCompositionEngine, getRenderEngine()).WillRepeatedly(ReturnRef(mRenderEngine)); | 
 | 160 |         EXPECT_CALL(mRenderEngine, supportsProtectedContent()).WillRepeatedly(Return(false)); | 
| Peiyong Lin | 09f910f | 2020-09-25 10:54:13 -0700 | [diff] [blame] | 161 |         EXPECT_CALL(mRenderEngine, isProtected()).WillRepeatedly(Return(false)); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 162 |     } | 
 | 163 |  | 
 | 164 |     DisplayCreationArgs getDisplayCreationArgsForPhysicalHWCDisplay() { | 
 | 165 |         return DisplayCreationArgsBuilder() | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 166 |                 .setId(DEFAULT_DISPLAY_ID) | 
 | 167 |                 .setConnectionType(ui::DisplayConnectionType::Internal) | 
 | 168 |                 .setPixels(DEFAULT_RESOLUTION) | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 169 |                 .setIsSecure(true) | 
 | 170 |                 .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 171 |                 .setPowerAdvisor(&mPowerAdvisor) | 
 | 172 |                 .build(); | 
 | 173 |     } | 
 | 174 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 175 |     DisplayCreationArgs getDisplayCreationArgsForGpuVirtualDisplay() { | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 176 |         return DisplayCreationArgsBuilder() | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 177 |                 .setId(GPU_VIRTUAL_DISPLAY_ID) | 
 | 178 |                 .setPixels(DEFAULT_RESOLUTION) | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 179 |                 .setIsSecure(false) | 
 | 180 |                 .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 181 |                 .setPowerAdvisor(&mPowerAdvisor) | 
 | 182 |                 .build(); | 
 | 183 |     } | 
 | 184 |  | 
 | 185 |     StrictMock<android::mock::HWComposer> mHwComposer; | 
 | 186 |     StrictMock<Hwc2::mock::PowerAdvisor> mPowerAdvisor; | 
| Lloyd Pique | e9eff97 | 2020-05-05 12:36:44 -0700 | [diff] [blame] | 187 |     StrictMock<renderengine::mock::RenderEngine> mRenderEngine; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 188 |     StrictMock<mock::CompositionEngine> mCompositionEngine; | 
 | 189 |     sp<mock::NativeWindow> mNativeWindow = new StrictMock<mock::NativeWindow>(); | 
 | 190 | }; | 
 | 191 |  | 
 | 192 | struct PartialMockDisplayTestCommon : public DisplayTestCommon { | 
 | 193 |     using Display = DisplayTestCommon::PartialMockDisplay; | 
 | 194 |     std::shared_ptr<Display> mDisplay = | 
 | 195 |             createPartialMockDisplay<Display>(mCompositionEngine, | 
 | 196 |                                               getDisplayCreationArgsForPhysicalHWCDisplay()); | 
 | 197 | }; | 
 | 198 |  | 
 | 199 | struct FullDisplayImplTestCommon : public DisplayTestCommon { | 
 | 200 |     using Display = DisplayTestCommon::FullImplDisplay; | 
 | 201 |     std::shared_ptr<Display> mDisplay = | 
 | 202 |             createDisplay<Display>(mCompositionEngine, | 
 | 203 |                                    getDisplayCreationArgsForPhysicalHWCDisplay()); | 
 | 204 | }; | 
 | 205 |  | 
 | 206 | struct DisplayWithLayersTestCommon : public FullDisplayImplTestCommon { | 
 | 207 |     DisplayWithLayersTestCommon() { | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 208 |         mDisplay->injectOutputLayerForTest( | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 209 |                 std::unique_ptr<compositionengine::OutputLayer>(mLayer1.outputLayer)); | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 210 |         mDisplay->injectOutputLayerForTest( | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 211 |                 std::unique_ptr<compositionengine::OutputLayer>(mLayer2.outputLayer)); | 
| Lloyd Pique | 01c77c1 | 2019-04-17 12:48:32 -0700 | [diff] [blame] | 212 |         mDisplay->injectOutputLayerForTest( | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 213 |                 std::unique_ptr<compositionengine::OutputLayer>(mLayer3.outputLayer)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 214 |     } | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 215 |  | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 216 |     Layer mLayer1; | 
 | 217 |     Layer mLayer2; | 
 | 218 |     LayerNoHWC2Layer mLayer3; | 
 | 219 |     StrictMock<HWC2::mock::Layer> hwc2LayerUnknown; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 220 |     std::shared_ptr<Display> mDisplay = | 
 | 221 |             createDisplay<Display>(mCompositionEngine, | 
 | 222 |                                    getDisplayCreationArgsForPhysicalHWCDisplay()); | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 223 | }; | 
 | 224 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 225 | /* | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 226 |  * Basic construction | 
 | 227 |  */ | 
 | 228 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 229 | struct DisplayCreationTest : public DisplayTestCommon { | 
 | 230 |     using Display = DisplayTestCommon::FullImplDisplay; | 
 | 231 | }; | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 232 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 233 | TEST_F(DisplayCreationTest, createPhysicalInternalDisplay) { | 
 | 234 |     auto display = | 
 | 235 |             impl::createDisplay(mCompositionEngine, getDisplayCreationArgsForPhysicalHWCDisplay()); | 
 | 236 |     EXPECT_TRUE(display->isSecure()); | 
 | 237 |     EXPECT_FALSE(display->isVirtual()); | 
 | 238 |     EXPECT_EQ(DEFAULT_DISPLAY_ID, display->getId()); | 
 | 239 | } | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 240 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 241 | TEST_F(DisplayCreationTest, createGpuVirtualDisplay) { | 
 | 242 |     auto display = | 
 | 243 |             impl::createDisplay(mCompositionEngine, getDisplayCreationArgsForGpuVirtualDisplay()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 244 |     EXPECT_FALSE(display->isSecure()); | 
 | 245 |     EXPECT_TRUE(display->isVirtual()); | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 246 |     EXPECT_TRUE(GpuVirtualDisplayId::tryCast(display->getId())); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 247 | } | 
 | 248 |  | 
 | 249 | /* | 
 | 250 |  * Display::setConfiguration() | 
 | 251 |  */ | 
 | 252 |  | 
 | 253 | using DisplaySetConfigurationTest = PartialMockDisplayTestCommon; | 
 | 254 |  | 
 | 255 | TEST_F(DisplaySetConfigurationTest, configuresInternalSecurePhysicalDisplay) { | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 256 |     mDisplay->setConfiguration(DisplayCreationArgsBuilder() | 
 | 257 |                                        .setId(DEFAULT_DISPLAY_ID) | 
 | 258 |                                        .setConnectionType(ui::DisplayConnectionType::Internal) | 
 | 259 |                                        .setPixels(DEFAULT_RESOLUTION) | 
 | 260 |                                        .setIsSecure(true) | 
 | 261 |                                        .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 262 |                                        .setPowerAdvisor(&mPowerAdvisor) | 
 | 263 |                                        .setName(getDisplayNameFromCurrentTest()) | 
 | 264 |                                        .build()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 265 |  | 
 | 266 |     EXPECT_EQ(DEFAULT_DISPLAY_ID, mDisplay->getId()); | 
 | 267 |     EXPECT_TRUE(mDisplay->isSecure()); | 
 | 268 |     EXPECT_FALSE(mDisplay->isVirtual()); | 
 | 269 |     EXPECT_EQ(DEFAULT_LAYER_STACK, mDisplay->getState().layerStackId); | 
 | 270 |     EXPECT_TRUE(mDisplay->getState().layerStackInternal); | 
 | 271 |     EXPECT_FALSE(mDisplay->isValid()); | 
 | 272 | } | 
 | 273 |  | 
 | 274 | TEST_F(DisplaySetConfigurationTest, configuresExternalInsecurePhysicalDisplay) { | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 275 |     mDisplay->setConfiguration(DisplayCreationArgsBuilder() | 
 | 276 |                                        .setId(DEFAULT_DISPLAY_ID) | 
 | 277 |                                        .setConnectionType(ui::DisplayConnectionType::External) | 
 | 278 |                                        .setPixels(DEFAULT_RESOLUTION) | 
 | 279 |                                        .setIsSecure(false) | 
 | 280 |                                        .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 281 |                                        .setPowerAdvisor(&mPowerAdvisor) | 
 | 282 |                                        .setName(getDisplayNameFromCurrentTest()) | 
 | 283 |                                        .build()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 284 |  | 
 | 285 |     EXPECT_EQ(DEFAULT_DISPLAY_ID, mDisplay->getId()); | 
 | 286 |     EXPECT_FALSE(mDisplay->isSecure()); | 
 | 287 |     EXPECT_FALSE(mDisplay->isVirtual()); | 
 | 288 |     EXPECT_EQ(DEFAULT_LAYER_STACK, mDisplay->getState().layerStackId); | 
 | 289 |     EXPECT_FALSE(mDisplay->getState().layerStackInternal); | 
 | 290 |     EXPECT_FALSE(mDisplay->isValid()); | 
 | 291 | } | 
 | 292 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 293 | TEST_F(DisplaySetConfigurationTest, configuresHalVirtualDisplay) { | 
 | 294 |     mDisplay->setConfiguration(DisplayCreationArgsBuilder() | 
 | 295 |                                        .setId(HAL_VIRTUAL_DISPLAY_ID) | 
 | 296 |                                        .setPixels(DEFAULT_RESOLUTION) | 
 | 297 |                                        .setIsSecure(false) | 
 | 298 |                                        .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 299 |                                        .setPowerAdvisor(&mPowerAdvisor) | 
 | 300 |                                        .setName(getDisplayNameFromCurrentTest()) | 
 | 301 |                                        .build()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 302 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 303 |     EXPECT_EQ(HAL_VIRTUAL_DISPLAY_ID, mDisplay->getId()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 304 |     EXPECT_FALSE(mDisplay->isSecure()); | 
 | 305 |     EXPECT_TRUE(mDisplay->isVirtual()); | 
 | 306 |     EXPECT_EQ(DEFAULT_LAYER_STACK, mDisplay->getState().layerStackId); | 
 | 307 |     EXPECT_FALSE(mDisplay->getState().layerStackInternal); | 
 | 308 |     EXPECT_FALSE(mDisplay->isValid()); | 
 | 309 | } | 
 | 310 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 311 | TEST_F(DisplaySetConfigurationTest, configuresGpuVirtualDisplay) { | 
 | 312 |     mDisplay->setConfiguration(DisplayCreationArgsBuilder() | 
 | 313 |                                        .setId(GPU_VIRTUAL_DISPLAY_ID) | 
 | 314 |                                        .setPixels(DEFAULT_RESOLUTION) | 
 | 315 |                                        .setIsSecure(false) | 
 | 316 |                                        .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 317 |                                        .setPowerAdvisor(&mPowerAdvisor) | 
 | 318 |                                        .setName(getDisplayNameFromCurrentTest()) | 
 | 319 |                                        .build()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 320 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 321 |     EXPECT_EQ(GPU_VIRTUAL_DISPLAY_ID, mDisplay->getId()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 322 |     EXPECT_FALSE(mDisplay->isSecure()); | 
 | 323 |     EXPECT_TRUE(mDisplay->isVirtual()); | 
 | 324 |     EXPECT_EQ(DEFAULT_LAYER_STACK, mDisplay->getState().layerStackId); | 
 | 325 |     EXPECT_FALSE(mDisplay->getState().layerStackInternal); | 
 | 326 |     EXPECT_FALSE(mDisplay->isValid()); | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 327 | } | 
 | 328 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 329 | /* | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 330 |  * Display::disconnect() | 
 | 331 |  */ | 
 | 332 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 333 | using DisplayDisconnectTest = PartialMockDisplayTestCommon; | 
 | 334 |  | 
 | 335 | TEST_F(DisplayDisconnectTest, disconnectsDisplay) { | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 336 |     // The first call to disconnect will disconnect the display with the HWC. | 
 | 337 |     EXPECT_CALL(mHwComposer, disconnectDisplay(HalDisplayId(DEFAULT_DISPLAY_ID))).Times(1); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 338 |     mDisplay->disconnect(); | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 339 |  | 
 | 340 |     // Subsequent calls will do nothing, | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 341 |     EXPECT_CALL(mHwComposer, disconnectDisplay(HalDisplayId(DEFAULT_DISPLAY_ID))).Times(0); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 342 |     mDisplay->disconnect(); | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 343 | } | 
 | 344 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 345 | /* | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 346 |  * Display::setColorTransform() | 
 | 347 |  */ | 
 | 348 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 349 | using DisplaySetColorTransformTest = PartialMockDisplayTestCommon; | 
 | 350 |  | 
 | 351 | TEST_F(DisplaySetColorTransformTest, setsTransform) { | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 352 |     // No change does nothing | 
 | 353 |     CompositionRefreshArgs refreshArgs; | 
 | 354 |     refreshArgs.colorTransformMatrix = std::nullopt; | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 355 |     mDisplay->setColorTransform(refreshArgs); | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 356 |  | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 357 |     // Identity matrix sets an identity state value | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 358 |     const mat4 kIdentity; | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 359 |  | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 360 |     EXPECT_CALL(mHwComposer, setColorTransform(HalDisplayId(DEFAULT_DISPLAY_ID), kIdentity)) | 
 | 361 |             .Times(1); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 362 |  | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 363 |     refreshArgs.colorTransformMatrix = kIdentity; | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 364 |     mDisplay->setColorTransform(refreshArgs); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 365 |  | 
 | 366 |     // Non-identity matrix sets a non-identity state value | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 367 |     const mat4 kNonIdentity = mat4() * 2; | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 368 |  | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 369 |     EXPECT_CALL(mHwComposer, setColorTransform(HalDisplayId(DEFAULT_DISPLAY_ID), kNonIdentity)) | 
 | 370 |             .Times(1); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 371 |  | 
| Lloyd Pique | 3eb1b21 | 2019-03-07 21:15:40 -0800 | [diff] [blame] | 372 |     refreshArgs.colorTransformMatrix = kNonIdentity; | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 373 |     mDisplay->setColorTransform(refreshArgs); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 374 | } | 
 | 375 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 376 | /* | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 377 |  * Display::setColorMode() | 
 | 378 |  */ | 
 | 379 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 380 | using DisplaySetColorModeTest = PartialMockDisplayTestCommon; | 
 | 381 |  | 
 | 382 | TEST_F(DisplaySetColorModeTest, setsModeUnlessNoChange) { | 
| Lloyd Pique | 6a3b446 | 2019-03-07 20:58:12 -0800 | [diff] [blame] | 383 |     using ColorProfile = Output::ColorProfile; | 
 | 384 |  | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 385 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 386 |     mDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 387 |     mock::DisplayColorProfile* colorProfile = new StrictMock<mock::DisplayColorProfile>(); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 388 |     mDisplay->setDisplayColorProfileForTest(std::unique_ptr<DisplayColorProfile>(colorProfile)); | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 389 |  | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 390 |     EXPECT_CALL(*colorProfile, getTargetDataspace(_, _, _)) | 
 | 391 |             .WillRepeatedly(Return(ui::Dataspace::UNKNOWN)); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 392 |  | 
 | 393 |     // These values are expected to be the initial state. | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 394 |     ASSERT_EQ(ui::ColorMode::NATIVE, mDisplay->getState().colorMode); | 
 | 395 |     ASSERT_EQ(ui::Dataspace::UNKNOWN, mDisplay->getState().dataspace); | 
 | 396 |     ASSERT_EQ(ui::RenderIntent::COLORIMETRIC, mDisplay->getState().renderIntent); | 
 | 397 |     ASSERT_EQ(ui::Dataspace::UNKNOWN, mDisplay->getState().targetDataspace); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 398 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 399 |     // Otherwise if the values are unchanged, nothing happens | 
 | 400 |     mDisplay->setColorProfile(ColorProfile{ui::ColorMode::NATIVE, ui::Dataspace::UNKNOWN, | 
 | 401 |                                            ui::RenderIntent::COLORIMETRIC, ui::Dataspace::UNKNOWN}); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 402 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 403 |     EXPECT_EQ(ui::ColorMode::NATIVE, mDisplay->getState().colorMode); | 
 | 404 |     EXPECT_EQ(ui::Dataspace::UNKNOWN, mDisplay->getState().dataspace); | 
 | 405 |     EXPECT_EQ(ui::RenderIntent::COLORIMETRIC, mDisplay->getState().renderIntent); | 
 | 406 |     EXPECT_EQ(ui::Dataspace::UNKNOWN, mDisplay->getState().targetDataspace); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 407 |  | 
 | 408 |     // Otherwise if the values are different, updates happen | 
| Lloyd Pique | ef95812 | 2019-02-05 18:00:12 -0800 | [diff] [blame] | 409 |     EXPECT_CALL(*renderSurface, setBufferDataspace(ui::Dataspace::DISPLAY_P3)).Times(1); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 410 |     EXPECT_CALL(mHwComposer, | 
| Lloyd Pique | ef95812 | 2019-02-05 18:00:12 -0800 | [diff] [blame] | 411 |                 setActiveColorMode(DEFAULT_DISPLAY_ID, ui::ColorMode::DISPLAY_P3, | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 412 |                                    ui::RenderIntent::TONE_MAP_COLORIMETRIC)) | 
 | 413 |             .Times(1); | 
 | 414 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 415 |     mDisplay->setColorProfile(ColorProfile{ui::ColorMode::DISPLAY_P3, ui::Dataspace::DISPLAY_P3, | 
 | 416 |                                            ui::RenderIntent::TONE_MAP_COLORIMETRIC, | 
 | 417 |                                            ui::Dataspace::UNKNOWN}); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 418 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 419 |     EXPECT_EQ(ui::ColorMode::DISPLAY_P3, mDisplay->getState().colorMode); | 
 | 420 |     EXPECT_EQ(ui::Dataspace::DISPLAY_P3, mDisplay->getState().dataspace); | 
 | 421 |     EXPECT_EQ(ui::RenderIntent::TONE_MAP_COLORIMETRIC, mDisplay->getState().renderIntent); | 
 | 422 |     EXPECT_EQ(ui::Dataspace::UNKNOWN, mDisplay->getState().targetDataspace); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 423 | } | 
 | 424 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 425 | TEST_F(DisplaySetColorModeTest, doesNothingForVirtualDisplay) { | 
| Lloyd Pique | 6a3b446 | 2019-03-07 20:58:12 -0800 | [diff] [blame] | 426 |     using ColorProfile = Output::ColorProfile; | 
 | 427 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 428 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 429 |     std::shared_ptr<impl::Display> virtualDisplay = impl::createDisplay(mCompositionEngine, args); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 430 |  | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 431 |     mock::DisplayColorProfile* colorProfile = new StrictMock<mock::DisplayColorProfile>(); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 432 |     virtualDisplay->setDisplayColorProfileForTest( | 
| Lloyd Pique | f527548 | 2019-01-29 18:42:42 -0800 | [diff] [blame] | 433 |             std::unique_ptr<DisplayColorProfile>(colorProfile)); | 
 | 434 |  | 
 | 435 |     EXPECT_CALL(*colorProfile, | 
 | 436 |                 getTargetDataspace(ui::ColorMode::DISPLAY_P3, ui::Dataspace::DISPLAY_P3, | 
 | 437 |                                    ui::Dataspace::UNKNOWN)) | 
 | 438 |             .WillOnce(Return(ui::Dataspace::UNKNOWN)); | 
 | 439 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 440 |     virtualDisplay->setColorProfile( | 
| Lloyd Pique | 6a3b446 | 2019-03-07 20:58:12 -0800 | [diff] [blame] | 441 |             ColorProfile{ui::ColorMode::DISPLAY_P3, ui::Dataspace::DISPLAY_P3, | 
 | 442 |                          ui::RenderIntent::TONE_MAP_COLORIMETRIC, ui::Dataspace::UNKNOWN}); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 443 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 444 |     EXPECT_EQ(ui::ColorMode::NATIVE, virtualDisplay->getState().colorMode); | 
 | 445 |     EXPECT_EQ(ui::Dataspace::UNKNOWN, virtualDisplay->getState().dataspace); | 
 | 446 |     EXPECT_EQ(ui::RenderIntent::COLORIMETRIC, virtualDisplay->getState().renderIntent); | 
 | 447 |     EXPECT_EQ(ui::Dataspace::UNKNOWN, virtualDisplay->getState().targetDataspace); | 
| Lloyd Pique | 32cbe28 | 2018-10-19 13:09:22 -0700 | [diff] [blame] | 448 | } | 
 | 449 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 450 | /* | 
| Lloyd Pique | 3d0c02e | 2018-10-19 18:38:12 -0700 | [diff] [blame] | 451 |  * Display::createDisplayColorProfile() | 
 | 452 |  */ | 
 | 453 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 454 | using DisplayCreateColorProfileTest = PartialMockDisplayTestCommon; | 
 | 455 |  | 
 | 456 | TEST_F(DisplayCreateColorProfileTest, setsDisplayColorProfile) { | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 457 |     EXPECT_TRUE(mDisplay->getDisplayColorProfile() == nullptr); | 
 | 458 |     mDisplay->createDisplayColorProfile( | 
| Lloyd Pique | 3d0c02e | 2018-10-19 18:38:12 -0700 | [diff] [blame] | 459 |             DisplayColorProfileCreationArgs{false, HdrCapabilities(), 0, | 
 | 460 |                                             DisplayColorProfileCreationArgs::HwcColorModes()}); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 461 |     EXPECT_TRUE(mDisplay->getDisplayColorProfile() != nullptr); | 
| Lloyd Pique | 3d0c02e | 2018-10-19 18:38:12 -0700 | [diff] [blame] | 462 | } | 
 | 463 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 464 | /* | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 465 |  * Display::createRenderSurface() | 
 | 466 |  */ | 
 | 467 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 468 | using DisplayCreateRenderSurfaceTest = PartialMockDisplayTestCommon; | 
 | 469 |  | 
 | 470 | TEST_F(DisplayCreateRenderSurfaceTest, setsRenderSurface) { | 
| chaviw | 8beb414 | 2019-04-11 13:09:05 -0700 | [diff] [blame] | 471 |     EXPECT_CALL(*mNativeWindow, disconnect(NATIVE_WINDOW_API_EGL)).WillRepeatedly(Return(NO_ERROR)); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 472 |     EXPECT_TRUE(mDisplay->getRenderSurface() == nullptr); | 
| Dominik Laskowski | b9ac3e1 | 2021-04-23 13:01:16 -0700 | [diff] [blame] | 473 |     mDisplay->createRenderSurface(RenderSurfaceCreationArgsBuilder() | 
 | 474 |                                           .setDisplayWidth(640) | 
 | 475 |                                           .setDisplayHeight(480) | 
 | 476 |                                           .setNativeWindow(mNativeWindow) | 
 | 477 |                                           .build()); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 478 |     EXPECT_TRUE(mDisplay->getRenderSurface() != nullptr); | 
| Lloyd Pique | 31cb294 | 2018-10-19 17:23:03 -0700 | [diff] [blame] | 479 | } | 
 | 480 |  | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 481 | /* | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 482 |  * Display::createOutputLayer() | 
 | 483 |  */ | 
 | 484 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 485 | using DisplayCreateOutputLayerTest = FullDisplayImplTestCommon; | 
 | 486 |  | 
 | 487 | TEST_F(DisplayCreateOutputLayerTest, setsHwcLayer) { | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 488 |     sp<mock::LayerFE> layerFE = new StrictMock<mock::LayerFE>(); | 
| Lloyd Pique | 1b33fc3 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 489 |     auto hwcLayer = std::make_shared<StrictMock<HWC2::mock::Layer>>(); | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 490 |  | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 491 |     EXPECT_CALL(mHwComposer, createLayer(HalDisplayId(DEFAULT_DISPLAY_ID))) | 
| Lloyd Pique | 1b33fc3 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 492 |             .WillOnce(Return(hwcLayer)); | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 493 |  | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 494 |     auto outputLayer = mDisplay->createOutputLayer(layerFE); | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 495 |  | 
| Lloyd Pique | 1b33fc3 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 496 |     EXPECT_EQ(hwcLayer.get(), outputLayer->getHwcLayer()); | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 497 |  | 
| Lloyd Pique | df336d9 | 2019-03-07 21:38:42 -0800 | [diff] [blame] | 498 |     outputLayer.reset(); | 
 | 499 | } | 
 | 500 |  | 
 | 501 | /* | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 502 |  * Display::setReleasedLayers() | 
 | 503 |  */ | 
 | 504 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 505 | using DisplaySetReleasedLayersTest = DisplayWithLayersTestCommon; | 
 | 506 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 507 | TEST_F(DisplaySetReleasedLayersTest, doesNothingIfGpuDisplay) { | 
 | 508 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 509 |     std::shared_ptr<impl::Display> gpuDisplay = impl::createDisplay(mCompositionEngine, args); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 510 |  | 
 | 511 |     sp<mock::LayerFE> layerXLayerFE = new StrictMock<mock::LayerFE>(); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 512 |  | 
 | 513 |     { | 
 | 514 |         Output::ReleasedLayers releasedLayers; | 
 | 515 |         releasedLayers.emplace_back(layerXLayerFE); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 516 |         gpuDisplay->setReleasedLayers(std::move(releasedLayers)); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 517 |     } | 
 | 518 |  | 
 | 519 |     CompositionRefreshArgs refreshArgs; | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 520 |     refreshArgs.layersWithQueuedFrames.push_back(layerXLayerFE); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 521 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 522 |     gpuDisplay->setReleasedLayers(refreshArgs); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 523 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 524 |     const auto& releasedLayers = gpuDisplay->getReleasedLayersForTest(); | 
 | 525 |     ASSERT_EQ(1u, releasedLayers.size()); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 526 | } | 
 | 527 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 528 | TEST_F(DisplaySetReleasedLayersTest, doesNothingIfNoLayersWithQueuedFrames) { | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 529 |     sp<mock::LayerFE> layerXLayerFE = new StrictMock<mock::LayerFE>(); | 
 | 530 |  | 
 | 531 |     { | 
 | 532 |         Output::ReleasedLayers releasedLayers; | 
 | 533 |         releasedLayers.emplace_back(layerXLayerFE); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 534 |         mDisplay->setReleasedLayers(std::move(releasedLayers)); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 535 |     } | 
 | 536 |  | 
 | 537 |     CompositionRefreshArgs refreshArgs; | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 538 |     mDisplay->setReleasedLayers(refreshArgs); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 539 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 540 |     const auto& releasedLayers = mDisplay->getReleasedLayersForTest(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 541 |     ASSERT_EQ(1u, releasedLayers.size()); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 542 | } | 
 | 543 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 544 | TEST_F(DisplaySetReleasedLayersTest, setReleasedLayers) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 545 |     sp<mock::LayerFE> unknownLayer = new StrictMock<mock::LayerFE>(); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 546 |  | 
 | 547 |     CompositionRefreshArgs refreshArgs; | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 548 |     refreshArgs.layersWithQueuedFrames.push_back(mLayer1.layerFE); | 
 | 549 |     refreshArgs.layersWithQueuedFrames.push_back(mLayer2.layerFE); | 
 | 550 |     refreshArgs.layersWithQueuedFrames.push_back(unknownLayer); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 551 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 552 |     mDisplay->setReleasedLayers(refreshArgs); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 553 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 554 |     const auto& releasedLayers = mDisplay->getReleasedLayersForTest(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 555 |     ASSERT_EQ(2u, releasedLayers.size()); | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 556 |     ASSERT_EQ(mLayer1.layerFE.get(), releasedLayers[0].promote().get()); | 
 | 557 |     ASSERT_EQ(mLayer2.layerFE.get(), releasedLayers[1].promote().get()); | 
| Lloyd Pique | c29e4c6 | 2019-03-07 21:48:19 -0800 | [diff] [blame] | 558 | } | 
 | 559 |  | 
 | 560 | /* | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 561 |  * Display::chooseCompositionStrategy() | 
 | 562 |  */ | 
 | 563 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 564 | using DisplayChooseCompositionStrategyTest = PartialMockDisplayTestCommon; | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 565 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 566 | TEST_F(DisplayChooseCompositionStrategyTest, takesEarlyOutIfGpuDisplay) { | 
 | 567 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 568 |     std::shared_ptr<Display> gpuDisplay = | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 569 |             createPartialMockDisplay<Display>(mCompositionEngine, args); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 570 |     EXPECT_TRUE(GpuVirtualDisplayId::tryCast(gpuDisplay->getId())); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 571 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 572 |     gpuDisplay->chooseCompositionStrategy(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 573 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 574 |     auto& state = gpuDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 575 |     EXPECT_TRUE(state.usesClientComposition); | 
 | 576 |     EXPECT_FALSE(state.usesDeviceComposition); | 
 | 577 | } | 
 | 578 |  | 
 | 579 | TEST_F(DisplayChooseCompositionStrategyTest, takesEarlyOutOnHwcError) { | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 580 |     EXPECT_CALL(*mDisplay, anyLayersRequireClientComposition()).WillOnce(Return(false)); | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 581 |     EXPECT_CALL(mHwComposer, | 
| Ady Abraham | ec7aa8a | 2021-06-28 12:37:09 -0700 | [diff] [blame] | 582 |                 getDeviceCompositionChanges(HalDisplayId(DEFAULT_DISPLAY_ID), false, _, _, _)) | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 583 |             .WillOnce(Return(INVALID_OPERATION)); | 
 | 584 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 585 |     mDisplay->chooseCompositionStrategy(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 586 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 587 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 588 |     EXPECT_TRUE(state.usesClientComposition); | 
 | 589 |     EXPECT_FALSE(state.usesDeviceComposition); | 
 | 590 | } | 
 | 591 |  | 
 | 592 | TEST_F(DisplayChooseCompositionStrategyTest, normalOperation) { | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 593 |     // Since two calls are made to anyLayersRequireClientComposition with different return | 
 | 594 |     // values, use a Sequence to control the matching so the values are returned in a known | 
 | 595 |     // order. | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 596 |     Sequence s; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 597 |     EXPECT_CALL(*mDisplay, anyLayersRequireClientComposition()) | 
 | 598 |             .InSequence(s) | 
 | 599 |             .WillOnce(Return(true)); | 
 | 600 |     EXPECT_CALL(*mDisplay, anyLayersRequireClientComposition()) | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 601 |             .InSequence(s) | 
 | 602 |             .WillOnce(Return(false)); | 
 | 603 |  | 
| Ady Abraham | b42cdc1 | 2021-05-11 14:31:26 -0700 | [diff] [blame] | 604 |     EXPECT_CALL(mHwComposer, | 
| Ady Abraham | ec7aa8a | 2021-06-28 12:37:09 -0700 | [diff] [blame] | 605 |                 getDeviceCompositionChanges(HalDisplayId(DEFAULT_DISPLAY_ID), true, _, _, _)) | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 606 |             .WillOnce(Return(NO_ERROR)); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 607 |     EXPECT_CALL(*mDisplay, allLayersRequireClientComposition()).WillOnce(Return(false)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 608 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 609 |     mDisplay->chooseCompositionStrategy(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 610 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 611 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 612 |     EXPECT_FALSE(state.usesClientComposition); | 
 | 613 |     EXPECT_TRUE(state.usesDeviceComposition); | 
 | 614 | } | 
 | 615 |  | 
 | 616 | TEST_F(DisplayChooseCompositionStrategyTest, normalOperationWithChanges) { | 
 | 617 |     android::HWComposer::DeviceRequestedChanges changes{ | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 618 |             {{nullptr, hal::Composition::CLIENT}}, | 
 | 619 |             hal::DisplayRequest::FLIP_CLIENT_TARGET, | 
 | 620 |             {{nullptr, hal::LayerRequest::CLEAR_CLIENT_TARGET}}, | 
| Peiyong Lin | dfc3f7c | 2020-05-07 20:15:50 -0700 | [diff] [blame] | 621 |             {hal::PixelFormat::RGBA_8888, hal::Dataspace::UNKNOWN}, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 622 |     }; | 
 | 623 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 624 |     // Since two calls are made to anyLayersRequireClientComposition with different return | 
 | 625 |     // values, use a Sequence to control the matching so the values are returned in a known | 
 | 626 |     // order. | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 627 |     Sequence s; | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 628 |     EXPECT_CALL(*mDisplay, anyLayersRequireClientComposition()) | 
 | 629 |             .InSequence(s) | 
 | 630 |             .WillOnce(Return(true)); | 
 | 631 |     EXPECT_CALL(*mDisplay, anyLayersRequireClientComposition()) | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 632 |             .InSequence(s) | 
 | 633 |             .WillOnce(Return(false)); | 
 | 634 |  | 
| Ady Abraham | b42cdc1 | 2021-05-11 14:31:26 -0700 | [diff] [blame] | 635 |     EXPECT_CALL(mHwComposer, | 
| Ady Abraham | ec7aa8a | 2021-06-28 12:37:09 -0700 | [diff] [blame] | 636 |                 getDeviceCompositionChanges(HalDisplayId(DEFAULT_DISPLAY_ID), true, _, _, _)) | 
 | 637 |             .WillOnce(DoAll(SetArgPointee<4>(changes), Return(NO_ERROR))); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 638 |     EXPECT_CALL(*mDisplay, applyChangedTypesToLayers(changes.changedTypes)).Times(1); | 
 | 639 |     EXPECT_CALL(*mDisplay, applyDisplayRequests(changes.displayRequests)).Times(1); | 
 | 640 |     EXPECT_CALL(*mDisplay, applyLayerRequestsToLayers(changes.layerRequests)).Times(1); | 
 | 641 |     EXPECT_CALL(*mDisplay, allLayersRequireClientComposition()).WillOnce(Return(false)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 642 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 643 |     mDisplay->chooseCompositionStrategy(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 644 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 645 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 646 |     EXPECT_FALSE(state.usesClientComposition); | 
 | 647 |     EXPECT_TRUE(state.usesDeviceComposition); | 
 | 648 | } | 
 | 649 |  | 
 | 650 | /* | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 651 |  * Display::getSkipColorTransform() | 
 | 652 |  */ | 
 | 653 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 654 | using DisplayGetSkipColorTransformTest = DisplayWithLayersTestCommon; | 
 | 655 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 656 | TEST_F(DisplayGetSkipColorTransformTest, checksCapabilityIfGpuDisplay) { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 657 |     EXPECT_CALL(mHwComposer, hasCapability(hal::Capability::SKIP_CLIENT_COLOR_TRANSFORM)) | 
| Dominik Laskowski | 1162e47 | 2020-04-02 19:02:47 -0700 | [diff] [blame] | 658 |             .WillOnce(Return(true)); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 659 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 660 |     auto gpuDisplay{impl::createDisplay(mCompositionEngine, args)}; | 
 | 661 |     EXPECT_TRUE(gpuDisplay->getSkipColorTransform()); | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 662 | } | 
 | 663 |  | 
| Dominik Laskowski | 1162e47 | 2020-04-02 19:02:47 -0700 | [diff] [blame] | 664 | TEST_F(DisplayGetSkipColorTransformTest, checksDisplayCapability) { | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 665 |     EXPECT_CALL(mHwComposer, | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 666 |                 hasDisplayCapability(HalDisplayId(DEFAULT_DISPLAY_ID), | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 667 |                                      hal::DisplayCapability::SKIP_CLIENT_COLOR_TRANSFORM)) | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 668 |             .WillOnce(Return(true)); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 669 |     EXPECT_TRUE(mDisplay->getSkipColorTransform()); | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 670 | } | 
 | 671 |  | 
 | 672 | /* | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 673 |  * Display::anyLayersRequireClientComposition() | 
 | 674 |  */ | 
 | 675 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 676 | using DisplayAnyLayersRequireClientCompositionTest = DisplayWithLayersTestCommon; | 
 | 677 |  | 
 | 678 | TEST_F(DisplayAnyLayersRequireClientCompositionTest, returnsFalse) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 679 |     EXPECT_CALL(*mLayer1.outputLayer, requiresClientComposition()).WillOnce(Return(false)); | 
 | 680 |     EXPECT_CALL(*mLayer2.outputLayer, requiresClientComposition()).WillOnce(Return(false)); | 
 | 681 |     EXPECT_CALL(*mLayer3.outputLayer, requiresClientComposition()).WillOnce(Return(false)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 682 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 683 |     EXPECT_FALSE(mDisplay->anyLayersRequireClientComposition()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 684 | } | 
 | 685 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 686 | TEST_F(DisplayAnyLayersRequireClientCompositionTest, returnsTrue) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 687 |     EXPECT_CALL(*mLayer1.outputLayer, requiresClientComposition()).WillOnce(Return(false)); | 
 | 688 |     EXPECT_CALL(*mLayer2.outputLayer, requiresClientComposition()).WillOnce(Return(true)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 689 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 690 |     EXPECT_TRUE(mDisplay->anyLayersRequireClientComposition()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 691 | } | 
 | 692 |  | 
 | 693 | /* | 
 | 694 |  * Display::allLayersRequireClientComposition() | 
 | 695 |  */ | 
 | 696 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 697 | using DisplayAllLayersRequireClientCompositionTest = DisplayWithLayersTestCommon; | 
 | 698 |  | 
 | 699 | TEST_F(DisplayAllLayersRequireClientCompositionTest, returnsTrue) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 700 |     EXPECT_CALL(*mLayer1.outputLayer, requiresClientComposition()).WillOnce(Return(true)); | 
 | 701 |     EXPECT_CALL(*mLayer2.outputLayer, requiresClientComposition()).WillOnce(Return(true)); | 
 | 702 |     EXPECT_CALL(*mLayer3.outputLayer, requiresClientComposition()).WillOnce(Return(true)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 703 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 704 |     EXPECT_TRUE(mDisplay->allLayersRequireClientComposition()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 705 | } | 
 | 706 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 707 | TEST_F(DisplayAllLayersRequireClientCompositionTest, returnsFalse) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 708 |     EXPECT_CALL(*mLayer1.outputLayer, requiresClientComposition()).WillOnce(Return(true)); | 
 | 709 |     EXPECT_CALL(*mLayer2.outputLayer, requiresClientComposition()).WillOnce(Return(false)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 710 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 711 |     EXPECT_FALSE(mDisplay->allLayersRequireClientComposition()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 712 | } | 
 | 713 |  | 
 | 714 | /* | 
 | 715 |  * Display::applyChangedTypesToLayers() | 
 | 716 |  */ | 
 | 717 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 718 | using DisplayApplyChangedTypesToLayersTest = DisplayWithLayersTestCommon; | 
 | 719 |  | 
 | 720 | TEST_F(DisplayApplyChangedTypesToLayersTest, takesEarlyOutIfNoChangedLayers) { | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 721 |     mDisplay->applyChangedTypesToLayers(impl::Display::ChangedTypes()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 722 | } | 
 | 723 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 724 | TEST_F(DisplayApplyChangedTypesToLayersTest, appliesChanges) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 725 |     EXPECT_CALL(*mLayer1.outputLayer, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 726 |                 applyDeviceCompositionTypeChange(Hwc2::IComposerClient::Composition::CLIENT)) | 
 | 727 |             .Times(1); | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 728 |     EXPECT_CALL(*mLayer2.outputLayer, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 729 |                 applyDeviceCompositionTypeChange(Hwc2::IComposerClient::Composition::DEVICE)) | 
 | 730 |             .Times(1); | 
 | 731 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 732 |     mDisplay->applyChangedTypesToLayers(impl::Display::ChangedTypes{ | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 733 |             {&mLayer1.hwc2Layer, hal::Composition::CLIENT}, | 
 | 734 |             {&mLayer2.hwc2Layer, hal::Composition::DEVICE}, | 
 | 735 |             {&hwc2LayerUnknown, hal::Composition::SOLID_COLOR}, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 736 |     }); | 
 | 737 | } | 
 | 738 |  | 
 | 739 | /* | 
 | 740 |  * Display::applyDisplayRequests() | 
 | 741 |  */ | 
 | 742 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 743 | using DisplayApplyDisplayRequestsTest = DisplayWithLayersTestCommon; | 
 | 744 |  | 
 | 745 | TEST_F(DisplayApplyDisplayRequestsTest, handlesNoRequests) { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 746 |     mDisplay->applyDisplayRequests(static_cast<hal::DisplayRequest>(0)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 747 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 748 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 749 |     EXPECT_FALSE(state.flipClientTarget); | 
 | 750 | } | 
 | 751 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 752 | TEST_F(DisplayApplyDisplayRequestsTest, handlesFlipClientTarget) { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 753 |     mDisplay->applyDisplayRequests(hal::DisplayRequest::FLIP_CLIENT_TARGET); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 754 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 755 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 756 |     EXPECT_TRUE(state.flipClientTarget); | 
 | 757 | } | 
 | 758 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 759 | TEST_F(DisplayApplyDisplayRequestsTest, handlesWriteClientTargetToOutput) { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 760 |     mDisplay->applyDisplayRequests(hal::DisplayRequest::WRITE_CLIENT_TARGET_TO_OUTPUT); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 761 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 762 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 763 |     EXPECT_FALSE(state.flipClientTarget); | 
 | 764 | } | 
 | 765 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 766 | TEST_F(DisplayApplyDisplayRequestsTest, handlesAllRequestFlagsSet) { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 767 |     mDisplay->applyDisplayRequests(static_cast<hal::DisplayRequest>(~0)); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 768 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 769 |     auto& state = mDisplay->getState(); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 770 |     EXPECT_TRUE(state.flipClientTarget); | 
 | 771 | } | 
 | 772 |  | 
 | 773 | /* | 
 | 774 |  * Display::applyLayerRequestsToLayers() | 
 | 775 |  */ | 
 | 776 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 777 | using DisplayApplyLayerRequestsToLayersTest = DisplayWithLayersTestCommon; | 
 | 778 |  | 
 | 779 | TEST_F(DisplayApplyLayerRequestsToLayersTest, preparesAllLayers) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 780 |     EXPECT_CALL(*mLayer1.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
 | 781 |     EXPECT_CALL(*mLayer2.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
 | 782 |     EXPECT_CALL(*mLayer3.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 783 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 784 |     mDisplay->applyLayerRequestsToLayers(impl::Display::LayerRequests()); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 785 | } | 
 | 786 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 787 | TEST_F(DisplayApplyLayerRequestsToLayersTest, appliesDeviceLayerRequests) { | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 788 |     EXPECT_CALL(*mLayer1.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
 | 789 |     EXPECT_CALL(*mLayer2.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
 | 790 |     EXPECT_CALL(*mLayer3.outputLayer, prepareForDeviceLayerRequests()).Times(1); | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 791 |  | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 792 |     EXPECT_CALL(*mLayer1.outputLayer, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 793 |                 applyDeviceLayerRequest(Hwc2::IComposerClient::LayerRequest::CLEAR_CLIENT_TARGET)) | 
 | 794 |             .Times(1); | 
 | 795 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 796 |     mDisplay->applyLayerRequestsToLayers(impl::Display::LayerRequests{ | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 797 |             {&mLayer1.hwc2Layer, hal::LayerRequest::CLEAR_CLIENT_TARGET}, | 
 | 798 |             {&hwc2LayerUnknown, hal::LayerRequest::CLEAR_CLIENT_TARGET}, | 
| Lloyd Pique | 66d6860 | 2019-02-13 14:23:31 -0800 | [diff] [blame] | 799 |     }); | 
 | 800 | } | 
 | 801 |  | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 802 | /* | 
| Ady Abraham | 0094dc6 | 2021-06-03 10:08:33 -0700 | [diff] [blame] | 803 |  * Display::applyClientTargetRequests() | 
 | 804 |  */ | 
 | 805 |  | 
 | 806 | using DisplayApplyClientTargetRequests = DisplayWithLayersTestCommon; | 
 | 807 |  | 
 | 808 | TEST_F(DisplayApplyLayerRequestsToLayersTest, applyClientTargetRequests) { | 
 | 809 |     Display::ClientTargetProperty clientTargetProperty = { | 
 | 810 |             .pixelFormat = hal::PixelFormat::RGB_565, | 
 | 811 |             .dataspace = hal::Dataspace::STANDARD_BT470M, | 
 | 812 |     }; | 
 | 813 |  | 
 | 814 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
 | 815 |     mDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
 | 816 |  | 
 | 817 |     EXPECT_CALL(*renderSurface, setBufferPixelFormat(clientTargetProperty.pixelFormat)); | 
 | 818 |     EXPECT_CALL(*renderSurface, setBufferDataspace(clientTargetProperty.dataspace)); | 
 | 819 |     mDisplay->applyClientTargetRequests(clientTargetProperty); | 
 | 820 |  | 
 | 821 |     auto& state = mDisplay->getState(); | 
 | 822 |     EXPECT_EQ(clientTargetProperty.dataspace, state.dataspace); | 
 | 823 | } | 
 | 824 |  | 
 | 825 | /* | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 826 |  * Display::presentAndGetFrameFences() | 
 | 827 |  */ | 
 | 828 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 829 | using DisplayPresentAndGetFrameFencesTest = DisplayWithLayersTestCommon; | 
 | 830 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 831 | TEST_F(DisplayPresentAndGetFrameFencesTest, returnsNoFencesOnGpuDisplay) { | 
 | 832 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 833 |     auto gpuDisplay{impl::createDisplay(mCompositionEngine, args)}; | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 834 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 835 |     auto result = gpuDisplay->presentAndGetFrameFences(); | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 836 |  | 
 | 837 |     ASSERT_TRUE(result.presentFence.get()); | 
 | 838 |     EXPECT_FALSE(result.presentFence->isValid()); | 
 | 839 |     EXPECT_EQ(0u, result.layerFences.size()); | 
 | 840 | } | 
 | 841 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 842 | TEST_F(DisplayPresentAndGetFrameFencesTest, returnsPresentAndLayerFences) { | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 843 |     sp<Fence> presentFence = new Fence(); | 
 | 844 |     sp<Fence> layer1Fence = new Fence(); | 
 | 845 |     sp<Fence> layer2Fence = new Fence(); | 
 | 846 |  | 
| Ady Abraham | ec7aa8a | 2021-06-28 12:37:09 -0700 | [diff] [blame] | 847 |     EXPECT_CALL(mHwComposer, presentAndGetReleaseFences(HalDisplayId(DEFAULT_DISPLAY_ID), _, _)) | 
| Ady Abraham | b42cdc1 | 2021-05-11 14:31:26 -0700 | [diff] [blame] | 848 |             .Times(1); | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 849 |     EXPECT_CALL(mHwComposer, getPresentFence(HalDisplayId(DEFAULT_DISPLAY_ID))) | 
 | 850 |             .WillOnce(Return(presentFence)); | 
 | 851 |     EXPECT_CALL(mHwComposer, | 
 | 852 |                 getLayerReleaseFence(HalDisplayId(DEFAULT_DISPLAY_ID), &mLayer1.hwc2Layer)) | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 853 |             .WillOnce(Return(layer1Fence)); | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 854 |     EXPECT_CALL(mHwComposer, | 
 | 855 |                 getLayerReleaseFence(HalDisplayId(DEFAULT_DISPLAY_ID), &mLayer2.hwc2Layer)) | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 856 |             .WillOnce(Return(layer2Fence)); | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 857 |     EXPECT_CALL(mHwComposer, clearReleaseFences(HalDisplayId(DEFAULT_DISPLAY_ID))).Times(1); | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 858 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 859 |     auto result = mDisplay->presentAndGetFrameFences(); | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 860 |  | 
 | 861 |     EXPECT_EQ(presentFence, result.presentFence); | 
 | 862 |  | 
 | 863 |     EXPECT_EQ(2u, result.layerFences.size()); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 864 |     ASSERT_EQ(1u, result.layerFences.count(&mLayer1.hwc2Layer)); | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 865 |     EXPECT_EQ(layer1Fence, result.layerFences[&mLayer1.hwc2Layer]); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 866 |     ASSERT_EQ(1u, result.layerFences.count(&mLayer2.hwc2Layer)); | 
| Lloyd Pique | de19665 | 2020-01-22 17:29:58 -0800 | [diff] [blame] | 867 |     EXPECT_EQ(layer2Fence, result.layerFences[&mLayer2.hwc2Layer]); | 
| Lloyd Pique | 35fca9d | 2019-02-13 14:24:11 -0800 | [diff] [blame] | 868 | } | 
 | 869 |  | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 870 | /* | 
 | 871 |  * Display::setExpensiveRenderingExpected() | 
 | 872 |  */ | 
 | 873 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 874 | using DisplaySetExpensiveRenderingExpectedTest = DisplayWithLayersTestCommon; | 
 | 875 |  | 
 | 876 | TEST_F(DisplaySetExpensiveRenderingExpectedTest, forwardsToPowerAdvisor) { | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 877 |     EXPECT_CALL(mPowerAdvisor, setExpensiveRenderingExpected(DEFAULT_DISPLAY_ID, true)).Times(1); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 878 |     mDisplay->setExpensiveRenderingExpected(true); | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 879 |  | 
 | 880 |     EXPECT_CALL(mPowerAdvisor, setExpensiveRenderingExpected(DEFAULT_DISPLAY_ID, false)).Times(1); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 881 |     mDisplay->setExpensiveRenderingExpected(false); | 
| Lloyd Pique | 688abd4 | 2019-02-15 15:42:24 -0800 | [diff] [blame] | 882 | } | 
 | 883 |  | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 884 | /* | 
 | 885 |  * Display::finishFrame() | 
 | 886 |  */ | 
 | 887 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 888 | using DisplayFinishFrameTest = DisplayWithLayersTestCommon; | 
 | 889 |  | 
 | 890 | TEST_F(DisplayFinishFrameTest, doesNotSkipCompositionIfNotDirtyOnHwcDisplay) { | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 891 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 892 |     mDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 893 |  | 
 | 894 |     // We expect no calls to queueBuffer if composition was skipped. | 
 | 895 |     EXPECT_CALL(*renderSurface, queueBuffer(_)).Times(1); | 
 | 896 |  | 
| Lloyd Pique | a76ce46 | 2020-01-14 13:06:37 -0800 | [diff] [blame] | 897 |     // Expect a call to signal no expensive rendering since there is no client composition. | 
 | 898 |     EXPECT_CALL(mPowerAdvisor, setExpensiveRenderingExpected(DEFAULT_DISPLAY_ID, false)); | 
 | 899 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 900 |     mDisplay->editState().isEnabled = true; | 
 | 901 |     mDisplay->editState().usesClientComposition = false; | 
| Marin Shalamanov | 6ad317c | 2020-07-29 23:34:07 +0200 | [diff] [blame] | 902 |     mDisplay->editState().layerStackSpace.content = Rect(0, 0, 1, 1); | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 903 |     mDisplay->editState().dirtyRegion = Region::INVALID_REGION; | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 904 |  | 
 | 905 |     CompositionRefreshArgs refreshArgs; | 
 | 906 |     refreshArgs.repaintEverything = false; | 
 | 907 |  | 
| Lloyd Pique | a38ea7e | 2019-04-16 18:10:26 -0700 | [diff] [blame] | 908 |     mDisplay->finishFrame(refreshArgs); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 909 | } | 
 | 910 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 911 | TEST_F(DisplayFinishFrameTest, skipsCompositionIfNotDirty) { | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 912 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 913 |     std::shared_ptr<impl::Display> gpuDisplay = impl::createDisplay(mCompositionEngine, args); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 914 |  | 
 | 915 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 916 |     gpuDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 917 |  | 
 | 918 |     // We expect no calls to queueBuffer if composition was skipped. | 
 | 919 |     EXPECT_CALL(*renderSurface, queueBuffer(_)).Times(0); | 
 | 920 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 921 |     gpuDisplay->editState().isEnabled = true; | 
 | 922 |     gpuDisplay->editState().usesClientComposition = false; | 
 | 923 |     gpuDisplay->editState().layerStackSpace.content = Rect(0, 0, 1, 1); | 
 | 924 |     gpuDisplay->editState().dirtyRegion = Region::INVALID_REGION; | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 925 |  | 
 | 926 |     CompositionRefreshArgs refreshArgs; | 
 | 927 |     refreshArgs.repaintEverything = false; | 
 | 928 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 929 |     gpuDisplay->finishFrame(refreshArgs); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 930 | } | 
 | 931 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 932 | TEST_F(DisplayFinishFrameTest, performsCompositionIfDirty) { | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 933 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 934 |     std::shared_ptr<impl::Display> gpuDisplay = impl::createDisplay(mCompositionEngine, args); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 935 |  | 
 | 936 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 937 |     gpuDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 938 |  | 
 | 939 |     // We expect a single call to queueBuffer when composition is not skipped. | 
 | 940 |     EXPECT_CALL(*renderSurface, queueBuffer(_)).Times(1); | 
 | 941 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 942 |     gpuDisplay->editState().isEnabled = true; | 
 | 943 |     gpuDisplay->editState().usesClientComposition = false; | 
 | 944 |     gpuDisplay->editState().layerStackSpace.content = Rect(0, 0, 1, 1); | 
 | 945 |     gpuDisplay->editState().dirtyRegion = Region(Rect(0, 0, 1, 1)); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 946 |  | 
 | 947 |     CompositionRefreshArgs refreshArgs; | 
 | 948 |     refreshArgs.repaintEverything = false; | 
 | 949 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 950 |     gpuDisplay->finishFrame(refreshArgs); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 951 | } | 
 | 952 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 953 | TEST_F(DisplayFinishFrameTest, performsCompositionIfRepaintEverything) { | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 954 |     auto args = getDisplayCreationArgsForGpuVirtualDisplay(); | 
 | 955 |     std::shared_ptr<impl::Display> gpuDisplay = impl::createDisplay(mCompositionEngine, args); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 956 |  | 
 | 957 |     mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 958 |     gpuDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface)); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 959 |  | 
 | 960 |     // We expect a single call to queueBuffer when composition is not skipped. | 
 | 961 |     EXPECT_CALL(*renderSurface, queueBuffer(_)).Times(1); | 
 | 962 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 963 |     gpuDisplay->editState().isEnabled = true; | 
 | 964 |     gpuDisplay->editState().usesClientComposition = false; | 
 | 965 |     gpuDisplay->editState().layerStackSpace.content = Rect(0, 0, 1, 1); | 
 | 966 |     gpuDisplay->editState().dirtyRegion = Region::INVALID_REGION; | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 967 |  | 
 | 968 |     CompositionRefreshArgs refreshArgs; | 
 | 969 |     refreshArgs.repaintEverything = true; | 
 | 970 |  | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 971 |     gpuDisplay->finishFrame(refreshArgs); | 
| Lloyd Pique | d3d6988 | 2019-02-28 16:03:46 -0800 | [diff] [blame] | 972 | } | 
 | 973 |  | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 974 | /* | 
 | 975 |  * Display functional tests | 
 | 976 |  */ | 
 | 977 |  | 
 | 978 | struct DisplayFunctionalTest : public testing::Test { | 
 | 979 |     class Display : public impl::Display { | 
 | 980 |     public: | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 981 |         using impl::Display::injectOutputLayerForTest; | 
 | 982 |         virtual void injectOutputLayerForTest(std::unique_ptr<compositionengine::OutputLayer>) = 0; | 
 | 983 |     }; | 
 | 984 |  | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 985 |     DisplayFunctionalTest() { | 
 | 986 |         EXPECT_CALL(mCompositionEngine, getHwComposer()).WillRepeatedly(ReturnRef(mHwComposer)); | 
 | 987 |  | 
 | 988 |         mDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(mRenderSurface)); | 
 | 989 |     } | 
 | 990 |  | 
 | 991 |     NiceMock<android::mock::HWComposer> mHwComposer; | 
 | 992 |     NiceMock<Hwc2::mock::PowerAdvisor> mPowerAdvisor; | 
 | 993 |     NiceMock<mock::CompositionEngine> mCompositionEngine; | 
 | 994 |     sp<mock::NativeWindow> mNativeWindow = new NiceMock<mock::NativeWindow>(); | 
 | 995 |     sp<mock::DisplaySurface> mDisplaySurface = new NiceMock<mock::DisplaySurface>(); | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 996 |  | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 997 |     std::shared_ptr<Display> mDisplay = impl::createDisplayTemplated< | 
 | 998 |             Display>(mCompositionEngine, | 
 | 999 |                      DisplayCreationArgsBuilder() | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 1000 |                              .setId(DEFAULT_DISPLAY_ID) | 
 | 1001 |                              .setConnectionType(ui::DisplayConnectionType::Internal) | 
 | 1002 |                              .setPixels(DEFAULT_RESOLUTION) | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 1003 |                              .setIsSecure(true) | 
 | 1004 |                              .setLayerStackId(DEFAULT_LAYER_STACK) | 
 | 1005 |                              .setPowerAdvisor(&mPowerAdvisor) | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 1006 |                              .build()); | 
| Lloyd Pique | aad4ebf | 2019-10-03 17:58:30 -0700 | [diff] [blame] | 1007 |  | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 1008 |     impl::RenderSurface* mRenderSurface = | 
 | 1009 |             new impl::RenderSurface{mCompositionEngine, *mDisplay, | 
| Dominik Laskowski | b9ac3e1 | 2021-04-23 13:01:16 -0700 | [diff] [blame] | 1010 |                                     RenderSurfaceCreationArgsBuilder() | 
| Dominik Laskowski | 1394860 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 1011 |                                             .setDisplayWidth(DEFAULT_RESOLUTION.width) | 
 | 1012 |                                             .setDisplayHeight(DEFAULT_RESOLUTION.height) | 
| Dominik Laskowski | b9ac3e1 | 2021-04-23 13:01:16 -0700 | [diff] [blame] | 1013 |                                             .setNativeWindow(mNativeWindow) | 
 | 1014 |                                             .setDisplaySurface(mDisplaySurface) | 
 | 1015 |                                             .build()}; | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 1016 | }; | 
 | 1017 |  | 
 | 1018 | TEST_F(DisplayFunctionalTest, postFramebufferCriticalCallsAreOrdered) { | 
 | 1019 |     InSequence seq; | 
 | 1020 |  | 
 | 1021 |     mDisplay->editState().isEnabled = true; | 
 | 1022 |  | 
| Ady Abraham | ec7aa8a | 2021-06-28 12:37:09 -0700 | [diff] [blame] | 1023 |     EXPECT_CALL(mHwComposer, presentAndGetReleaseFences(_, _, _)); | 
| Lloyd Pique | c660755 | 2019-12-02 17:57:39 -0800 | [diff] [blame] | 1024 |     EXPECT_CALL(*mDisplaySurface, onFrameCommitted()); | 
 | 1025 |  | 
 | 1026 |     mDisplay->postFramebuffer(); | 
 | 1027 | } | 
 | 1028 |  | 
| Lloyd Pique | 45a165a | 2018-10-19 11:54:47 -0700 | [diff] [blame] | 1029 | } // namespace | 
 | 1030 | } // namespace android::compositionengine |