Reset conicWeights in SkPath::consumeDegenerateSegments when rewinding to last Move op

Reset conicWeights in SkPath::consumeDegenerateSegments when rewinding to last Move op

Without this patch the iterator can end up running off the end of the conic weights if there is a mixture of degenerate and non-degenerate ops

Note: we might want to suppress the generation of degenerate conics and lines in SkPath::addRRect

BUG=459897
Committed: https://skia.googlesource.com/skia/+/b8de1f46594c3cd9c537f0b128c6d6eb30a9f463

[robertphillips, 2015-02-23 17:43:09]

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

[robertphillips, 2015-02-23 18:52:47]

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

[robertphillips, 2015-02-23 18:53:48]

https://codereview.chromium.org/954453003/diff/20001/tests/PathTest.cpp
File tests/PathTest.cpp (right):

https://codereview.chromium.org/954453003/diff/20001/tests/PathTest.cpp#newco...
tests/PathTest.cpp:2490: REPORTER_ASSERT(reporter, SkPath::kConic_Verb ==
iter.next(pts));
This next line will trigger a valgrind complaint w/o the reset of this CL.

[caryclark, 2015-02-23 18:55:42]

lgtm

[robertphillips, 2015-02-23 18:58:28]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-02-23 18:58:41]

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

[commit-bot: I haz the power, 2015-02-23 19:17:04]

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