diff options
author | Brian Osman <brianosman@google.com> | 2018-07-06 13:57:01 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-06 18:34:58 +0000 |
commit | a50205fca5f7cff36dd3adac841b4c23a90a0b7d (patch) | |
tree | 4aa111a0c4b80400dfa39ef5b36e8ef391c164ab /src/image | |
parent | 5a619a740f12f0b1f504f305b8964efe4c85ee26 (diff) |
Add color type to the image filter DAG's OutputProperties
Don't try to guess the pixel config to use for intermediates. Instead,
just make the intermediates in the same color type (and space) as the
final destination. This removes some no-longer-correct logic that was
using sRGB configs, resulting in linear blending and precision loss.
Change-Id: I627c47193a9f2889c3dc121170ff3e7d5d315fa0
Reviewed-on: https://skia-review.googlesource.com/139547
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Diffstat (limited to 'src/image')
-rw-r--r-- | src/image/SkImage.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/image/SkImage.cpp b/src/image/SkImage.cpp index 43c4cbaa39..128e693ad2 100644 --- a/src/image/SkImage.cpp +++ b/src/image/SkImage.cpp @@ -256,6 +256,7 @@ sk_sp<SkImage> SkImage::makeWithFilter(const SkImageFilter* filter, const SkIRec if (!filter || !outSubset || !offset || !this->bounds().contains(subset)) { return nullptr; } + SkColorType colorType = as_IB(this)->onImageInfo().colorType(); SkColorSpace* colorSpace = as_IB(this)->onImageInfo().colorSpace(); sk_sp<SkSpecialImage> srcSpecialImage = SkSpecialImage::MakeFromImage( subset, sk_ref_sp(const_cast<SkImage*>(this)), colorSpace); @@ -265,7 +266,7 @@ sk_sp<SkImage> SkImage::makeWithFilter(const SkImageFilter* filter, const SkIRec sk_sp<SkImageFilterCache> cache( SkImageFilterCache::Create(SkImageFilterCache::kDefaultTransientSize)); - SkImageFilter::OutputProperties outputProperties(colorSpace); + SkImageFilter::OutputProperties outputProperties(colorType, colorSpace); SkImageFilter::Context context(SkMatrix::I(), clipBounds, cache.get(), outputProperties); sk_sp<SkSpecialImage> result = filter->filterImage(srcSpecialImage.get(), context, offset); |