diff options
author | msarett <msarett@google.com> | 2016-08-14 13:51:16 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-14 13:51:16 -0700 |
commit | 0c685ee188a5e500df25a34fa6edfceb4cbb795c (patch) | |
tree | 8902eb17cc42519f3f61ba4e953eb24662a07e2f /src/core | |
parent | 40ff5fe59b77b0b3e34467cc2f8666e4e88356f9 (diff) |
Assert fDeviceClipBounds is always empty when clip is empty
I've fixed a few issues where this assert triggers. The
nastiest case is when the client sets the clip to a strange
version of an empty rect (Ex: L,T,R,B=0,0,0,10).
I think I've made sufficient fixes so that
fDeviceClipBounds is always empty when clip is empty.
Let's test it though.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2246503003
Review-Url: https://codereview.chromium.org/2246503003
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/SkCanvas.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp index d199e5dd2d..80ff34d417 100644 --- a/src/core/SkCanvas.cpp +++ b/src/core/SkCanvas.cpp @@ -1863,7 +1863,7 @@ bool SkCanvas::quickReject(const SkRect& src) const { // Verify that fDeviceClipBounds are set properly. SkRect tmp = qr_clip_bounds(fMCRec->fRasterClip.getBounds()); if (fMCRec->fRasterClip.isEmpty()) { - SkASSERT(fDeviceClipBounds.isEmpty() || tmp == fDeviceClipBounds); + SkASSERT(fDeviceClipBounds.isEmpty()); } else { SkASSERT(tmp == fDeviceClipBounds); } |