Updated SkiaGLRenderEngine stretch logic
Removed explicit flag to disable stretching
surfaceflinger layers with the stretch shader in favor
of leveraging a adb setprop to determine the type of
stretch to apply.
Updated some variable names + types as response
to earlier CL feedback
Bug: 184297961
Test: manual
Change-Id: I4f1d7a7b4a33095d920b3149eb474cbe07c87cb0
diff --git a/libs/renderengine/skia/SkiaGLRenderEngine.cpp b/libs/renderengine/skia/SkiaGLRenderEngine.cpp
index f95c6c6..c2cf2a4 100644
--- a/libs/renderengine/skia/SkiaGLRenderEngine.cpp
+++ b/libs/renderengine/skia/SkiaGLRenderEngine.cpp
@@ -564,10 +564,10 @@
// which gets similar results for most surfaces
// Determine later on if we need to leverage the stertch shader within
// surface flinger
- if (stretchEffect.hasEffect() && /* DISABLES CODE */ (false)) {
+ if (stretchEffect.hasEffect()) {
const auto targetBuffer = layer->source.buffer.buffer;
- const auto graphicsBuffer = targetBuffer ? targetBuffer->getBuffer() : nullptr;
- if (graphicsBuffer && shader) {
+ const auto graphicBuffer = targetBuffer ? targetBuffer->getBuffer() : nullptr;
+ if (graphicBuffer && shader) {
shader = mStretchShaderFactory.createSkShader(shader, stretchEffect);
}
}