diff options
author | Kevin Lubick <kjlubick@google.com> | 2018-06-04 17:02:46 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-06-06 13:47:06 +0000 |
commit | 1b1a557d25a7d23aa11f23a2eae7b3560c49db7f (patch) | |
tree | b8fa736b32ccb41577e3725cdf2adea1bde2bbbb /fuzz | |
parent | 74b390d6b136a60f1df15ac5ecd19bd8ad5a394b (diff) |
Add fuzzer option for Blur w/o occluder
I hypothesize that some backends look at the occluder and may have
accidentally (on purpose) been avoiding the linked bug.
Bug: skia:7956
Change-Id: If8d1de099f778b2faaa2c5ed4c562047b0dd14e7
Reviewed-on: https://skia-review.googlesource.com/131922
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
Diffstat (limited to 'fuzz')
-rw-r--r-- | fuzz/FuzzCanvas.cpp | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/fuzz/FuzzCanvas.cpp b/fuzz/FuzzCanvas.cpp index 8e84e88b3c..5c92fd1eb5 100644 --- a/fuzz/FuzzCanvas.cpp +++ b/fuzz/FuzzCanvas.cpp @@ -498,13 +498,17 @@ static sk_sp<SkMaskFilter> make_fuzz_maskfilter(Fuzz* fuzz) { SkScalar sigma; fuzz->next(&sigma); SkRect occluder{0.0f, 0.0f, 0.0f, 0.0f}; - if (make_fuzz_t<bool>(fuzz)) { + bool useOccluder; + fuzz->next(&useOccluder); + if (useOccluder) { fuzz->next(&occluder); } - uint32_t flags; - fuzz->nextRange(&flags, 0, 1); - bool respectCTM = flags != 0; - return SkMaskFilter::MakeBlur(blurStyle, sigma, occluder, respectCTM); + bool respectCTM; + fuzz->next(&respectCTM); + if (useOccluder) { + return SkMaskFilter::MakeBlur(blurStyle, sigma, occluder, respectCTM); + } + return SkMaskFilter::MakeBlur(blurStyle, sigma, respectCTM); } default: SkASSERT(false); |