tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Revert "VTS: Test that configs in a group differ only by vsync period"

Browse source 
This reverts commit 6f36dd6e31.

Differing only by vsync period was not a requirement on Android R, and
it shouldn't be added after-the-fact to an existing HAL version.

There are devices on the market that switch seamlessly between configs
that differ in resolution/dpi, and therefore have those configs in a
common group.

Bug: 200184776
Change-Id: Id0bfc67e55d5139fddb2b359cabafd9281c33734
This commit is contained in:
Snild Dolkow 2021-09-16 10:20:45 +02:00 committed by Marin Shalamanov
parent 5dc5f2633f
commit bf484a6adb
1 changed files with 0 additions and 55 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

55
graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp
View file

@ -19,8 +19,6 @@
#include <algorithm> #include <algorithm>
#include <regex> #include <regex>
#include <thread> #include <thread>
#include <unordered_map>
#include <utility>
#include <android-base/logging.h> #include <android-base/logging.h>
#include <android-base/properties.h> #include <android-base/properties.h>
@ -317,59 +315,6 @@ TEST_P(GraphicsComposerHidlTest, GetDisplayAttribute_2_4) {
} }
} }
TEST_P(GraphicsComposerHidlTest, GetDisplayAttribute_2_4_ConfigsInAGroupDifferOnlyByVsyncPeriod) {
struct Resolution {
int32_t width, height;
};
struct Dpi {
int32_t x, y;
};
for (const auto& display : mDisplays) {
std::vector<Config> configs = mComposerClient->getDisplayConfigs(display.get());
std::unordered_map<int32_t, Resolution> configGroupToResolutionMap;
std::unordered_map<int32_t, Dpi> configGroupToDpiMap;
for (auto config : configs) {
const auto configGroup = mComposerClient->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::CONFIG_GROUP);
const auto width = mComposerClient->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::WIDTH);
const auto height = mComposerClient->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::HEIGHT);
if (configGroupToResolutionMap.find(configGroup) == configGroupToResolutionMap.end()) {
configGroupToResolutionMap[configGroup] = {width, height};
}
EXPECT_EQ(configGroupToResolutionMap[configGroup].width, width);
EXPECT_EQ(configGroupToResolutionMap[configGroup].height, height);
int32_t dpiX = -1;
mComposerClient->getRaw()->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::DPI_X,
[&](const auto& tmpError, const auto& value) {
if (tmpError == Error::NONE) {
dpiX = value;
}
});
int32_t dpiY = -1;
mComposerClient->getRaw()->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::DPI_Y,
[&](const auto& tmpError, const auto& value) {
if (tmpError == Error::NONE) {
dpiY = value;
}
});
if (dpiX == -1 && dpiY == -1) {
continue;
}
if (configGroupToDpiMap.find(configGroup) == configGroupToDpiMap.end()) {
configGroupToDpiMap[configGroup] = {dpiX, dpiY};
}
EXPECT_EQ(configGroupToDpiMap[configGroup].x, dpiX);
EXPECT_EQ(configGroupToDpiMap[configGroup].y, dpiY);
}
}
}
TEST_P(GraphicsComposerHidlTest, getDisplayVsyncPeriod_BadDisplay) { TEST_P(GraphicsComposerHidlTest, getDisplayVsyncPeriod_BadDisplay) {
VsyncPeriodNanos vsyncPeriodNanos; VsyncPeriodNanos vsyncPeriodNanos;
EXPECT_EQ(Error::BAD_DISPLAY, EXPECT_EQ(Error::BAD_DISPLAY,