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

Commit 501a46b0 authored by Nader Jawad's avatar Nader Jawad Committed by Automerger Merge Worker
Browse files

Merge "Fix aggressive caching of RenderNode snapshot" into sc-v2-dev am: ddd6bac7 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15795422

Change-Id: Ia4d4879a2b832bcf14078594548eb953856e0f0a
parents ffa32f51 ddd6bac7

libs/hwui/RenderNode.cpp

+6 −3
Original line number Diff line number Diff line
@@ -341,6 +341,7 @@ std::optional<RenderNode::SnapshotResult> RenderNode::updateSnapshotIfRequired( 
    sk_sp<SkImage> snapshot = layerSurface->makeImageSnapshot();

    const auto subset = SkIRect::MakeWH(properties().getWidth(),

                                        properties().getHeight());

    uint32_t layerSurfaceGenerationId = layerSurface->generationID();

    // If we don't have an ImageFilter just return the snapshot

    if (imageFilter == nullptr) {

        mSnapshotResult.snapshot = snapshot;
@@ -348,9 +349,10 @@ std::optional<RenderNode::SnapshotResult> RenderNode::updateSnapshotIfRequired( 
        mSnapshotResult.outOffset = SkIPoint::Make(0.0f, 0.0f);

        mImageFilterClipBounds = clipBounds;

        mTargetImageFilter = nullptr;

    } else if (mSnapshotResult.snapshot == nullptr ||

        imageFilter != mTargetImageFilter.get() ||

        mImageFilterClipBounds != clipBounds) {

        mTargetImageFilterLayerSurfaceGenerationId = 0;

    } else if (mSnapshotResult.snapshot == nullptr || imageFilter != mTargetImageFilter.get() ||

               mImageFilterClipBounds != clipBounds ||

               mTargetImageFilterLayerSurfaceGenerationId != layerSurfaceGenerationId) {

        // Otherwise create a new snapshot with the given filter and snapshot

        mSnapshotResult.snapshot =

                snapshot->makeWithFilter(context,
@@ -361,6 +363,7 @@ std::optional<RenderNode::SnapshotResult> RenderNode::updateSnapshotIfRequired( 
                                         &mSnapshotResult.outOffset);

        mTargetImageFilter = sk_ref_sp(imageFilter);

        mImageFilterClipBounds = clipBounds;

        mTargetImageFilterLayerSurfaceGenerationId = layerSurfaceGenerationId;

    }



    return mSnapshotResult;

libs/hwui/RenderNode.h

+1 −0
Original line number Diff line number Diff line
@@ -396,6 +396,7 @@ private: 
     * SkImageFilter used to create the mSnapshotResult

     */

    sk_sp<SkImageFilter> mTargetImageFilter;

    uint32_t mTargetImageFilterLayerSurfaceGenerationId = 0;



    /**

     * Clip bounds used to create the mSnapshotResult