Make cap only affect the keys of GrShapes that are possibly-open

Make cap only affect the keys of GrShapes that are possibly-open

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1949193002

Committed: https://skia.googlesource.com/skia/+/06077565b18714ff3fc0db9118e2c21f6f25243f

[bsalomon, 2016-05-04 20:08:41]

Description was changed from

==========
Make cap only affect the keys of GrShapes that are possibly-open

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

to

==========
Make cap only affect the keys of GrShapes that are possibly-open

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-05-04 20:15:35]

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

[bsalomon, 2016-05-04 20:15:35]

This was in the long tail of possible optimizations to do for GrShape. However,
landing this now avoids semi-awkward workarounds in the test code for special
cases introduced in a forthcoming patch that adds support for stroke-and-fill
(which is necessary to displace GrStrokeInfo).

[bsalomon, 2016-05-04 20:17:10]

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h
File src/gpu/GrStyle.h (right):

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h#newco...
src/gpu/GrStyle.h:66: static int KeySize(const GrStyle& , Apply, uint32_t flags
= 0);
KeySize doesn't actually use this but figured it was best to keep the params
between KeySize and WriteKey in sync in case it matters for new flags.

[robertphillips, 2016-05-04 20:27:00]

lgtm

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h
File src/gpu/GrStyle.h (right):

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h#newco...
src/gpu/GrStyle.h:51: /**
do to -> due to ?

[bsalomon, 2016-05-04 20:29:22]

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h
File src/gpu/GrStyle.h (right):

https://codereview.chromium.org/1949193002/diff/20001/src/gpu/GrStyle.h#newco...
src/gpu/GrStyle.h:51: /**
On 2016/05/04 20:27:00, robertphillips wrote:
> do to -> due to ?

Done.

[bsalomon, 2016-05-04 20:29:26]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-05-04 20:29:27]

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/1949193002/#ps40001
(title: "due")

[commit-bot: I haz the power, 2016-05-04 20:29:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1949193002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1949193002/40001

[commit-bot: I haz the power, 2016-05-04 20:50:31]

Description was changed from

==========
Make cap only affect the keys of GrShapes that are possibly-open

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make cap only affect the keys of GrShapes that are possibly-open

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/06077565b18714ff3fc0db9118e2c21f6f25243f
==========

[commit-bot: I haz the power, 2016-05-04 20:50:32]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/06077565b18714ff3fc0db9118e2c21f6f25243f