diff options
| author |  Xianzhu Wang <wangxianzhu@chromium.org> | 2017-09-15 19:57:53 -0700 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-09-16 18:55:53 +0000 |
| commit | bf12c079707a89f683ad0a6c989127fa20d9d202 (patch) | |
| tree | 2fa4b72ee252ba02e5a46be17ba24f6453efd8a4 /include/effects | |
| parent | b8a7c8cd1c410465806deda32181618b04e8517d (diff) | |
Reland "Fix SkImageSource::filterBounds()"
This relands commit cb4d58766668b003fea67798027fc25d84acd059
which was reverted by commit b6d2be1330f16fe05f1eed5abda927c0fbb50055
because the original CL broke some blink layout tests.
This reland let SkImageSource::filterBounds() return the dst rect with
ctm applied regardless of direction.
Original description:
> Previously SkImageSource::filterBounds() uses the default
> SkImageFilter::onFilterNodeBounds() which returns the input rect.
>
> Now override onFilterNodeBounds() in SkImageSource to return src
> or dst rect (with transform applied).
Change-Id: I915b7889ff59829ddbc4479cd66d75a0bb581e54
Reviewed-on: https://skia-review.googlesource.com/47501
Commit-Queue: Stephen White <senorblanco@chromium.org>
Reviewed-by: Stephen White <senorblanco@chromium.org>
Diffstat (limited to 'include/effects')
| -rw-r--r-- | include/effects/SkImageSource.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/effects/SkImageSource.h b/include/effects/SkImageSource.h index 04348c68e8..f2c1274f29 100644 --- a/include/effects/SkImageSource.h +++ b/include/effects/SkImageSource.h @@ -31,6 +31,8 @@ protected: SkIPoint* offset) const override; sk_sp<SkImageFilter> onMakeColorSpace(SkColorSpaceXformer*) const override; + SkIRect onFilterNodeBounds(const SkIRect&, const SkMatrix&, MapDirection) const override; + private: explicit SkImageSource(sk_sp<SkImage>); SkImageSource(sk_sp<SkImage>, |