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

Commit f2b8ec47 authored by Leon Scroggins III's avatar Leon Scroggins III
Browse files

Fix log statement on failure to setVisibleRegion 

Follow on to I2b7b3b4b5630011323a802908c5a80917bdb86ef, which makes it
so we might pass the overrideInfo's visibleRegion instead of the one on
outputDependentState. If the HWC2::Layer call fails, dump the actual
region passed to it, whether it's the original or the override.

Bug: 183456093
Test: make
Change-Id: I5dbb106f7c855d328b5c72a1e4ccacda20bda2c7
parent d77d316c

services/surfaceflinger/CompositionEngine/src/OutputLayer.cpp

+1 −1
Original line number Diff line number Diff line
@@ -481,7 +481,7 @@ void OutputLayer::writeOutputDependentPerFrameStateToHWC(HWC2::Layer* hwcLayer) 
    if (auto error = hwcLayer->setVisibleRegion(visibleRegion); error != hal::Error::NONE) {

        ALOGE("[%s] Failed to set visible region: %s (%d)", getLayerFE().getDebugName(),

              to_string(error).c_str(), static_cast<int32_t>(error));

        outputDependentState.outputSpaceVisibleRegion.dump(LOG_TAG);

        visibleRegion.dump(LOG_TAG);

    }



    const auto dataspace = outputDependentState.overrideInfo.buffer