diff options
| author |  Stan Iliev <stani@google.com> | 2018-03-16 21:01:09 +0000 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-03-16 21:01:16 +0000 |
| commit | 0a86820f2fe3b62f2a5ac2917a1ef342dbe53722 (patch) | |
| tree | b49356c759de9f0cc731dbdad3a7ec0dde8321dc /tests/PathMeasureTest.cpp | |
| parent | 55fa647596be0952464820b46724ce4f358de7f7 (diff) | |
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>
Diffstat (limited to 'tests/PathMeasureTest.cpp')
| -rw-r--r-- | tests/PathMeasureTest.cpp | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/tests/PathMeasureTest.cpp b/tests/PathMeasureTest.cpp index bc75c5f122..df66578f1b 100644 --- a/tests/PathMeasureTest.cpp +++ b/tests/PathMeasureTest.cpp @@ -217,13 +217,3 @@ DEF_TEST(PathMeasureConic, reporter) { 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()); -} |