From 0a19601eaedf42cf8270ffb113a990be19a2c101 Mon Sep 17 00:00:00 2001 From: Ady Abraham Date: Mon, 13 Jul 2020 16:27:15 -0700 Subject: [PATCH] composer: 2.4: VTS using old value for parameter Fix a test that is using the incorrect parameter. Bug: 161023185 Test: adb shell data/nativetest64/VtsHalGraphicsComposerV2_4TargetTest/VtsHalGraphicsComposerV2_4TargetTest Change-Id: I8574abed50c23657f5d96e9d396709cb9b92e1fd --- .../VtsHalGraphicsComposerV2_4TargetTest.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp b/graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp index 00df7c78f3..6b0d106075 100644 --- a/graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp +++ b/graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp @@ -493,16 +493,16 @@ void GraphicsComposerHidlCommandTest::Test_setActiveConfigWithConstraints( // At this point the refresh rate should have changed already, however in rare // cases the implementation might have missed the deadline. In this case a new // timeline should have been provided. - auto newTimelime = mComposerCallback->takeLastVsyncPeriodChangeTimeline(); + auto newTimeline = mComposerCallback->takeLastVsyncPeriodChangeTimeline(); if (timeline.refreshRequired && refreshMiss) { - EXPECT_TRUE(newTimelime.has_value()); + EXPECT_TRUE(newTimeline.has_value()); } - if (newTimelime.has_value()) { - if (timeline.refreshRequired) { - sendRefreshFrame(&newTimelime.value()); + if (newTimeline.has_value()) { + if (newTimeline->refreshRequired) { + sendRefreshFrame(&newTimeline.value()); } - waitForVsyncPeriodChange(display, newTimelime.value(), constraints.desiredTimeNanos, + waitForVsyncPeriodChange(display, newTimeline.value(), constraints.desiredTimeNanos, vsyncPeriod1, vsyncPeriod2); }