aboutsummaryrefslogtreecommitdiffhomepage
path: root/fuzz
diff options
context:
space:
mode:
authorGravatar Kevin Lubick <kjlubick@google.com>2018-06-04 17:02:46 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2018-06-06 13:47:06 +0000
commit1b1a557d25a7d23aa11f23a2eae7b3560c49db7f (patch)
treeb8fa736b32ccb41577e3725cdf2adea1bde2bbbb /fuzz
parent74b390d6b136a60f1df15ac5ecd19bd8ad5a394b (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.cpp14
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);