| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright (C) 2011 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 |  | 
|  | 17 | #include <gtest/gtest.h> | 
|  | 18 |  | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 19 | #include <android/native_window.h> | 
|  | 20 |  | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 21 | #include <binder/IMemory.h> | 
| Mathias Agopian | 90ac799 | 2012-02-25 18:48:35 -0800 | [diff] [blame] | 22 |  | 
|  | 23 | #include <gui/ISurfaceComposer.h> | 
|  | 24 | #include <gui/Surface.h> | 
|  | 25 | #include <gui/SurfaceComposerClient.h> | 
|  | 26 | #include <private/gui/ComposerService.h> | 
|  | 27 |  | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 28 | #include <utils/String8.h> | 
| Mathias Agopian | c666cae | 2012-07-25 18:56:13 -0700 | [diff] [blame] | 29 | #include <ui/DisplayInfo.h> | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 30 |  | 
|  | 31 | namespace android { | 
|  | 32 |  | 
|  | 33 | // Fill an RGBA_8888 formatted surface with a single color. | 
|  | 34 | static void fillSurfaceRGBA8(const sp<SurfaceControl>& sc, | 
|  | 35 | uint8_t r, uint8_t g, uint8_t b) { | 
| Mathias Agopian | e3c697f | 2013-02-14 17:11:02 -0800 | [diff] [blame] | 36 | ANativeWindow_Buffer outBuffer; | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 37 | sp<Surface> s = sc->getSurface(); | 
|  | 38 | ASSERT_TRUE(s != NULL); | 
| Mathias Agopian | e3c697f | 2013-02-14 17:11:02 -0800 | [diff] [blame] | 39 | ASSERT_EQ(NO_ERROR, s->lock(&outBuffer, NULL)); | 
|  | 40 | uint8_t* img = reinterpret_cast<uint8_t*>(outBuffer.bits); | 
|  | 41 | for (uint32_t y = 0; y < outBuffer.height; y++) { | 
|  | 42 | for (uint32_t x = 0; x < outBuffer.width; x++) { | 
|  | 43 | uint8_t* pixel = img + (4 * (y*outBuffer.stride + x)); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 44 | pixel[0] = r; | 
|  | 45 | pixel[1] = g; | 
|  | 46 | pixel[2] = b; | 
|  | 47 | pixel[3] = 255; | 
|  | 48 | } | 
|  | 49 | } | 
|  | 50 | ASSERT_EQ(NO_ERROR, s->unlockAndPost()); | 
|  | 51 | } | 
|  | 52 |  | 
|  | 53 | // A ScreenCapture is a screenshot from SurfaceFlinger that can be used to check | 
|  | 54 | // individual pixel values for testing purposes. | 
|  | 55 | class ScreenCapture : public RefBase { | 
|  | 56 | public: | 
|  | 57 | static void captureScreen(sp<ScreenCapture>* sc) { | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 58 | sp<IGraphicBufferProducer> producer; | 
|  | 59 | sp<IGraphicBufferConsumer> consumer; | 
|  | 60 | BufferQueue::createBufferQueue(&producer, &consumer); | 
|  | 61 | IGraphicBufferProducer::QueueBufferOutput bufferOutput; | 
|  | 62 | sp<CpuConsumer> cpuConsumer = new CpuConsumer(consumer, 1); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 63 | sp<ISurfaceComposer> sf(ComposerService::getComposerService()); | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 64 | sp<IBinder> display(sf->getBuiltInDisplay( | 
|  | 65 | ISurfaceComposer::eDisplayIdMain)); | 
|  | 66 | ASSERT_EQ(NO_ERROR, sf->captureScreen(display, producer, Rect(), 0, 0, | 
|  | 67 | 0, INT_MAX, false)); | 
|  | 68 | *sc = new ScreenCapture(cpuConsumer); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 69 | } | 
|  | 70 |  | 
|  | 71 | void checkPixel(uint32_t x, uint32_t y, uint8_t r, uint8_t g, uint8_t b) { | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 72 | ASSERT_EQ(HAL_PIXEL_FORMAT_RGBA_8888, mBuf.format); | 
|  | 73 | const uint8_t* img = static_cast<const uint8_t*>(mBuf.data); | 
|  | 74 | const uint8_t* pixel = img + (4 * (y * mBuf.stride + x)); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 75 | if (r != pixel[0] || g != pixel[1] || b != pixel[2]) { | 
|  | 76 | String8 err(String8::format("pixel @ (%3d, %3d): " | 
|  | 77 | "expected [%3d, %3d, %3d], got [%3d, %3d, %3d]", | 
|  | 78 | x, y, r, g, b, pixel[0], pixel[1], pixel[2])); | 
|  | 79 | EXPECT_EQ(String8(), err); | 
|  | 80 | } | 
|  | 81 | } | 
|  | 82 |  | 
|  | 83 | private: | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 84 | ScreenCapture(const sp<CpuConsumer>& cc) : | 
|  | 85 | mCC(cc) { | 
|  | 86 | EXPECT_EQ(NO_ERROR, mCC->lockNextBuffer(&mBuf)); | 
|  | 87 | } | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 88 |  | 
| Michael Lentine | 5a16a62 | 2015-05-21 13:48:24 -0700 | [diff] [blame] | 89 | ~ScreenCapture() { | 
|  | 90 | mCC->unlockBuffer(mBuf); | 
|  | 91 | } | 
|  | 92 |  | 
|  | 93 | sp<CpuConsumer> mCC; | 
|  | 94 | CpuConsumer::LockedBuffer mBuf; | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 95 | }; | 
|  | 96 |  | 
|  | 97 | class LayerUpdateTest : public ::testing::Test { | 
|  | 98 | protected: | 
|  | 99 | virtual void SetUp() { | 
|  | 100 | mComposerClient = new SurfaceComposerClient; | 
|  | 101 | ASSERT_EQ(NO_ERROR, mComposerClient->initCheck()); | 
|  | 102 |  | 
| Jeff Brown | 9d4e3d2 | 2012-08-24 20:00:51 -0700 | [diff] [blame] | 103 | sp<IBinder> display(SurfaceComposerClient::getBuiltInDisplay( | 
|  | 104 | ISurfaceComposer::eDisplayIdMain)); | 
| Mathias Agopian | c666cae | 2012-07-25 18:56:13 -0700 | [diff] [blame] | 105 | DisplayInfo info; | 
| Jeff Brown | 9d4e3d2 | 2012-08-24 20:00:51 -0700 | [diff] [blame] | 106 | SurfaceComposerClient::getDisplayInfo(display, &info); | 
| Mathias Agopian | c666cae | 2012-07-25 18:56:13 -0700 | [diff] [blame] | 107 |  | 
|  | 108 | ssize_t displayWidth = info.w; | 
|  | 109 | ssize_t displayHeight = info.h; | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 110 |  | 
|  | 111 | // Background surface | 
|  | 112 | mBGSurfaceControl = mComposerClient->createSurface( | 
| Jeff Brown | 9d4e3d2 | 2012-08-24 20:00:51 -0700 | [diff] [blame] | 113 | String8("BG Test Surface"), displayWidth, displayHeight, | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 114 | PIXEL_FORMAT_RGBA_8888, 0); | 
|  | 115 | ASSERT_TRUE(mBGSurfaceControl != NULL); | 
|  | 116 | ASSERT_TRUE(mBGSurfaceControl->isValid()); | 
|  | 117 | fillSurfaceRGBA8(mBGSurfaceControl, 63, 63, 195); | 
|  | 118 |  | 
|  | 119 | // Foreground surface | 
|  | 120 | mFGSurfaceControl = mComposerClient->createSurface( | 
| Jeff Brown | 9d4e3d2 | 2012-08-24 20:00:51 -0700 | [diff] [blame] | 121 | String8("FG Test Surface"), 64, 64, PIXEL_FORMAT_RGBA_8888, 0); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 122 | ASSERT_TRUE(mFGSurfaceControl != NULL); | 
|  | 123 | ASSERT_TRUE(mFGSurfaceControl->isValid()); | 
|  | 124 |  | 
|  | 125 | fillSurfaceRGBA8(mFGSurfaceControl, 195, 63, 63); | 
|  | 126 |  | 
|  | 127 | // Synchronization surface | 
|  | 128 | mSyncSurfaceControl = mComposerClient->createSurface( | 
| Jeff Brown | 9d4e3d2 | 2012-08-24 20:00:51 -0700 | [diff] [blame] | 129 | String8("Sync Test Surface"), 1, 1, PIXEL_FORMAT_RGBA_8888, 0); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 130 | ASSERT_TRUE(mSyncSurfaceControl != NULL); | 
|  | 131 | ASSERT_TRUE(mSyncSurfaceControl->isValid()); | 
|  | 132 |  | 
|  | 133 | fillSurfaceRGBA8(mSyncSurfaceControl, 31, 31, 31); | 
|  | 134 |  | 
|  | 135 | SurfaceComposerClient::openGlobalTransaction(); | 
|  | 136 |  | 
|  | 137 | ASSERT_EQ(NO_ERROR, mBGSurfaceControl->setLayer(INT_MAX-2)); | 
|  | 138 | ASSERT_EQ(NO_ERROR, mBGSurfaceControl->show()); | 
|  | 139 |  | 
|  | 140 | ASSERT_EQ(NO_ERROR, mFGSurfaceControl->setLayer(INT_MAX-1)); | 
|  | 141 | ASSERT_EQ(NO_ERROR, mFGSurfaceControl->setPosition(64, 64)); | 
|  | 142 | ASSERT_EQ(NO_ERROR, mFGSurfaceControl->show()); | 
|  | 143 |  | 
|  | 144 | ASSERT_EQ(NO_ERROR, mSyncSurfaceControl->setLayer(INT_MAX-1)); | 
|  | 145 | ASSERT_EQ(NO_ERROR, mSyncSurfaceControl->setPosition(displayWidth-2, | 
|  | 146 | displayHeight-2)); | 
|  | 147 | ASSERT_EQ(NO_ERROR, mSyncSurfaceControl->show()); | 
|  | 148 |  | 
|  | 149 | SurfaceComposerClient::closeGlobalTransaction(true); | 
|  | 150 | } | 
|  | 151 |  | 
|  | 152 | virtual void TearDown() { | 
|  | 153 | mComposerClient->dispose(); | 
|  | 154 | mBGSurfaceControl = 0; | 
|  | 155 | mFGSurfaceControl = 0; | 
|  | 156 | mSyncSurfaceControl = 0; | 
|  | 157 | mComposerClient = 0; | 
|  | 158 | } | 
|  | 159 |  | 
|  | 160 | void waitForPostedBuffers() { | 
|  | 161 | // Since the sync surface is in synchronous mode (i.e. double buffered) | 
|  | 162 | // posting three buffers to it should ensure that at least two | 
|  | 163 | // SurfaceFlinger::handlePageFlip calls have been made, which should | 
|  | 164 | // guaranteed that a buffer posted to another Surface has been retired. | 
|  | 165 | fillSurfaceRGBA8(mSyncSurfaceControl, 31, 31, 31); | 
|  | 166 | fillSurfaceRGBA8(mSyncSurfaceControl, 31, 31, 31); | 
|  | 167 | fillSurfaceRGBA8(mSyncSurfaceControl, 31, 31, 31); | 
|  | 168 | } | 
|  | 169 |  | 
|  | 170 | sp<SurfaceComposerClient> mComposerClient; | 
|  | 171 | sp<SurfaceControl> mBGSurfaceControl; | 
|  | 172 | sp<SurfaceControl> mFGSurfaceControl; | 
|  | 173 |  | 
|  | 174 | // This surface is used to ensure that the buffers posted to | 
|  | 175 | // mFGSurfaceControl have been picked up by SurfaceFlinger. | 
|  | 176 | sp<SurfaceControl> mSyncSurfaceControl; | 
|  | 177 | }; | 
|  | 178 |  | 
|  | 179 | TEST_F(LayerUpdateTest, LayerMoveWorks) { | 
|  | 180 | sp<ScreenCapture> sc; | 
|  | 181 | { | 
|  | 182 | SCOPED_TRACE("before move"); | 
|  | 183 | ScreenCapture::captureScreen(&sc); | 
|  | 184 | sc->checkPixel(  0,  12,  63,  63, 195); | 
|  | 185 | sc->checkPixel( 75,  75, 195,  63,  63); | 
|  | 186 | sc->checkPixel(145, 145,  63,  63, 195); | 
|  | 187 | } | 
|  | 188 |  | 
|  | 189 | SurfaceComposerClient::openGlobalTransaction(); | 
|  | 190 | ASSERT_EQ(NO_ERROR, mFGSurfaceControl->setPosition(128, 128)); | 
|  | 191 | SurfaceComposerClient::closeGlobalTransaction(true); | 
|  | 192 | { | 
|  | 193 | // This should reflect the new position, but not the new color. | 
|  | 194 | SCOPED_TRACE("after move, before redraw"); | 
|  | 195 | ScreenCapture::captureScreen(&sc); | 
|  | 196 | sc->checkPixel( 24,  24,  63,  63, 195); | 
|  | 197 | sc->checkPixel( 75,  75,  63,  63, 195); | 
|  | 198 | sc->checkPixel(145, 145, 195,  63,  63); | 
|  | 199 | } | 
|  | 200 |  | 
|  | 201 | fillSurfaceRGBA8(mFGSurfaceControl, 63, 195, 63); | 
|  | 202 | waitForPostedBuffers(); | 
|  | 203 | { | 
|  | 204 | // This should reflect the new position and the new color. | 
|  | 205 | SCOPED_TRACE("after redraw"); | 
|  | 206 | ScreenCapture::captureScreen(&sc); | 
|  | 207 | sc->checkPixel( 24,  24,  63,  63, 195); | 
|  | 208 | sc->checkPixel( 75,  75,  63,  63, 195); | 
|  | 209 | sc->checkPixel(145, 145,  63, 195,  63); | 
|  | 210 | } | 
|  | 211 | } | 
|  | 212 |  | 
|  | 213 | TEST_F(LayerUpdateTest, LayerResizeWorks) { | 
|  | 214 | sp<ScreenCapture> sc; | 
|  | 215 | { | 
|  | 216 | SCOPED_TRACE("before resize"); | 
|  | 217 | ScreenCapture::captureScreen(&sc); | 
|  | 218 | sc->checkPixel(  0,  12,  63,  63, 195); | 
|  | 219 | sc->checkPixel( 75,  75, 195,  63,  63); | 
|  | 220 | sc->checkPixel(145, 145,  63,  63, 195); | 
|  | 221 | } | 
|  | 222 |  | 
| Steve Block | 9d45368 | 2011-12-20 16:23:08 +0000 | [diff] [blame] | 223 | ALOGD("resizing"); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 224 | SurfaceComposerClient::openGlobalTransaction(); | 
|  | 225 | ASSERT_EQ(NO_ERROR, mFGSurfaceControl->setSize(128, 128)); | 
|  | 226 | SurfaceComposerClient::closeGlobalTransaction(true); | 
| Steve Block | 9d45368 | 2011-12-20 16:23:08 +0000 | [diff] [blame] | 227 | ALOGD("resized"); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 228 | { | 
|  | 229 | // This should not reflect the new size or color because SurfaceFlinger | 
|  | 230 | // has not yet received a buffer of the correct size. | 
|  | 231 | SCOPED_TRACE("after resize, before redraw"); | 
|  | 232 | ScreenCapture::captureScreen(&sc); | 
|  | 233 | sc->checkPixel(  0,  12,  63,  63, 195); | 
|  | 234 | sc->checkPixel( 75,  75, 195,  63,  63); | 
|  | 235 | sc->checkPixel(145, 145,  63,  63, 195); | 
|  | 236 | } | 
|  | 237 |  | 
| Steve Block | 9d45368 | 2011-12-20 16:23:08 +0000 | [diff] [blame] | 238 | ALOGD("drawing"); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 239 | fillSurfaceRGBA8(mFGSurfaceControl, 63, 195, 63); | 
|  | 240 | waitForPostedBuffers(); | 
| Steve Block | 9d45368 | 2011-12-20 16:23:08 +0000 | [diff] [blame] | 241 | ALOGD("drawn"); | 
| Jamie Gennis | 23c2c5d | 2011-10-11 19:22:19 -0700 | [diff] [blame] | 242 | { | 
|  | 243 | // This should reflect the new size and the new color. | 
|  | 244 | SCOPED_TRACE("after redraw"); | 
|  | 245 | ScreenCapture::captureScreen(&sc); | 
|  | 246 | sc->checkPixel( 24,  24,  63,  63, 195); | 
|  | 247 | sc->checkPixel( 75,  75,  63, 195,  63); | 
|  | 248 | sc->checkPixel(145, 145,  63, 195,  63); | 
|  | 249 | } | 
|  | 250 | } | 
|  | 251 |  | 
|  | 252 | } |