diff options
| author |  caryclark <caryclark@google.com> | 2014-06-17 05:15:38 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2014-06-17 05:15:38 -0700 |
| commit | dac1d17027dcaa5596885a9f333979418b35001c (patch) | |
| tree | 923c6ca762654144254565240de5e9ec6598c41f /tests/PathOpsAngleTest.cpp | |
| parent | d6043b20b63f895d384b4794205ac914abfafa71 (diff) | |
Enabling the canvas bit to turn the clip stack into a flat replace exposed around 100 failures when testing the 800K skp set generated from the top 1M web sites.
This fixes all but one of those failures.
Major changes include:
- Replace angle indices with angle pointers. This was motivated by the need to add angles later but not renumber existing angles.
- Aggressive segment chase. When the winding is known on a segment, more aggressively passing that winding to adjacent segments allows fragmented data sets to succeed.
- Line segments with ends nearly the same are treated as coincident first.
- Transfer partial coincidence by observing that if segment A is partially coincident to B and C then B and C may be partially coincident.
TBR=reed
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/272153002
Diffstat (limited to 'tests/PathOpsAngleTest.cpp')
| -rw-r--r-- | tests/PathOpsAngleTest.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/PathOpsAngleTest.cpp b/tests/PathOpsAngleTest.cpp index 1aae27a847..faf61584e6 100644 --- a/tests/PathOpsAngleTest.cpp +++ b/tests/PathOpsAngleTest.cpp @@ -264,7 +264,7 @@ DEF_TEST(PathOpsAngleCircle, reporter) { break; } } - PathOpsAngleTester::Orderable(segment[0].angle(0), segment[1].angle(0)); + PathOpsAngleTester::Orderable(*segment[0].debugLastAngle(), *segment[1].debugLastAngle()); } struct IntersectData { @@ -438,9 +438,9 @@ DEF_TEST(PathOpsAngleAfter, reporter) { } break; } } - SkOpAngle& angle1 = const_cast<SkOpAngle&>(segment[0].angle(0)); - SkOpAngle& angle2 = const_cast<SkOpAngle&>(segment[1].angle(0)); - SkOpAngle& angle3 = const_cast<SkOpAngle&>(segment[2].angle(0)); + SkOpAngle& angle1 = *const_cast<SkOpAngle*>(segment[0].debugLastAngle()); + SkOpAngle& angle2 = *const_cast<SkOpAngle*>(segment[1].debugLastAngle()); + SkOpAngle& angle3 = *const_cast<SkOpAngle*>(segment[2].debugLastAngle()); PathOpsAngleTester::SetNext(angle1, angle3); // These data sets are seeded when the set itself fails, so likely the dataset does not // match the expected result. The tests above return 1 when first added, but |