Skip to content

Commit ae566f2

Browse files
Treehugger RobotAndroid (Google) Code Review
authored andcommitted
Merge "[Lut HAL backend] skip libtonemap if the lut(s) is in use" into main
2 parents 2b9ffc0 + 9cea07c commit ae566f2

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

libs/renderengine/skia/SkiaRenderEngine.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -572,8 +572,10 @@ sk_sp<SkShader> SkiaRenderEngine::createRuntimeEffectShader(
572572
}
573573

574574
// disable tonemapping if we already locally tonemapped
575-
auto inputDataspace =
576-
usingLocalTonemap ? parameters.outputDataSpace : parameters.layer.sourceDataspace;
575+
// skip tonemapping if the luts is in use
576+
auto inputDataspace = usingLocalTonemap || (graphicBuffer && parameters.layer.luts)
577+
? parameters.outputDataSpace
578+
: parameters.layer.sourceDataspace;
577579
auto effect =
578580
shaders::LinearEffect{.inputDataspace = inputDataspace,
579581
.outputDataspace = parameters.outputDataSpace,

0 commit comments

Comments
 (0)