diff options
author | 2016-04-01 16:41:10 -0700 | |
---|---|---|
committer | 2016-04-01 16:41:10 -0700 | |
commit | 6db0a7bdceb6be85721bfb0db8dea7fd27db5970 (patch) | |
tree | b963703827c0a0b99e40b6a4d7e5a9afed9947a1 /include/effects/SkPaintImageFilter.h | |
parent | 2d9c6f81353597aebf5934547e5cba7a872196fb (diff) |
Image filters: optimize crop rect application.
If a filter does not affect transparent black, there's no
reason to allow the crop rect to expand beyond the optimal
size determined by onFilterNodeBounds(). So don't enlarge
the bounds unless the filter affects transparent black.
In order to determine which filters affect transparent
black, I've pulled the inverse of the canComputeFastBounds()
logic into a non-recursive, affectsTransparentBlack()
virtual, and left canComputeFastBounds() as a non-virtual,
recursive function that calls it.
BUG=599933
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1848953002
TBR=reed@google.com
Review URL: https://codereview.chromium.org/1848953002
Diffstat (limited to 'include/effects/SkPaintImageFilter.h')
-rw-r--r-- | include/effects/SkPaintImageFilter.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/effects/SkPaintImageFilter.h b/include/effects/SkPaintImageFilter.h index 2876c0431b..35a76b80fb 100644 --- a/include/effects/SkPaintImageFilter.h +++ b/include/effects/SkPaintImageFilter.h @@ -26,7 +26,7 @@ public: return sk_sp<SkImageFilter>(new SkPaintImageFilter(paint, cropRect)); } - bool canComputeFastBounds() const override; + bool affectsTransparentBlack() const override; SK_TO_STRING_OVERRIDE() SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkPaintImageFilter) |