fix valgrind

fix valgrind

(A recently added fuzzer test broke the valgrind bot, intentionally,
to isolate an uninitialized memory bug.)

If the conic has ordered points, but cannot be broken into parts,
treat it as if it were monotonic.

R=reed@google.com
BUG=skia:4757
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1568513002

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

[caryclark, 2016-01-06 15:58:18]

Description was changed from

==========
fix valgrind

(A recently added fuzzer test broke the valgrind bot, intentionally,
to isolate an uninitialized memory bug.)

If the conic has ordered points, but cannot be broken into parts,
treat it as if it were monotonic.

R=reed@google.com
BUG=skia:4757
==========

to

==========
fix valgrind

(A recently added fuzzer test broke the valgrind bot, intentionally,
to isolate an uninitialized memory bug.)

If the conic has ordered points, but cannot be broken into parts,
treat it as if it were monotonic.

R=reed@google.com
BUG=skia:4757
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[caryclark, 2016-01-06 15:59:30]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2016-01-06 15:59:41]

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

[commit-bot: I haz the power, 2016-01-06 15:59:42]

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 2016-01-06 21:59 UTC

[reed1, 2016-01-06 16:12:06]

lgtm w/ request for comment

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

https://codereview.chromium.org/1568513002/diff/1/src/core/SkPath.cpp#newcode...
src/core/SkPath.cpp:2768: bool isMono = is_mono_quad(pts[0].fY, pts[1].fY,
pts[2].fY) || !conic.chopAtYExtrema(chopped);
maybe a comment describing why we check for the chopper possibly failing, even
though the control points are known to be non-monotonic

[commit-bot: I haz the power, 2016-01-06 16:15:48]

Description was changed from

==========
fix valgrind

(A recently added fuzzer test broke the valgrind bot, intentionally,
to isolate an uninitialized memory bug.)

If the conic has ordered points, but cannot be broken into parts,
treat it as if it were monotonic.

R=reed@google.com
BUG=skia:4757
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
fix valgrind

(A recently added fuzzer test broke the valgrind bot, intentionally,
to isolate an uninitialized memory bug.)

If the conic has ordered points, but cannot be broken into parts,
treat it as if it were monotonic.

R=reed@google.com
BUG=skia:4757
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-01-06 16:15:48]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/e114dd63161b5efffbffb17e4d3facad77f34507