break out of cubic stroker loop on degenerate case

break out of cubic stroker loop on degenerate case

The looper can generate more than one quad, but if any one is degenerate,
give up, but not before generating the state for the line join to
produce the correct end.

Before, the early return allowed the inside path to contain multiple
movetos that caused reversePath to assert.

R=reed@google.com

Committed: https://skia.googlesource.com/skia/+/6df8e3495a677a5df2476d2ff3bbe878fd178e4b

[caryclark, 2015-02-23 15:42:08]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-23 15:42:29]

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

[commit-bot: I haz the power, 2015-02-23 15:42:29]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-23 21:42 UTC

[reed1, 2015-02-23 15:44:12]

https://codereview.chromium.org/948043002/diff/1/src/core/SkStroke.cpp
File src/core/SkStroke.cpp (right):

https://codereview.chromium.org/948043002/diff/1/src/core/SkStroke.cpp#newcod...
src/core/SkStroke.cpp:1414: if (!this->cubicStroke(cubic, &quadPts)) {
Can we add a brief comment as to why this might occur, and what the resulting
outer/inner will be as a result?

[caryclark, 2015-02-23 17:10:47]

https://codereview.chromium.org/948043002/diff/1/src/core/SkStroke.cpp
File src/core/SkStroke.cpp (right):

https://codereview.chromium.org/948043002/diff/1/src/core/SkStroke.cpp#newcod...
src/core/SkStroke.cpp:1414: if (!this->cubicStroke(cubic, &quadPts)) {
On 2015/02/23 15:44:12, reed1 wrote:
> Can we add a brief comment as to why this might occur, and what the resulting
> outer/inner will be as a result?

Done.

[reed1, 2015-02-23 20:21:02]

lgtm

[caryclark, 2015-02-23 20:37:18]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-23 20:38:18]

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

[commit-bot: I haz the power, 2015-02-23 20:47:07]

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