Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 02da997f authored by Vishnu Nair's avatar Vishnu Nair
Browse files

Change RenderEngine::drawLayers to take a vector of LayerSettings pointers 

Prework for the client composition cache logic which extends
LayerSettings. Passing in a set of pointers will reduce the
amount of copying needed when passing the LayerSettings to
RenderEngine.

Bug: 136561771
Test: atest librenderengine_test libcompositionengine_test
Test: atest VtsHalGraphicsComposerV2_2TargetTest
Test: go/wm-smoke

Change-Id: I0581be7762bf56cd7a3aab0511adf95cd6e2df76
parent 234830af

graphics/composer/2.2/utils/vts/RenderEngineVts.cpp

+9 −1
Original line number Diff line number Diff line
@@ -57,7 +57,15 @@ void TestRenderEngine::initGraphicBuffer(uint32_t width, uint32_t height, uint32 
void TestRenderEngine::drawLayers() {

    base::unique_fd bufferFence;

    base::unique_fd readyFence;

    mRenderEngine->drawLayers(mDisplaySettings, mCompositionLayers,



    std::vector<const renderengine::LayerSettings*> compositionLayerPointers;

    compositionLayerPointers.reserve(mCompositionLayers.size());

    std::transform(mCompositionLayers.begin(), mCompositionLayers.end(),

                   std::back_insert_iterator(compositionLayerPointers),

                   [](renderengine::LayerSettings& settings) -> renderengine::LayerSettings* {

                       return &settings;

                   });

    mRenderEngine->drawLayers(mDisplaySettings, compositionLayerPointers,

                              mGraphicBuffer->getNativeBuffer(), true, std::move(bufferFence),

                              &readyFence);

    int fd = readyFence.release();