diff options
| author |  reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-16 15:14:04 +0000 |
|---|---|---|
| committer | reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-16 15:14:04 +0000 |
| commit | f32322b9cefd950013ead6f914678580eff20b5b (patch) | |
| tree | 396e0ee490174543d245365138c76aea87bb4bd7 /tests/PathTest.cpp | |
| parent | 46484ec5bf9e2ba468672b019dbe92cf42ff847f (diff) | |
add disabled tests, in pref for experimental isRect behavior
BUG=
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/27513003
git-svn-id: http://skia.googlecode.com/svn/trunk@11810 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tests/PathTest.cpp')
| -rw-r--r-- | tests/PathTest.cpp | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/tests/PathTest.cpp b/tests/PathTest.cpp index 61f06e099e..c47b2c4858 100644 --- a/tests/PathTest.cpp +++ b/tests/PathTest.cpp @@ -1312,9 +1312,31 @@ static void test_conservativelyContains(skiatest::Reporter* reporter) { } +static void test_isRect_open_close(skiatest::Reporter* reporter) { + SkPath path; + bool isClosed; + + path.moveTo(0, 0); path.lineTo(1, 0); path.lineTo(1, 1); path.lineTo(0, 1); + + if (false) { + // I think these should pass, but isRect() doesn't behave + // this way... yet + REPORTER_ASSERT(reporter, path.isRect(NULL, NULL)); + REPORTER_ASSERT(reporter, path.isRect(&isClosed, NULL)); + REPORTER_ASSERT(reporter, !isClosed); + } + + path.close(); + REPORTER_ASSERT(reporter, path.isRect(NULL, NULL)); + REPORTER_ASSERT(reporter, path.isRect(&isClosed, NULL)); + REPORTER_ASSERT(reporter, isClosed); +} + // Simple isRect test is inline TestPath, below. // test_isRect provides more extensive testing. static void test_isRect(skiatest::Reporter* reporter) { + test_isRect_open_close(reporter); + // passing tests (all moveTo / lineTo... SkPoint r1[] = {{0, 0}, {1, 0}, {1, 1}, {0, 1}}; SkPoint r2[] = {{1, 0}, {1, 1}, {0, 1}, {0, 0}}; |