make matrix color filter runtime-only

The unit test Paint_nothingToDraw expects isAlphaUnchanged() to work
correctly on a matrix color filter, so we take care to make it easy
to prove alpha is unchanged when its row is [0 0 0 1 0].

Cq-Include-Trybots: luci.chromium.try:linux-blink-rel
Change-Id: I1bc537b2c1c4892bd65831806cf32b933539bff0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/370713
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
This commit is contained in:
Mike Klein 2021-02-16 10:36:31 -06:00 committed by Skia Commit-Bot
parent 1499249539
commit a42c6192ff

View File

@ -9,12 +9,14 @@
#include "include/core/SkString.h"
#include "include/core/SkUnPreMultiply.h"
#include "include/effects/SkColorMatrix.h"
#include "include/effects/SkRuntimeEffect.h"
#include "include/private/SkColorData.h"
#include "include/private/SkNx.h"
#include "src/core/SkColorFilter_Matrix.h"
#include "src/core/SkColorSpacePriv.h"
#include "src/core/SkRasterPipeline.h"
#include "src/core/SkReadBuffer.h"
#include "src/core/SkRuntimeEffectPriv.h"
#include "src/core/SkVM.h"
#include "src/core/SkWriteBuffer.h"
@ -156,9 +158,72 @@ GrFPResult SkColorFilter_Matrix::asFragmentProcessor(std::unique_ptr<GrFragmentP
static sk_sp<SkColorFilter> MakeMatrix(const float array[20],
SkColorFilter_Matrix::Domain domain) {
return sk_floats_are_finite(array, 20)
? sk_make_sp<SkColorFilter_Matrix>(array, domain)
: nullptr;
if (!sk_floats_are_finite(array, 20)) {
return nullptr;
}
#if defined(SK_SUPPORT_LEGACY_RUNTIME_EFFECTS)
return sk_make_sp<SkColorFilter_Matrix>(array, domain);
#else
const bool alphaUnchanged = SkScalarNearlyEqual(array[15], 0)
&& SkScalarNearlyEqual(array[16], 0)
&& SkScalarNearlyEqual(array[17], 0)
&& SkScalarNearlyEqual(array[18], 1)
&& SkScalarNearlyEqual(array[19], 0);
struct { SkM44 m; SkV4 b; } uniforms;
SkString code {
"uniform shader input;"
"uniform half4x4 m;"
"uniform half4 b;"
};
if (domain == SkColorFilter_Matrix::Domain::kHSLA) {
code += kRGB_to_HSL_sksl;
code += kHSL_to_RGB_sksl;
}
code += "half4 main() {";
if (true) {
code += "half4 c = sample(input);"; // unpremul
}
if (alphaUnchanged) {
code += "half a = c.a;";
}
if (domain == SkColorFilter_Matrix::Domain::kHSLA) {
code += "c.rgb = rgb_to_hsl(c.rgb);";
}
if (true) {
uniforms.m = SkM44{array[ 0], array[ 1], array[ 2], array[ 3],
array[ 5], array[ 6], array[ 7], array[ 8],
array[10], array[11], array[12], array[13],
array[15], array[16], array[17], array[18]};
uniforms.b = SkV4{array[4], array[9], array[14], array[19]};
code += "c = m*c + b;";
}
if (domain == SkColorFilter_Matrix::Domain::kHSLA) {
code += "c.rgb = hsl_to_rgb(c.rgb);";
}
if (alphaUnchanged) {
code += "return half4(saturate(c.rgb), a);";
} else {
code += "return saturate(c);";
}
code += "}";
auto [effect, err] = SkRuntimeEffect::Make(code);
if (!err.isEmpty()) {
SkDebugf("%s\n%s\n", code.c_str(), err.c_str());
}
SkASSERT(effect && err.isEmpty());
sk_sp<SkColorFilter> input = nullptr;
SkAlphaType unpremul = kUnpremul_SkAlphaType;
return SkColorFilters::WithWorkingFormat(
effect->makeColorFilter(SkData::MakeWithCopy(&uniforms,sizeof(uniforms)), &input, 1),
nullptr/*keep dst TF encoding*/,
nullptr/*stay in dst gamut*/,
&unpremul);
#endif
}
sk_sp<SkColorFilter> SkColorFilters::Matrix(const float array[20]) {