Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-01 21:37:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

composer: VtsDisplay::setDimensions shouldn't be const am: 2fff170562

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1686815

Change-Id: I7f7a949e8b1474c88d927002c3ef1e6bab671a3e
This commit is contained in:
Ady Abraham
2021-04-26 20:49:44 +00:00
committed by Automerger Merge Worker
parent d6537b480a 2fff170562
commit e8bc48b777
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
graphics/composer/2.4/vts/functional/VtsHalGraphicsComposerV2_4TargetTest.cpp
View File

@@ -73,15 +73,15 @@ class VtsDisplay {
IComposerClient::Rect getFrameRect() const { return {0, 0, mDisplayWidth, mDisplayHeight}; }
void setDimensions(int32_t displayWidth, int32_t displayHeight) const {
void setDimensions(int32_t displayWidth, int32_t displayHeight) {
mDisplayWidth = displayWidth;
mDisplayHeight = displayHeight;
}
private:
const Display mDisplay;
mutable int32_t mDisplayWidth;
mutable int32_t mDisplayHeight;
int32_t mDisplayWidth;
int32_t mDisplayHeight;
};
class GraphicsComposerHidlTest : public ::testing::TestWithParam<std::string> {
@@ -200,7 +200,7 @@ class GraphicsComposerHidlTest : public ::testing::TestWithParam<std::string> {
const ContentType& contentType, const char* contentTypeStr);
Error setActiveConfigWithConstraints(
const VtsDisplay& display, Config config,
VtsDisplay& display, Config config,
const IComposerClient::VsyncPeriodChangeConstraints& constraints,
VsyncPeriodChangeTimeline* timeline) {
const auto error = mComposerClient->setActiveConfigWithConstraints(display.get(), config,
@@ -215,7 +215,7 @@ class GraphicsComposerHidlTest : public ::testing::TestWithParam<std::string> {
return error;
}
void setActiveConfig(const VtsDisplay& display, Config config) {
void setActiveConfig(VtsDisplay& display, Config config) {
mComposerClient->setActiveConfig(display.get(), config);
const int32_t displayWidth = mComposerClient->getDisplayAttribute_2_4(
display.get(), config, IComposerClient::Attribute::WIDTH);
@@ -377,7 +377,7 @@ TEST_P(GraphicsComposerHidlTest, getDisplayVsyncPeriod_BadDisplay) {
}
TEST_P(GraphicsComposerHidlTest, getDisplayVsyncPeriod) {
for (const auto& display : mDisplays) {
for (VtsDisplay& display : mDisplays) {
for (Config config : mComposerClient->getDisplayConfigs(display.get())) {
VsyncPeriodNanos expectedVsyncPeriodNanos = mComposerClient->getDisplayAttribute_2_4(
display.get(), config,
@@ -441,7 +441,7 @@ TEST_P(GraphicsComposerHidlTest, setActiveConfigWithConstraints_BadConfig) {
constraints.seamlessRequired = false;
constraints.desiredTimeNanos = systemTime();
for (const auto& display : mDisplays) {
for (VtsDisplay& display : mDisplays) {
Config invalidConfigId = GetInvalidConfigId(display.get());
EXPECT_EQ(Error::BAD_CONFIG,
setActiveConfigWithConstraints(display, invalidConfigId, constraints, &timeline));
@@ -455,7 +455,7 @@ TEST_P(GraphicsComposerHidlTest, setActiveConfigWithConstraints_SeamlessNotAllow
constraints.seamlessRequired = true;
constraints.desiredTimeNanos = systemTime();
for (const auto& display : mDisplays) {
for (VtsDisplay& display : mDisplays) {
forEachTwoConfigs(display.get(), [&](Config config1, Config config2) {
const auto configGroup1 = mComposerClient->getDisplayAttribute_2_4(
display.get(), config1,
@@ -553,7 +553,7 @@ void GraphicsComposerHidlTest::waitForVsyncPeriodChange(Display display,
}
void GraphicsComposerHidlTest::Test_setActiveConfigWithConstraints(const TestParameters& params) {
for (const auto& display : mDisplays) {
for (VtsDisplay& display : mDisplays) {
forEachTwoConfigs(display.get(), [&](Config config1, Config config2) {
setActiveConfig(display, config1);
sendRefreshFrame(display, nullptr);