Reland "fix nextContour bug"
This reverts commit 0a86820f2fe3b62f2a5ac2917a1ef342dbe53722.
Reason for revert: fix has landed in CTS
Original change's description:
> Revert "fix nextContour bug"
>
> This reverts commit 473f69276f1854db80e094cc4af091f7f8a5b00b.
>
> Reason for revert: a CTS test PathMeasureTest.android.graphics.cts.PathMeasureTest.testNextContour is failing
>
> Original change's description:
> > fix nextContour bug
> >
> > Calling nextContour() needs to ensure that we've already measured the "current" one.
> >
> > Bug: skia:
> > Change-Id: I501c26d0b068028d67103888f06ec89125a5407a
> > Reviewed-on: https://skia-review.googlesource.com/114692
> > Reviewed-by: Cary Clark <caryclark@google.com>
> > Commit-Queue: Mike Reed <reed@google.com>
>
> TBR=caryclark@google.com,reed@google.com
>
> Change-Id: I7f1e08651dfe73c02158b209dc28af2b174e246f
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: skia:
> Reviewed-on: https://skia-review.googlesource.com/114823
> Reviewed-by: Stan Iliev <stani@google.com>
> Commit-Queue: Stan Iliev <stani@google.com>
TBR=caryclark@google.com,reed@google.com,stani@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: skia:
Change-Id: I0fd61473ef0df62722cf6c6b86f1277f580e4ab1
Reviewed-on: https://skia-review.googlesource.com/115340
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
diff --git a/tests/PathMeasureTest.cpp b/tests/PathMeasureTest.cpp
index df66578..bc75c5f 100644
--- a/tests/PathMeasureTest.cpp
+++ b/tests/PathMeasureTest.cpp
@@ -217,3 +217,13 @@
REPORTER_ASSERT(reporter, 19.5f < stdP.fX && stdP.fX < 20.5f);
REPORTER_ASSERT(reporter, 19.5f < hiP.fX && hiP.fX < 20.5f);
}
+
+// Regression test for b/26425223
+DEF_TEST(PathMeasure_nextctr, reporter) {
+ SkPath path;
+ path.moveTo(0, 0); path.lineTo(100, 0);
+
+ SkPathMeasure meas(path, false);
+ // only expect 1 contour, even if we didn't explicitly call getLength() ourselves
+ REPORTER_ASSERT(reporter, !meas.nextContour());
+}