Attempt to throw away rrect clips of rrects.

Attempt to throw away rrect clips of rrects.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2241273003

Committed: https://skia.googlesource.com/skia/+/7f0d9f39206d0bf67e0a14e9cf3351243c9b5a1b

[bsalomon, 2016-08-15 17:53:46]

Description was changed from

==========
Attempt to throw away rrect clips of rrects.

BUG=skia:
==========

to

==========
Attempt to throw away rrect clips of rrects.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2241273003
==========

[bsalomon, 2016-08-15 17:55:24]

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

[bsalomon, 2016-08-15 17:55:24]

Food for thought... this removes the non-aa rrect clips from blurs.skp on skia
issue 5635

[robertphillips, 2016-08-15 18:38:47]

On 2016/08/15 17:55:24, bsalomon wrote:
> Food for thought... this removes the non-aa rrect clips from blurs.skp on skia
> issue 5635

Shouldn't it be getting rid of the entire clip stack?

[bsalomon, 2016-08-15 18:56:06]

On 2016/08/15 18:38:47, robertphillips wrote:
> On 2016/08/15 17:55:24, bsalomon wrote:
> > Food for thought... this removes the non-aa rrect clips from blurs.skp on
skia
> > issue 5635
> 
> Shouldn't it be getting rid of the entire clip stack?

Yes it does remove the entire stack. So if there is say a clip rect higher up
the stack that can't be removed then this doesn't help.

[robertphillips, 2016-08-15 19:02:21]

lgtm if you want it.

[bsalomon, 2016-08-15 19:21:59]

On 2016/08/15 19:02:21, robertphillips wrote:
> lgtm if you want it.

I hid the check in GrDC::drawRRect behind SK_BUILD_FOR_ANDROID_FRAMEWORK and
added a comment about why we're doing this terrible thing. Stan found that this
shaved 5ms from testShadowGridListFling.

[bsalomon, 2016-08-15 20:12:22]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-08-15 20:12:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/2241273003/#ps40001
(title: "minor")

[commit-bot: I haz the power, 2016-08-15 20:12:34]

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-08-15 20:32:08]

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

[commit-bot: I haz the power, 2016-08-15 20:32:09]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on master.client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[bsalomon, 2016-08-15 21:13:01]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-08-15 21:13:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/2241273003/#ps60001
(title: "fix int to scalar warnings")

[commit-bot: I haz the power, 2016-08-15 21:13:09]

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-08-15 21:49:13]

Description was changed from

==========
Attempt to throw away rrect clips of rrects.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2241273003
==========

to

==========
Attempt to throw away rrect clips of rrects.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2241273003

Committed:
https://skia.googlesource.com/skia/+/7f0d9f39206d0bf67e0a14e9cf3351243c9b5a1b
==========

[commit-bot: I haz the power, 2016-08-15 21:49:14]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/7f0d9f39206d0bf67e0a14e9cf3351243c9b5a1b