diff options
| author |  robertphillips <robertphillips@google.com> | 2015-01-07 12:16:10 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-01-07 12:16:10 -0800 |
| commit | f4e5995ac70d4614e0a05b92a8a03e2b7d76bd9c (patch) | |
| tree | dc8fa61047e8e81aa32945031d13233f31059fde /gm/offsetimagefilter.cpp | |
| parent | a27b82ddf0fad959371a4ab1fba931826edb2f54 (diff) | |
It is dangerous to ignore SkRect::intersect's return value
Committed: https://skia.googlesource.com/skia/+/152f524fd325b7776b01f84afbfe2fa071648a05
Review URL: https://codereview.chromium.org/833943002
Diffstat (limited to 'gm/offsetimagefilter.cpp')
| -rw-r--r-- | gm/offsetimagefilter.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/gm/offsetimagefilter.cpp b/gm/offsetimagefilter.cpp index 7fcbe0a04d..3dafed8823 100644 --- a/gm/offsetimagefilter.cpp +++ b/gm/offsetimagefilter.cpp @@ -82,8 +82,9 @@ protected: scaleMatrix.setScale(scale, scale); SkRect cropRectFloat; scaleMatrix.mapRect(&cropRectFloat, SkRect::Make(cropRect)); - clipRect.intersect(cropRectFloat); - canvas->drawRect(clipRect, strokePaint); + if (clipRect.intersect(cropRectFloat)) { + canvas->drawRect(clipRect, strokePaint); + } } virtual void onDraw(SkCanvas* canvas) { |