SF: populate state correctly in Display::applyLayerRequestsToLayers
Fix a bug where we update a local copy of the state instead of updating it.
Bug: 189076031
Test: db shell /data/nativetest64/libcompositionengine_test/libcompositionengine_test
Change-Id: I15bd19e3ba468e39a938584246a02337087fa612
diff --git a/services/surfaceflinger/CompositionEngine/src/Display.cpp b/services/surfaceflinger/CompositionEngine/src/Display.cpp
index 953eb76..3d49183 100644
--- a/services/surfaceflinger/CompositionEngine/src/Display.cpp
+++ b/services/surfaceflinger/CompositionEngine/src/Display.cpp
@@ -344,8 +344,8 @@
if (clientTargetProperty.dataspace == ui::Dataspace::UNKNOWN) {
return;
}
- auto outputState = editState();
- outputState.dataspace = clientTargetProperty.dataspace;
+
+ editState().dataspace = clientTargetProperty.dataspace;
getRenderSurface()->setBufferDataspace(clientTargetProperty.dataspace);
getRenderSurface()->setBufferPixelFormat(clientTargetProperty.pixelFormat);
}
diff --git a/services/surfaceflinger/CompositionEngine/tests/DisplayTest.cpp b/services/surfaceflinger/CompositionEngine/tests/DisplayTest.cpp
index e12cb57..ac3ba0d 100644
--- a/services/surfaceflinger/CompositionEngine/tests/DisplayTest.cpp
+++ b/services/surfaceflinger/CompositionEngine/tests/DisplayTest.cpp
@@ -847,6 +847,29 @@
}
/*
+ * Display::applyClientTargetRequests()
+ */
+
+using DisplayApplyClientTargetRequests = DisplayWithLayersTestCommon;
+
+TEST_F(DisplayApplyLayerRequestsToLayersTest, applyClientTargetRequests) {
+ Display::ClientTargetProperty clientTargetProperty = {
+ .pixelFormat = hal::PixelFormat::RGB_565,
+ .dataspace = hal::Dataspace::STANDARD_BT470M,
+ };
+
+ mock::RenderSurface* renderSurface = new StrictMock<mock::RenderSurface>();
+ mDisplay->setRenderSurfaceForTest(std::unique_ptr<RenderSurface>(renderSurface));
+
+ EXPECT_CALL(*renderSurface, setBufferPixelFormat(clientTargetProperty.pixelFormat));
+ EXPECT_CALL(*renderSurface, setBufferDataspace(clientTargetProperty.dataspace));
+ mDisplay->applyClientTargetRequests(clientTargetProperty);
+
+ auto& state = mDisplay->getState();
+ EXPECT_EQ(clientTargetProperty.dataspace, state.dataspace);
+}
+
+/*
* Display::presentAndGetFrameFences()
*/