Make GrShape capable of representing inverse filled rrects.

Make GrShape capable of representing inverse filled rrects.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2056523002

Committed: https://skia.googlesource.com/skia/+/7049396b65660907af5292d899053280430d929a

[bsalomon, 2016-06-08 21:08:43]

Description was changed from

==========
Make GrShape capable of representing inverse filled rrects.
==========

to

==========
Make GrShape capable of representing inverse filled rrects.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2056523002
==========

[bsalomon, 2016-06-09 16:15:13]

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

[egdaniel, 2016-06-10 14:42:42]

lgtm

[bsalomon, 2016-06-10 14:48:38]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-06-10 14:48:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2056523002/80001

[commit-bot: I haz the power, 2016-06-10 15:05:16]

Description was changed from

==========
Make GrShape capable of representing inverse filled rrects.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2056523002
==========

to

==========
Make GrShape capable of representing inverse filled rrects.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2056523002

Committed:
https://skia.googlesource.com/skia/+/7049396b65660907af5292d899053280430d929a
==========

[commit-bot: I haz the power, 2016-06-10 15:05:17]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/7049396b65660907af5292d899053280430d929a

[mtklein, 2016-06-10 20:58:11]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-06-10 20:58:12]

Any chance we can convert some stack storage to static or heap in here?  Google3
build:

third_party/skia/HEAD/tests/GrShapeTest.cpp: In function 'void
test_rrect(skiatest::Reporter*, const SkRRect&)':
third_party/skia/HEAD/tests/GrShapeTest.cpp:1227:1: error: the frame size of
63888 bytes is larger than 16384 bytes; see http://go/big_stack_frame
[-Werror=frame-larger-than=]
At global scope:
cc1plus: error: unrecognized command line option "-Wno-implicit-fallthrough"
[-Werror]
cc1plus: all warnings being treated as errors

[bsalomon, 2016-06-11 00:20:00]

On 2016/06/10 20:58:12, mtklein wrote:
> Any chance we can convert some stack storage to static or heap in here? 
Google3
> build:
> 
> third_party/skia/HEAD/tests/GrShapeTest.cpp: In function 'void
> test_rrect(skiatest::Reporter*, const SkRRect&)':
> third_party/skia/HEAD/tests/GrShapeTest.cpp:1227:1: error: the frame size of
> 63888 bytes is larger than 16384 bytes; see http://go/big_stack_frame
> [-Werror=frame-larger-than=]
> At global scope:
> cc1plus: error: unrecognized command line option "-Wno-implicit-fallthrough"
> [-Werror]
> cc1plus: all warnings being treated as errors

Yep, will do.