commit 646f934394fa3cb0f26ebca8c189da373ad8d579
author Peiyong Lin <lpy@google.com> Thu Dec 27 15:24:25 2018 -0800
committer Peiyong Lin <lpy@google.com> Thu Dec 27 15:27:13 2018 -0800
tree 55c67265828e2441227180e6763f801435644fd8
parent e27954463f72c7bd778986098f0a286fcc714e5d

[RenderEngine] Fix transfer function bug.

Previously we failed to set the input and output transfer function when there's
no color space transformation or no color matrix, this patch fixes it.

BUG: 120445231
Test: Build, flash and boot. Verify with demo app.
Change-Id: I853b6b9944a05e6360dfdfca493351f755b500ba

libs/renderengine/gl/ProgramCache.cpp

diff --git a/libs/renderengine/gl/ProgramCache.cpp b/libs/renderengine/gl/ProgramCache.cpp
index bf2354d..49bdd2a 100644
--- a/libs/renderengine/gl/ProgramCache.cpp
+++ b/libs/renderengine/gl/ProgramCache.cpp
@@ -151,7 +151,8 @@
     needs.set(Key::Y410_BT2020_MASK,
               description.isY410BT2020 ? Key::Y410_BT2020_ON : Key::Y410_BT2020_OFF);
 
-    if (needs.hasTransformMatrix() || (needs.getInputTF() != needs.getOutputTF())) {
+    if (needs.hasTransformMatrix() ||
+        (description.inputTransferFunction != description.outputTransferFunction)) {
         switch (description.inputTransferFunction) {
             case Description::TransferFunction::LINEAR:
             default: