commit e80eb928ba0248a5a5dea6e1f0005aa08ecf8740
author robertphillips <robertphillips@google.com> Thu Dec 17 11:33:12 2015 -0800
committer Commit bot <commit-bot@chromium.org> Thu Dec 17 11:33:12 2015 -0800
tree 87bf6f73175a6a404308eb36b18337116a6396aa
parent 1250944aaa9251c08eda3a37b21f15106903f42d

Add default ctor to SkMask

The minimal fix here seems to be handling BoxBlur's return value in SkBlurMaskFilter.cpp::GrRRectBlurEffect::Create. We seem to do enough special handling of the fImage field though that always initialializing it may not be a bad idea.

BUG=570232
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1539553002

Review URL: https://codereview.chromium.org/1539553002

gm/blurrect.cpp

diff --git a/gm/blurrect.cpp b/gm/blurrect.cpp
index 4f15815..55c592b 100644
--- a/gm/blurrect.cpp
+++ b/gm/blurrect.cpp
@@ -199,7 +199,10 @@
                     SkBlurStyle style = styles[k];
 
                     SkMask mask;
-                    SkBlurMask::BlurRect(SkBlurMask::ConvertRadiusToSigma(radius), &mask, r, style);
+                    if (!SkBlurMask::BlurRect(SkBlurMask::ConvertRadiusToSigma(radius),
+                                              &mask, r, style)) {
+                        continue;
+                    }
 
                     SkAutoMaskFreeImage amfi(mask.fImage);