diff options
| author |  robertphillips <robertphillips@google.com> | 2015-01-07 09:12:43 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-01-07 09:12:43 -0800 |
| commit | 690fc594f11a987ba0e885a7eb01aff363395f12 (patch) | |
| tree | bca37044feffc8e2ae1b54f86fd46bd0f9732909 /gm/offsetimagefilter.cpp | |
| parent | 7e44bb191633e225fd0455c267dbf67f9ee8633e (diff) | |
Revert of It is dangerous to ignore SkRect::intersect's return value (patchset #6 id:100001 of https://codereview.chromium.org/833943002/)
Reason for revert:
Still more Chromium clean up to do
Original issue's description:
> It is dangerous to ignore SkRect::intersect's return value
>
> Committed: https://skia.googlesource.com/skia/+/152f524fd325b7776b01f84afbfe2fa071648a05
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/825983005
Diffstat (limited to 'gm/offsetimagefilter.cpp')
| -rw-r--r-- | gm/offsetimagefilter.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/gm/offsetimagefilter.cpp b/gm/offsetimagefilter.cpp index 3dafed8823..7fcbe0a04d 100644 --- a/gm/offsetimagefilter.cpp +++ b/gm/offsetimagefilter.cpp @@ -82,9 +82,8 @@ protected: scaleMatrix.setScale(scale, scale); SkRect cropRectFloat; scaleMatrix.mapRect(&cropRectFloat, SkRect::Make(cropRect)); - if (clipRect.intersect(cropRectFloat)) { - canvas->drawRect(clipRect, strokePaint); - } + clipRect.intersect(cropRectFloat); + canvas->drawRect(clipRect, strokePaint); } virtual void onDraw(SkCanvas* canvas) { |