diff options
| author |  Robert Phillips <robertphillips@google.com> | 2016-12-19 11:37:37 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-19 17:14:10 +0000 |
| commit | 98624d249d279f68127c76754d542ab5cd0f8eab (patch) | |
| tree | 86e5e534316aed3dd95001b32180f0df72e23ffd /tests | |
| parent | efa9d34ccbdeb541a1fa77a678552df7a08531be (diff) | |
"Fix" some ImageFilter fuzzer issues
SkClipOp.h & SkPictureFlat.h
Invalid SkClipOps were getting through - the question here is where (for a class enum) is a good place to put the k*Mask definition.
SkPath1DPathEffect
NaNs were getting past.
SkBlurMaskFilter
Assert wasn't necessary since we whacked the flag on the next line.
Change-Id: I87f95ad39f4760284f881d7c4500eb82fcdba282
Reviewed-on: https://skia-review.googlesource.com/6194
Commit-Queue: Robert Phillips <robertphillips@google.com>
Reviewed-by: Herb Derby <herb@google.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/BlurTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/BlurTest.cpp b/tests/BlurTest.cpp index 2893a0c937..f2c35d76bd 100644 --- a/tests/BlurTest.cpp +++ b/tests/BlurTest.cpp @@ -507,7 +507,7 @@ DEF_TEST(BlurAsABlur, reporter) { // Test asABlur for SkBlurMaskFilter // for (size_t i = 0; i < SK_ARRAY_COUNT(styles); ++i) { - const SkBlurStyle style = (SkBlurStyle)styles[i]; + const SkBlurStyle style = styles[i]; for (size_t j = 0; j < SK_ARRAY_COUNT(sigmas); ++j) { const SkScalar sigma = sigmas[j]; for (int flags = 0; flags <= SkBlurMaskFilter::kAll_BlurFlag; ++flags) { |