diff --git a/expectations/gm/ignored-tests.txt b/expectations/gm/ignored-tests.txt index d9db6bce52..9baba7eee0 100644 --- a/expectations/gm/ignored-tests.txt +++ b/expectations/gm/ignored-tests.txt @@ -51,3 +51,8 @@ rrect_clip_aa # This change removes an API that this GM was testing. If/when it lands and sticks, # I will likely just delete the GM. canvas-layer-state + +# senorblanco: https://codereview.chromium.org/196133006/ +# The setYUV2RGB sample in transparent mode in this GM this test has always +# drawn incorrectly on the GPU. The change above fixes it. +colormatrix diff --git a/src/effects/SkColorMatrixFilter.cpp b/src/effects/SkColorMatrixFilter.cpp index bb85485050..23be874b97 100644 --- a/src/effects/SkColorMatrixFilter.cpp +++ b/src/effects/SkColorMatrixFilter.cpp @@ -424,6 +424,7 @@ public: builder->getUniformCStr(fMatrixHandle), inputColor, builder->getUniformCStr(fVectorHandle)); + builder->fsCodeAppendf("\t%s = clamp(%s, 0.0, 1.0);\n", outputColor, outputColor); builder->fsCodeAppendf("\t%s.rgb *= %s.a;\n", outputColor, outputColor); }