| commit | df084f35fc18077b4e6026e8708a64e842d611c9 | [log] [tgz] |
|---|---|---|
| author | Ady Abraham <adyabr@google.com> | Mon Jan 10 14:10:23 2022 -0800 |
| committer | Ady Abraham <adyabr@google.com> | Tue Jan 11 00:39:00 2022 +0000 |
| tree | 68c6918c4a641abfcbf345d36f3109c47a36196e | |
| parent | e57e0962068389d0feab9ac36435b729d8f3efc1 [diff] |
composer: deflake Test_expectedPresentTime Test_expectedPresentTime might fail due to scheduling delays. To avoid that, we test that the actual presentation didn't happen too early, but we allow for a late presentation. Bug: 198186194 Test: VTS Change-Id: I9be4bfe5a848443e69ef7b62ffb3dae675a9b81f
diff --git a/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_TargetTest.cpp b/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_TargetTest.cpp index a591aaa..24b742e 100644 --- a/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_TargetTest.cpp +++ b/graphics/composer/aidl/android/hardware/graphics/composer3/vts/functional/VtsHalGraphicsComposer3_TargetTest.cpp
@@ -1426,8 +1426,7 @@ presentFence2->waitForever(LOG_TAG); const auto actualPresentTime = presentFence2->getSignalTime(); - const auto presentError = std::abs(expectedPresentTime - actualPresentTime); - EXPECT_LE(presentError, vsyncPeriod / 2); + EXPECT_GE(actualPresentTime, expectedPresentTime - vsyncPeriod / 2); ASSERT_TRUE(mComposerClient->setPowerMode(mPrimaryDisplay, PowerMode::OFF).isOk()); }