Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 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 | |
Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 17 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 18 | #pragma clang diagnostic push |
| 19 | #pragma clang diagnostic ignored "-Wconversion" |
| 20 | |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 21 | #include <common/FlagManager.h> |
Jim Shargo | f48bef8 | 2024-12-04 18:56:59 +0000 | [diff] [blame^] | 22 | #include <gui/IConsumerListener.h> |
Dominik Laskowski | 3cb3d4e | 2019-11-21 11:14:45 -0800 | [diff] [blame] | 23 | #include <ui/DisplayState.h> |
| 24 | |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 25 | #include "LayerTransactionTest.h" |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 26 | #include "gui/SurfaceComposerClient.h" |
| 27 | #include "ui/DisplayId.h" |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 28 | |
| 29 | namespace android { |
| 30 | |
| 31 | using android::hardware::graphics::common::V1_1::BufferUsage; |
| 32 | |
| 33 | ::testing::Environment* const binderEnv = |
| 34 | ::testing::AddGlobalTestEnvironment(new BinderEnvironment()); |
| 35 | |
| 36 | class MultiDisplayLayerBoundsTest : public LayerTransactionTest { |
| 37 | protected: |
| 38 | virtual void SetUp() { |
| 39 | LayerTransactionTest::SetUp(); |
| 40 | ASSERT_EQ(NO_ERROR, mClient->initCheck()); |
| 41 | |
Huihong Luo | 31b5ac2 | 2022-08-15 20:38:10 -0700 | [diff] [blame] | 42 | const auto ids = SurfaceComposerClient::getPhysicalDisplayIds(); |
| 43 | ASSERT_FALSE(ids.empty()); |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 44 | mMainDisplayId = ids.front(); |
| 45 | mMainDisplay = SurfaceComposerClient::getPhysicalDisplayToken(mMainDisplayId); |
Dominik Laskowski | 3cb3d4e | 2019-11-21 11:14:45 -0800 | [diff] [blame] | 46 | SurfaceComposerClient::getDisplayState(mMainDisplay, &mMainDisplayState); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 47 | SurfaceComposerClient::getActiveDisplayMode(mMainDisplay, &mMainDisplayMode); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 48 | |
Jim Shargo | f48bef8 | 2024-12-04 18:56:59 +0000 | [diff] [blame^] | 49 | BufferQueue::createBufferQueue(&mProducer, &mConsumer); |
| 50 | mConsumer->setConsumerName(String8("Virtual disp consumer (MultiDisplayLayerBounds)")); |
| 51 | mConsumer->setDefaultBufferSize(mMainDisplayMode.resolution.getWidth(), |
| 52 | mMainDisplayMode.resolution.getHeight()); |
| 53 | |
| 54 | class StubConsumerListener : public BnConsumerListener { |
| 55 | virtual void onFrameAvailable(const BufferItem&) override {} |
| 56 | virtual void onBuffersReleased() override {} |
| 57 | virtual void onSidebandStreamChanged() override {} |
| 58 | }; |
| 59 | mConsumer->consumerConnect(sp<StubConsumerListener>::make(), true); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 60 | } |
| 61 | |
| 62 | virtual void TearDown() { |
Alan Ding | d53801c | 2024-05-08 16:45:29 -0700 | [diff] [blame] | 63 | EXPECT_EQ(NO_ERROR, SurfaceComposerClient::destroyVirtualDisplay(mVirtualDisplay)); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 64 | LayerTransactionTest::TearDown(); |
| 65 | mColorLayer = 0; |
| 66 | } |
| 67 | |
Dominik Laskowski | 29fa146 | 2021-04-27 15:51:50 -0700 | [diff] [blame] | 68 | void createDisplay(const ui::Size& layerStackSize, ui::LayerStack layerStack) { |
Alan Ding | d53801c | 2024-05-08 16:45:29 -0700 | [diff] [blame] | 69 | static const std::string kDisplayName("VirtualDisplay"); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 70 | mVirtualDisplay = |
Alan Ding | d53801c | 2024-05-08 16:45:29 -0700 | [diff] [blame] | 71 | SurfaceComposerClient::createVirtualDisplay(kDisplayName, false /*isSecure*/); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 72 | asTransaction([&](Transaction& t) { |
| 73 | t.setDisplaySurface(mVirtualDisplay, mProducer); |
| 74 | t.setDisplayLayerStack(mVirtualDisplay, layerStack); |
Dominik Laskowski | 3cb3d4e | 2019-11-21 11:14:45 -0800 | [diff] [blame] | 75 | t.setDisplayProjection(mVirtualDisplay, mMainDisplayState.orientation, |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 76 | Rect(layerStackSize), Rect(mMainDisplayMode.resolution)); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 77 | }); |
| 78 | } |
| 79 | |
Dominik Laskowski | 29fa146 | 2021-04-27 15:51:50 -0700 | [diff] [blame] | 80 | void createColorLayer(ui::LayerStack layerStack) { |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 81 | mColorLayer = |
| 82 | createSurface(mClient, "ColorLayer", 0 /* buffer width */, 0 /* buffer height */, |
Vishnu Nair | fa247b1 | 2020-02-11 08:58:26 -0800 | [diff] [blame] | 83 | PIXEL_FORMAT_RGBA_8888, ISurfaceComposerClient::eFXSurfaceEffect); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 84 | ASSERT_TRUE(mColorLayer != nullptr); |
| 85 | ASSERT_TRUE(mColorLayer->isValid()); |
| 86 | asTransaction([&](Transaction& t) { |
| 87 | t.setLayerStack(mColorLayer, layerStack); |
chaviw | 2571450 | 2021-02-11 10:01:08 -0800 | [diff] [blame] | 88 | t.setCrop(mColorLayer, Rect(0, 0, 30, 40)); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 89 | t.setLayer(mColorLayer, INT32_MAX - 2); |
| 90 | t.setColor(mColorLayer, |
| 91 | half3{mExpectedColor.r / 255.0f, mExpectedColor.g / 255.0f, |
| 92 | mExpectedColor.b / 255.0f}); |
| 93 | t.show(mColorLayer); |
| 94 | }); |
| 95 | } |
| 96 | |
Dominik Laskowski | 3cb3d4e | 2019-11-21 11:14:45 -0800 | [diff] [blame] | 97 | ui::DisplayState mMainDisplayState; |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 98 | ui::DisplayMode mMainDisplayMode; |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 99 | sp<IBinder> mMainDisplay; |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 100 | PhysicalDisplayId mMainDisplayId; |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 101 | sp<IBinder> mVirtualDisplay; |
Jim Shargo | f48bef8 | 2024-12-04 18:56:59 +0000 | [diff] [blame^] | 102 | sp<IGraphicBufferConsumer> mConsumer; |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 103 | sp<IGraphicBufferProducer> mProducer; |
| 104 | sp<SurfaceControl> mColorLayer; |
| 105 | Color mExpectedColor = {63, 63, 195, 255}; |
| 106 | }; |
| 107 | |
| 108 | TEST_F(MultiDisplayLayerBoundsTest, RenderLayerInVirtualDisplay) { |
Dominik Laskowski | 29fa146 | 2021-04-27 15:51:50 -0700 | [diff] [blame] | 109 | constexpr ui::LayerStack kLayerStack{1u}; |
| 110 | createDisplay(mMainDisplayState.layerStackSpaceRect, kLayerStack); |
| 111 | createColorLayer(kLayerStack); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 112 | |
| 113 | asTransaction([&](Transaction& t) { t.setPosition(mColorLayer, 10, 10); }); |
| 114 | |
| 115 | // Verify color layer does not render on main display. |
| 116 | std::unique_ptr<ScreenCapture> sc; |
| 117 | ScreenCapture::captureScreen(&sc, mMainDisplay); |
| 118 | sc->expectColor(Rect(10, 10, 40, 50), {0, 0, 0, 255}); |
| 119 | sc->expectColor(Rect(0, 0, 9, 9), {0, 0, 0, 255}); |
| 120 | |
| 121 | // Verify color layer renders correctly on virtual display. |
| 122 | ScreenCapture::captureScreen(&sc, mVirtualDisplay); |
| 123 | sc->expectColor(Rect(10, 10, 40, 50), mExpectedColor); |
Alec Mouri | 5a6d857 | 2020-03-23 23:56:15 -0700 | [diff] [blame] | 124 | sc->expectColor(Rect(1, 1, 9, 9), {0, 0, 0, 255}); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 125 | } |
| 126 | |
| 127 | TEST_F(MultiDisplayLayerBoundsTest, RenderLayerInMirroredVirtualDisplay) { |
| 128 | // Create a display and set its layer stack to the main display's layer stack so |
| 129 | // the contents of the main display are mirrored on to the virtual display. |
| 130 | |
Marin Shalamanov | 6ad317c | 2020-07-29 23:34:07 +0200 | [diff] [blame] | 131 | // Assumption here is that the new mirrored display has the same layer stack rect as the |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 132 | // primary display that it is mirroring. |
Dominik Laskowski | 29fa146 | 2021-04-27 15:51:50 -0700 | [diff] [blame] | 133 | createDisplay(mMainDisplayState.layerStackSpaceRect, ui::DEFAULT_LAYER_STACK); |
| 134 | createColorLayer(ui::DEFAULT_LAYER_STACK); |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 135 | |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 136 | sp<SurfaceControl> mirrorSc = |
| 137 | SurfaceComposerClient::getDefault()->mirrorDisplay(mMainDisplayId); |
| 138 | |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 139 | asTransaction([&](Transaction& t) { t.setPosition(mColorLayer, 10, 10); }); |
| 140 | |
| 141 | // Verify color layer renders correctly on main display and it is mirrored on the |
| 142 | // virtual display. |
| 143 | std::unique_ptr<ScreenCapture> sc; |
| 144 | ScreenCapture::captureScreen(&sc, mMainDisplay); |
| 145 | sc->expectColor(Rect(10, 10, 40, 50), mExpectedColor); |
| 146 | sc->expectColor(Rect(0, 0, 9, 9), {0, 0, 0, 255}); |
| 147 | |
| 148 | ScreenCapture::captureScreen(&sc, mVirtualDisplay); |
| 149 | sc->expectColor(Rect(10, 10, 40, 50), mExpectedColor); |
| 150 | sc->expectColor(Rect(0, 0, 9, 9), {0, 0, 0, 255}); |
| 151 | } |
| 152 | |
Melody Hsu | 793f836 | 2024-01-08 20:00:35 +0000 | [diff] [blame] | 153 | TEST_F(MultiDisplayLayerBoundsTest, RenderLayerWithPromisedFenceInMirroredVirtualDisplay) { |
| 154 | // Create a display and use a unique layerstack ID for mirrorDisplay() so |
| 155 | // the contents of the main display are mirrored on to the virtual display. |
| 156 | |
| 157 | // A unique layerstack ID must be used because sharing the same layerFE |
| 158 | // with more than one display is unsupported. A unique layerstack ensures |
| 159 | // that a different layerFE is used between displays. |
| 160 | constexpr ui::LayerStack layerStack{77687666}; // ASCII for MDLB (MultiDisplayLayerBounds) |
| 161 | createDisplay(mMainDisplayState.layerStackSpaceRect, layerStack); |
| 162 | createColorLayer(ui::DEFAULT_LAYER_STACK); |
| 163 | |
| 164 | sp<SurfaceControl> mirrorSc = |
| 165 | SurfaceComposerClient::getDefault()->mirrorDisplay(mMainDisplayId); |
| 166 | |
| 167 | asTransaction([&](Transaction& t) { |
| 168 | t.setPosition(mColorLayer, 10, 10); |
| 169 | t.setLayerStack(mirrorSc, layerStack); |
| 170 | }); |
| 171 | |
| 172 | // Verify color layer renders correctly on main display and it is mirrored on the |
| 173 | // virtual display. |
| 174 | std::unique_ptr<ScreenCapture> sc; |
| 175 | ScreenCapture::captureScreen(&sc, mMainDisplay); |
| 176 | sc->expectColor(Rect(10, 10, 40, 50), mExpectedColor); |
| 177 | sc->expectColor(Rect(0, 0, 9, 9), {0, 0, 0, 255}); |
| 178 | |
| 179 | ScreenCapture::captureScreen(&sc, mVirtualDisplay); |
| 180 | sc->expectColor(Rect(10, 10, 40, 50), mExpectedColor); |
| 181 | sc->expectColor(Rect(0, 0, 9, 9), {0, 0, 0, 255}); |
| 182 | } |
| 183 | |
Valerie Hau | 9cfc6d8 | 2019-09-23 13:54:07 -0700 | [diff] [blame] | 184 | } // namespace android |
Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 185 | |
| 186 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 187 | #pragma clang diagnostic pop // ignored "-Wconversion" |