Don't scissor draws overlapping the render target's edges

Don't scissor draws overlapping the render target's edges
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135673002

Committed: https://skia.googlesource.com/skia/+/bd2bbe4a1750166136c7729adb0b48ebd7132b74

[bsalomon, 2016-07-08 13:37:04]

Description was changed from

==========
Don't scissor draws overlapping the render target's edges
==========

to

==========
Don't scissor draws overlapping the render target's edges
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135673002
==========

[bsalomon, 2016-07-08 13:38:18]

bsalomon@google.com changed reviewers:
+ robertphillips@google.com

[bsalomon, 2016-07-08 13:57:47]

The CQ bit was checked by bsalomon@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-08 13:57:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 14:19:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-08 14:19:07]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-07-08 14:21:58]

lgtm

https://codereview.chromium.org/2135673002/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/2135673002/diff/1/src/gpu/GrClipMaskManager.c...
src/gpu/GrClipMaskManager.cpp:246: if (origDevBounds) {
Are there any issues with this reject and hairlines ?

[bsalomon, 2016-07-08 14:25:56]

https://codereview.chromium.org/2135673002/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/2135673002/diff/1/src/gpu/GrClipMaskManager.c...
src/gpu/GrClipMaskManager.cpp:246: if (origDevBounds) {
On 2016/07/08 14:21:58, robertphillips wrote:
> Are there any issues with this reject and hairlines ?

Because of 'empty' rects? No, because GrDT bloats them out (either rounding out
or outsetting by 0.5).

[bsalomon, 2016-07-08 14:35:39]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-07-08 14:35:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 14:36:44]

Description was changed from

==========
Don't scissor draws overlapping the render target's edges
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135673002
==========

to

==========
Don't scissor draws overlapping the render target's edges
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135673002

Committed:
https://skia.googlesource.com/skia/+/bd2bbe4a1750166136c7729adb0b48ebd7132b74
==========

[commit-bot: I haz the power, 2016-07-08 14:36:48]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/bd2bbe4a1750166136c7729adb0b48ebd7132b74