diff options
| author |  robertphillips <robertphillips@google.com> | 2015-01-07 09:01:12 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-01-07 09:01:13 -0800 |
| commit | 152f524fd325b7776b01f84afbfe2fa071648a05 (patch) | |
| tree | 52255f53268067cf9e66272b67ca15711d8c4813 /gm/offsetimagefilter.cpp | |
| parent | bd0d9da981289504c9dcd8547eaad52b1f4a52cb (diff) | |
It is dangerous to ignore SkRect::intersect's return value
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) { |