fix msan bug in pathops

fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003

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

[caryclark, 2016-09-23 16:00:40]

Description was changed from

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
==========

to

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003
==========

[caryclark, 2016-09-23 16:01:50]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2016-09-23 16:02:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 16:02:04]

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-09-23 22:01 UTC

[borenet, 2016-09-23 16:07:29]

LGTM with the caveat that I've never seen this code before.

[caryclark, 2016-09-23 16:30:02]

Description was changed from

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003
==========

to

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003
==========

[caryclark, 2016-09-23 16:31:07]

The CQ bit was unchecked by caryclark@google.com

[caryclark, 2016-09-23 16:31:11]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2016-09-23 16:31:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 16:32:28]

Description was changed from

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003
==========

to

==========
fix msan bug in pathops

Msan and Valgrind found an uninitialized memory mistake in
pathops. This also fixes similar bugs where not all parts
of the geometry were covered in the loop iteration.

R=borenet@google.com
NOTREECHECKS=true
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2366893003

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

[commit-bot: I haz the power, 2016-09-23 16:32:30]

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