Use SkRRect to store rects in SkClipStack::Element

Use SkRRect to store rects in SkClipStack::Element


BUG=skia:2181
Committed: http://code.google.com/p/skia/source/detail?r=13544

[bsalomon, 2014-02-21 19:08:02]

On 2014/02/16 17:36:29, bsalomon wrote:

ping

[robertphillips, 2014-02-21 19:12:02]

Ooops - thought this had already landed. lgtm + a question.

https://codereview.chromium.org/163483003/diff/50001/include/core/SkClipStack.h
File include/core/SkClipStack.h (right):

https://codereview.chromium.org/163483003/diff/50001/include/core/SkClipStack...
include/core/SkClipStack.h:135: case kRect_Type:
Why not just fRRect.getBounds().contains(rect)?

[bsalomon, 2014-02-21 19:14:59]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-02-21 19:15:03]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/bsalomon@google.com/163483003/50001

[bsalomon, 2014-02-21 19:20:02]

https://codereview.chromium.org/163483003/diff/50001/include/core/SkClipStack.h
File include/core/SkClipStack.h (right):

https://codereview.chromium.org/163483003/diff/50001/include/core/SkClipStack...
include/core/SkClipStack.h:135: case kRect_Type:
On 2014/02/21 19:12:02, robertphillips wrote:
> Why not just fRRect.getBounds().contains(rect)?

just for the benefit of the assert. I assume it's the same in release.

[commit-bot: I haz the power, 2014-02-21 20:09:17]

Change committed as 13544