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

Commit 9d763b70 authored by Lucas Dupin's avatar Lucas Dupin
Browse files

Do not blur when doing region sampling 

The luma approximation will already be pretty good without the blurs.

Test: RegionSamplingTest
Test: libsurfaceflinger_test
Fixes: 154556259
Change-Id: I4010ac54aee2e23bd8cf155f591584eeec425702
parent 2af0642b

services/surfaceflinger/RegionSamplingThread.cpp

+2 −1
Original line number Diff line number Diff line
@@ -430,7 +430,8 @@ void RegionSamplingThread::captureSample() { 
    }



    bool ignored;

    mFlinger.captureScreenCommon(renderArea, traverseLayers, buffer, false, ignored);

    mFlinger.captureScreenCommon(renderArea, traverseLayers, buffer, false /* identityTransform */,

                                 true /* regionSampling */, ignored);



    std::vector<Descriptor> activeDescriptors;

    for (const auto& descriptor : descriptors) {

services/surfaceflinger/SurfaceFlinger.cpp

+15 −6
Original line number Diff line number Diff line
@@ -5674,13 +5674,13 @@ status_t SurfaceFlinger::captureScreenCommon(RenderArea& renderArea, 
                                             usage, "screenshot");



    return captureScreenCommon(renderArea, traverseLayers, *outBuffer, useIdentityTransform,

                               outCapturedSecureLayers);

                               false /* regionSampling */, outCapturedSecureLayers);

}



status_t SurfaceFlinger::captureScreenCommon(RenderArea& renderArea,

                                             TraverseLayersFunction traverseLayers,

                                             const sp<GraphicBuffer>& buffer,

                                             bool useIdentityTransform,

                                             bool useIdentityTransform, bool regionSampling,

                                             bool& outCapturedSecureLayers) {

    // This mutex protects syncFd and captureResult for communication of the return values from the

    // main thread back to this Binder thread
@@ -5711,7 +5711,7 @@ status_t SurfaceFlinger::captureScreenCommon(RenderArea& renderArea, 
            renderArea.render([&] {

                result = captureScreenImplLocked(renderArea, traverseLayers, buffer.get(),

                                                 useIdentityTransform, forSystem, &fd,

                                                 outCapturedSecureLayers);

                                                 regionSampling, outCapturedSecureLayers);

            });

        }
@@ -5748,7 +5748,8 @@ status_t SurfaceFlinger::captureScreenCommon(RenderArea& renderArea, 
void SurfaceFlinger::renderScreenImplLocked(const RenderArea& renderArea,

                                            TraverseLayersFunction traverseLayers,

                                            const sp<GraphicBuffer>& buffer,

                                            bool useIdentityTransform, int* outSyncFd) {

                                            bool useIdentityTransform, bool regionSampling,

                                            int* outSyncFd) {

    ATRACE_CALL();



    const auto reqWidth = renderArea.getReqWidth();
@@ -5804,6 +5805,12 @@ void SurfaceFlinger::renderScreenImplLocked(const RenderArea& renderArea, 
            for (auto& settings : results) {

                settings.geometry.positionTransform =

                        transform.asMatrix4() * settings.geometry.positionTransform;

                // There's no need to process blurs when we're executing region sampling,

                // we're just trying to understand what we're drawing, and doing so without

                // blurs is already a pretty good approximation.

                if (regionSampling) {

                    settings.backgroundBlurRadius = 0;

                }

            }

            clientCompositionLayers.insert(clientCompositionLayers.end(),

                                           std::make_move_iterator(results.begin()),
@@ -5841,7 +5848,8 @@ status_t SurfaceFlinger::captureScreenImplLocked(const RenderArea& renderArea, 
                                                 TraverseLayersFunction traverseLayers,

                                                 const sp<GraphicBuffer>& buffer,

                                                 bool useIdentityTransform, bool forSystem,

                                                 int* outSyncFd, bool& outCapturedSecureLayers) {

                                                 int* outSyncFd, bool regionSampling,

                                                 bool& outCapturedSecureLayers) {

    ATRACE_CALL();



    traverseLayers([&](Layer* layer) {
@@ -5856,7 +5864,8 @@ status_t SurfaceFlinger::captureScreenImplLocked(const RenderArea& renderArea, 
        ALOGW("FB is protected: PERMISSION_DENIED");

        return PERMISSION_DENIED;

    }

    renderScreenImplLocked(renderArea, traverseLayers, buffer, useIdentityTransform, outSyncFd);

    renderScreenImplLocked(renderArea, traverseLayers, buffer, useIdentityTransform, regionSampling,

                           outSyncFd);

    return NO_ERROR;

}

services/surfaceflinger/SurfaceFlinger.h

+4 −3
Original line number Diff line number Diff line
@@ -697,19 +697,20 @@ private: 


    void renderScreenImplLocked(const RenderArea& renderArea, TraverseLayersFunction traverseLayers,

                                const sp<GraphicBuffer>& buffer, bool useIdentityTransform,

                                int* outSyncFd);

                                bool regionSampling, int* outSyncFd);

    status_t captureScreenCommon(RenderArea& renderArea, TraverseLayersFunction traverseLayers,

                                 sp<GraphicBuffer>* outBuffer, const ui::PixelFormat reqPixelFormat,

                                 bool useIdentityTransform, bool& outCapturedSecureLayers);

    status_t captureScreenCommon(RenderArea& renderArea, TraverseLayersFunction traverseLayers,

                                 const sp<GraphicBuffer>& buffer, bool useIdentityTransform,

                                 bool& outCapturedSecureLayers);

                                 bool regionSampling, bool& outCapturedSecureLayers);

    const sp<DisplayDevice> getDisplayByIdOrLayerStack(uint64_t displayOrLayerStack);

    const sp<DisplayDevice> getDisplayByLayerStack(uint64_t layerStack);

    status_t captureScreenImplLocked(const RenderArea& renderArea,

                                     TraverseLayersFunction traverseLayers,

                                     const sp<GraphicBuffer>& buffer, bool useIdentityTransform,

                                     bool forSystem, int* outSyncFd, bool& outCapturedSecureLayers);

                                     bool forSystem, int* outSyncFd, bool regionSampling,

                                     bool& outCapturedSecureLayers);

    void traverseLayersInDisplay(const sp<const DisplayDevice>& display,

                                 const LayerVector::Visitor& visitor);

services/surfaceflinger/tests/unittests/CompositionTest.cpp

+2 −1
Original line number Diff line number Diff line
@@ -231,6 +231,7 @@ void CompositionTest::captureScreenComposition() { 
    const Rect sourceCrop(0, 0, DEFAULT_DISPLAY_WIDTH, DEFAULT_DISPLAY_HEIGHT);

    constexpr bool useIdentityTransform = true;

    constexpr bool forSystem = true;

    constexpr bool regionSampling = false;



    DisplayRenderArea renderArea(mDisplay, sourceCrop, DEFAULT_DISPLAY_WIDTH,

                                 DEFAULT_DISPLAY_HEIGHT, ui::Dataspace::V0_SRGB,
@@ -249,7 +250,7 @@ void CompositionTest::captureScreenComposition() { 
    int fd = -1;

    status_t result =

            mFlinger.captureScreenImplLocked(renderArea, traverseLayers, mCaptureScreenBuffer.get(),

                                             useIdentityTransform, forSystem, &fd);

                                             useIdentityTransform, forSystem, &fd, regionSampling);

    if (fd >= 0) {

        close(fd);

    }

services/surfaceflinger/tests/unittests/TestableSurfaceFlinger.h

+2 −2
Original line number Diff line number Diff line
@@ -326,11 +326,11 @@ public: 
    auto captureScreenImplLocked(const RenderArea& renderArea,

                                 SurfaceFlinger::TraverseLayersFunction traverseLayers,

                                 const sp<GraphicBuffer>& buffer, bool useIdentityTransform,

                                 bool forSystem, int* outSyncFd) {

                                 bool forSystem, int* outSyncFd, bool regionSampling) {

        bool ignored;

        return mFlinger->captureScreenImplLocked(renderArea, traverseLayers, buffer,

                                                 useIdentityTransform, forSystem, outSyncFd,

                                                 ignored);

                                                 regionSampling, ignored);

    }



    auto traverseLayersInDisplay(const sp<const DisplayDevice>& display,