Update IfExepensiveOutputDataspaceIsUsed test
Prior to I5c77769cf9fd2f07806c97022fa4bb940bfcc1ac, using a P3 output
space meant we would boost the GPU frequency. This was a shortcut for
having a different dataspace, since it would require more GPU work. The
more thorough check means that we boost only if the inputs don't match
the output. Update the test to include a single layer, which defaults to
a different dataspace, making the test pass again.
Bug: 223651281
Test: libcompositionengine_test
Change-Id: Iad084a8388e6203b83135ebfb632fda1b3cdb448
diff --git a/services/surfaceflinger/CompositionEngine/tests/OutputTest.cpp b/services/surfaceflinger/CompositionEngine/tests/OutputTest.cpp
index c2521b2..54eb8f8 100644
--- a/services/surfaceflinger/CompositionEngine/tests/OutputTest.cpp
+++ b/services/surfaceflinger/CompositionEngine/tests/OutputTest.cpp
@@ -3744,8 +3744,9 @@
TEST_F(OutputComposeSurfacesTest_SetsExpensiveRendering, IfExepensiveOutputDataspaceIsUsed) {
mOutput.mState.dataspace = kExpensiveOutputDataspace;
+ LayerFE::LayerSettings layerSettings;
EXPECT_CALL(mOutput, generateClientCompositionRequests(_, kExpensiveOutputDataspace, _))
- .WillOnce(Return(std::vector<LayerFE::LayerSettings>{}));
+ .WillOnce(Return(std::vector<LayerFE::LayerSettings>{layerSettings}));
// For this test, we also check the call order of key functions.
InSequence seq;