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

Commit eb8d82ed authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Fix: Release skia resources when RenderNode layer is configured" into... 

Merge "Fix: Release skia resources when RenderNode layer is configured" into sc-dev am: 43038ca7 am: 4a3b07e1

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

Change-Id: I77ba586b9de2d058c00b5dd5e9a31c96f6b43b28
parents 9d60722c 4a3b07e1

libs/hwui/RenderNode.h

+2 −0
Original line number Diff line number Diff line
@@ -331,6 +331,8 @@ public: 
            mSkiaLayer.reset();

        }



        mProperties.mutateLayerProperties().mutableStretchEffect().clear();

        mStretchMask.clear();

        // Clear out the previous snapshot and the image filter the previous

        // snapshot was created with whenever the layer changes.

        mSnapshotResult.snapshot = nullptr;

libs/hwui/effects/StretchEffect.cpp

+5 −2
Original line number Diff line number Diff line
@@ -186,6 +186,7 @@ static const SkString stretchShader = SkString(R"( 


static const float ZERO = 0.f;

static const float INTERPOLATION_STRENGTH_VALUE = 0.7f;

static const char CONTENT_TEXTURE[] = "uContentTexture";



sk_sp<SkShader> StretchEffect::getShader(float width, float height,

                                         const sk_sp<SkImage>& snapshotImage,
@@ -207,7 +208,7 @@ sk_sp<SkShader> StretchEffect::getShader(float width, float height, 
        mBuilder = std::make_unique<SkRuntimeShaderBuilder>(getStretchEffect());

    }



    mBuilder->child("uContentTexture") =

    mBuilder->child(CONTENT_TEXTURE) =

            snapshotImage->makeShader(SkTileMode::kClamp, SkTileMode::kClamp,

                                      SkSamplingOptions(SkFilterMode::kLinear), matrix);

    mBuilder->uniform("uInterpolationStrength").set(&INTERPOLATION_STRENGTH_VALUE, 1);
@@ -226,7 +227,9 @@ sk_sp<SkShader> StretchEffect::getShader(float width, float height, 
    mBuilder->uniform("viewportWidth").set(&width, 1);

    mBuilder->uniform("viewportHeight").set(&height, 1);



    return mBuilder->makeShader(nullptr, false);

    auto result = mBuilder->makeShader(nullptr, false);

    mBuilder->child(CONTENT_TEXTURE) = nullptr;

    return result;

}



sk_sp<SkRuntimeEffect> StretchEffect::getStretchEffect() {

libs/hwui/effects/StretchEffect.h

+4 −0
Original line number Diff line number Diff line
@@ -113,6 +113,10 @@ public: 
        return !isEmpty();

    }



    void clear() {

        mBuilder = nullptr;

    }



private:

    static sk_sp<SkRuntimeEffect> getStretchEffect();

    mutable SkVector mStretchDirection{0, 0};