pathops coincident work

pathops coincident work

This is working towards fixing all bugs around simplifying the tiger.

This installment simplifies the point-t intersection list as it is built rather than doing the analysis once the intersections are complete. This avoids getting the list in an inconsistent state and makes coincident checks faster and more stable.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002

TBR=reed@google.com
BUG=skia:5131
Committed: https://skia.googlesource.com/skia/+/30b9fdd6a1d607bde20c793af65b5e2e8a1737ca

[caryclark, 2016-08-11 21:05:33]

Description was changed from

==========
wip; coincident insert mismatch

Segments 5 and 6 are coincident at a number of points.
When segments 1 and 4 insert new intersections into 5 and 6,
the order in one doesn't match the order in the other.
==========

to

==========
wip; coincident insert mismatch

Segments 5 and 6 are coincident at a number of points.
When segments 1 and 4 insert new intersections into 5 and 6,
the order in one doesn't match the order in the other.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002
==========

[caryclark, 2016-08-31 20:56:18]

Description was changed from

==========
wip; coincident insert mismatch

Segments 5 and 6 are coincident at a number of points.
When segments 1 and 4 insert new intersections into 5 and 6,
the order in one doesn't match the order in the other.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002
==========

to

==========
pathops coincident work

This is working towards fixing all bugs around simplifying the tiger.

This installment simplifies the point-t intersection list as it is built rather
than doing the analysis once the intersections are complete. This avoids getting
the list in an inconsistent state and makes coincident checks faster and more
stable.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002

TBR=reed@google.com
BUG=skia:5131
==========

[caryclark, 2016-08-31 20:56:19]

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

[caryclark, 2016-08-31 20:56:46]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2016-08-31 20:56:51]

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-08-31 21:36:32]

Description was changed from

==========
pathops coincident work

This is working towards fixing all bugs around simplifying the tiger.

This installment simplifies the point-t intersection list as it is built rather
than doing the analysis once the intersections are complete. This avoids getting
the list in an inconsistent state and makes coincident checks faster and more
stable.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002

TBR=reed@google.com
BUG=skia:5131
==========

to

==========
pathops coincident work

This is working towards fixing all bugs around simplifying the tiger.

This installment simplifies the point-t intersection list as it is built rather
than doing the analysis once the intersections are complete. This avoids getting
the list in an inconsistent state and makes coincident checks faster and more
stable.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2237223002

TBR=reed@google.com
BUG=skia:5131

Committed:
https://skia.googlesource.com/skia/+/30b9fdd6a1d607bde20c793af65b5e2e8a1737ca
==========

[commit-bot: I haz the power, 2016-08-31 21:36:34]

Committed patchset #22 (id:420001) as
https://skia.googlesource.com/skia/+/30b9fdd6a1d607bde20c793af65b5e2e8a1737ca