Canonicalize path fill types for stroked paths in GrShape.

Canonicalize path fill types for stroked paths in GrShape.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2085913003

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

[bsalomon, 2016-06-21 17:23:57]

Description was changed from

==========
Canonicalize path fill types for stroked paths in GrShape.
==========

to

==========
Canonicalize path fill types for stroked paths in GrShape.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2085913003
==========

[bsalomon, 2016-06-21 17:28:42]

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

[bsalomon, 2016-06-21 17:28:42]

Follow-up to the last CL to standardize the fill type for stroked paths.

[bsalomon, 2016-06-22 13:39:52]

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

[commit-bot: I haz the power, 2016-06-22 13:39:59]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085913003/1

[commit-bot: I haz the power, 2016-06-22 14:04:53]

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

[commit-bot: I haz the power, 2016-06-22 14:04:54]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-06-23 13:42:39]

ping

[bsalomon, 2016-06-23 15:31:21]

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

[commit-bot: I haz the power, 2016-06-23 15:31:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085913003/20001

[commit-bot: I haz the power, 2016-06-23 15:51:55]

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

[commit-bot: I haz the power, 2016-06-23 15:51:56]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-06-23 18:34:43]

ping again, this is a small CL.

[egdaniel, 2016-06-23 18:43:52]

sorry for delay, lgtm

[bsalomon, 2016-06-23 18:45:20]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-06-23 18:45:32]

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

[robertphillips, 2016-06-23 18:47:53]

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

[robertphillips, 2016-06-23 18:47:54]

lgtm

https://codereview.chromium.org/2085913003/diff/20001/src/gpu/GrShape.cpp
File src/gpu/GrShape.cpp (right):

https://codereview.chromium.org/2085913003/diff/20001/src/gpu/GrShape.cpp#new...
src/gpu/GrShape.cpp:339: // Moreover, dashing ignores inverseness
(skbug.com/5421)
why fStyle here and not style() ?

https://codereview.chromium.org/2085913003/diff/20001/tests/GrShapeTest.cpp
File tests/GrShapeTest.cpp (right):

https://codereview.chromium.org/2085913003/diff/20001/tests/GrShapeTest.cpp#n...
tests/GrShapeTest.cpp:306: const SkStrokeRec::Style  strokeRecStyleB =
b.style().strokeRec().getStyle();
add a "static bool can_change(const GrStyle, const SkPath&)" ?

[commit-bot: I haz the power, 2016-06-23 18:48:28]

Description was changed from

==========
Canonicalize path fill types for stroked paths in GrShape.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2085913003
==========

to

==========
Canonicalize path fill types for stroked paths in GrShape.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2085913003

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

[commit-bot: I haz the power, 2016-06-23 18:48:30]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/a4817afe5095b67a68c907ff2e48e3e07907140c