diff options
author | brianosman <brianosman@google.com> | 2016-09-12 08:50:19 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-12 08:50:19 -0700 |
commit | 5192475bd8cb98e8e0c1192ab5ece7b8595701d6 (patch) | |
tree | 29461e55053726c93c8c612e6fe65dc89fc1a431 /src/image | |
parent | 9b0fe3d125f237d9884732a48414fa85fc71b4e3 (diff) |
Two changes:
1. Remove special premul handling from gamut xform code
Alpha is a constant, so the gamut transformation results remain unchanged
(it distributes across the linear matrix multiply).
2. Use SkMatrix44 rather than array of floats
Preserves semantic intention, and makes upcoming code (where we transform
colors on the CPU by that matrix) simpler.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2329553002
Review-Url: https://codereview.chromium.org/2329553002
Diffstat (limited to 'src/image')
-rw-r--r-- | src/image/SkImageShader.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/image/SkImageShader.cpp b/src/image/SkImageShader.cpp index 9afd207360..e52deff270 100644 --- a/src/image/SkImageShader.cpp +++ b/src/image/SkImageShader.cpp @@ -216,8 +216,7 @@ sk_sp<GrFragmentProcessor> SkImageShader::asFragmentProcessor(const AsFPArgs& ar SkImageInfo info = as_IB(fImage)->onImageInfo(); sk_sp<GrColorSpaceXform> colorSpaceXform = GrColorSpaceXform::Make(info.colorSpace(), - args.fDstColorSpace, - info.alphaType()); + args.fDstColorSpace); sk_sp<GrFragmentProcessor> inner; if (doBicubic) { inner = GrBicubicEffect::Make(texture, std::move(colorSpaceXform), matrix, tm); |