From 95e490bf23c82407d8c21774cb0933b182b5e989 Mon Sep 17 00:00:00 2001 From: Alec Mouri Date: Thu, 6 Jan 2022 13:46:58 -0800 Subject: [PATCH] Add VTS for setting layer white point No readback tests as of yet, as that will necessitate a new api to get the max display brightness, so that we can max out the brightness -> use well-defined dimming ratios Bug: 213493262 Test: builds Change-Id: I0166dc67f4c52919c02ab7abbf47701e6be3d983 --- .../VtsHalGraphicsComposer3_TargetTest.cpp | 21 +++++++++++++++++++ 1 file changed, 21 insertions(+) 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 829b0ff66f..928e506c9e 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 @@ -1922,6 +1922,27 @@ TEST_P(GraphicsComposerAidlCommandTest, SET_LAYER_PER_FRAME_METADATA) { EXPECT_TRUE(mComposerClient->destroyLayer(mPrimaryDisplay, layer).isOk()); } +TEST_P(GraphicsComposerAidlCommandTest, setLayerWhitePointNits) { + int64_t layer; + EXPECT_TRUE(mComposerClient->createLayer(mPrimaryDisplay, kBufferSlotCount, &layer).isOk()); + + mWriter.setLayerWhitePointNits(mPrimaryDisplay, layer, 200.f); + execute(); + ASSERT_TRUE(mReader.takeErrors().empty()); + + mWriter.setLayerWhitePointNits(mPrimaryDisplay, layer, 1000.f); + execute(); + ASSERT_TRUE(mReader.takeErrors().empty()); + + mWriter.setLayerWhitePointNits(mPrimaryDisplay, layer, 0.f); + execute(); + ASSERT_TRUE(mReader.takeErrors().empty()); + + mWriter.setLayerWhitePointNits(mPrimaryDisplay, layer, -1.f); + execute(); + ASSERT_TRUE(mReader.takeErrors().empty()); +} + TEST_P(GraphicsComposerAidlCommandTest, setActiveConfigWithConstraints) { Test_setActiveConfigWithConstraints({.delayForChange = 0, .refreshMiss = false}); }