| commit | 62ae37be3cbab314076c531435f52db91c3cc061 | [log] [tgz] |
|---|---|---|
| author | Alec Mouri <alecmouri@google.com> | Thu Jan 20 17:16:38 2022 -0800 |
| committer | Alec Mouri <alecmouri@google.com> | Fri Jan 21 19:42:32 2022 +0000 |
| tree | af708a6a2377610d3a3a48767de81ca2a524d57d | |
| parent | 7b4d5ee452aa6406afaf626c0626c73c41278bed [diff] |
Fix asserts for GetReadbackBufferFenceInactive If there's no readback buffer, then the status is NOT ok(), and the returned fence must be -1. Bug: 213493262 Test: VtsHalGraphicsComposer3_TargetTest Change-Id: Iff1b6e04126bfcc748f0f8d173f63244bfd83be4
diff --git a/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_ReadbackTest.cpp b/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_ReadbackTest.cpp index fc527ec..3f1e703 100644 --- a/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_ReadbackTest.cpp +++ b/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_ReadbackTest.cpp
@@ -521,8 +521,9 @@ ndk::ScopedFileDescriptor releaseFence; const auto error = mComposerClient->getReadbackBufferFence(mPrimaryDisplay, &releaseFence); - EXPECT_TRUE(error.isOk()); + ASSERT_FALSE(error.isOk()); EXPECT_EQ(IComposerClient::EX_UNSUPPORTED, error.getServiceSpecificError()); + EXPECT_EQ(-1, releaseFence.get()); } TEST_P(GraphicsCompositionTest, ClientComposition) {