diff options
| author |  robertphillips <robertphillips@google.com> | 2015-12-17 11:33:12 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-12-17 11:33:12 -0800 |
| commit | e80eb928ba0248a5a5dea6e1f0005aa08ecf8740 (patch) | |
| tree | 87bf6f73175a6a404308eb36b18337116a6396aa /tests/BlurTest.cpp | |
| parent | 1250944aaa9251c08eda3a37b21f15106903f42d (diff) | |
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
Diffstat (limited to 'tests/BlurTest.cpp')
| -rw-r--r-- | tests/BlurTest.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/BlurTest.cpp b/tests/BlurTest.cpp index f5bb215065..556930e6e9 100644 --- a/tests/BlurTest.cpp +++ b/tests/BlurTest.cpp @@ -170,8 +170,9 @@ static void ground_truth_2d(int width, int height, memset(src.fImage, 0xff, src.computeTotalImageSize()); - dst.fImage = nullptr; - SkBlurMask::BlurGroundTruth(sigma, &dst, src, kNormal_SkBlurStyle); + if (!SkBlurMask::BlurGroundTruth(sigma, &dst, src, kNormal_SkBlurStyle)) { + return; + } int midX = dst.fBounds.centerX(); int midY = dst.fBounds.centerY(); |