From 1b1a557d25a7d23aa11f23a2eae7b3560c49db7f Mon Sep 17 00:00:00 2001 From: Kevin Lubick Date: Mon, 4 Jun 2018 17:02:46 -0400 Subject: 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 Reviewed-by: Hal Canary --- fuzz/FuzzCanvas.cpp | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'fuzz') 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 make_fuzz_maskfilter(Fuzz* fuzz) { SkScalar sigma; fuzz->next(&sigma); SkRect occluder{0.0f, 0.0f, 0.0f, 0.0f}; - if (make_fuzz_t(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); -- cgit v1.2.3