| Index: tests/PathOpsSimplifyFailTest.cpp
 | 
| diff --git a/tests/PathOpsSimplifyFailTest.cpp b/tests/PathOpsSimplifyFailTest.cpp
 | 
| index 0245f878c1c5cd8384bae9872e973e4e0b6cf19a..8c0f9ba852078115802091d303a479cab7e0c882 100644
 | 
| --- a/tests/PathOpsSimplifyFailTest.cpp
 | 
| +++ b/tests/PathOpsSimplifyFailTest.cpp
 | 
| @@ -37,63 +37,82 @@ static const SkPoint finitePts[] = {
 | 
|  
 | 
|  const size_t finitePtsCount = sizeof(finitePts) / sizeof(finitePts[0]);
 | 
|  
 | 
| +static void failOne(skiatest::Reporter* reporter, int index) {
 | 
| +    SkPath path;
 | 
| +    int i = (int) (index % nonFinitePtsCount);
 | 
| +    int f = (int) (index % finitePtsCount);
 | 
| +    int g = (int) ((f + 1) % finitePtsCount);
 | 
| +    switch (index % 13) {
 | 
| +        case 0: path.lineTo(nonFinitePts[i]); break;
 | 
| +        case 1: path.quadTo(nonFinitePts[i], nonFinitePts[i]); break;
 | 
| +        case 2: path.quadTo(nonFinitePts[i], finitePts[f]); break;
 | 
| +        case 3: path.quadTo(finitePts[f], nonFinitePts[i]); break;
 | 
| +        case 4: path.cubicTo(nonFinitePts[i], finitePts[f], finitePts[f]); break;
 | 
| +        case 5: path.cubicTo(finitePts[f], nonFinitePts[i], finitePts[f]); break;
 | 
| +        case 6: path.cubicTo(finitePts[f], finitePts[f], nonFinitePts[i]); break;
 | 
| +        case 7: path.cubicTo(nonFinitePts[i], nonFinitePts[i], finitePts[f]); break;
 | 
| +        case 8: path.cubicTo(nonFinitePts[i], finitePts[f], nonFinitePts[i]); break;
 | 
| +        case 9: path.cubicTo(finitePts[f], nonFinitePts[i], nonFinitePts[i]); break;
 | 
| +        case 10: path.cubicTo(nonFinitePts[i], nonFinitePts[i], nonFinitePts[i]); break;
 | 
| +        case 11: path.cubicTo(nonFinitePts[i], finitePts[f], finitePts[g]); break;
 | 
| +        case 12: path.moveTo(nonFinitePts[i]); break;
 | 
| +    }
 | 
| +    SkPath result;
 | 
| +    result.setFillType(SkPath::kWinding_FillType);
 | 
| +    bool success = Simplify(path, &result);
 | 
| +    REPORTER_ASSERT(reporter, !success);
 | 
| +    REPORTER_ASSERT(reporter, result.isEmpty());
 | 
| +    REPORTER_ASSERT(reporter, result.getFillType() == SkPath::kWinding_FillType);
 | 
| +    reporter->bumpTestCount();
 | 
| +}
 | 
| +
 | 
| +static void dontFailOne(skiatest::Reporter* reporter, int index) {
 | 
| +    SkPath path;
 | 
| +    int f = (int) (index % finitePtsCount);
 | 
| +    int g = (int) ((f + 1) % finitePtsCount);
 | 
| +    switch (index % 11) {
 | 
| +        case 0: path.lineTo(finitePts[f]); break;
 | 
| +        case 1: path.quadTo(finitePts[f], finitePts[f]); break;
 | 
| +        case 2: path.quadTo(finitePts[f], finitePts[g]); break;
 | 
| +        case 3: path.quadTo(finitePts[g], finitePts[f]); break;
 | 
| +        case 4: path.cubicTo(finitePts[f], finitePts[f], finitePts[f]); break;
 | 
| +        case 5: path.cubicTo(finitePts[f], finitePts[f], finitePts[g]); break;
 | 
| +        case 6: path.cubicTo(finitePts[f], finitePts[g], finitePts[f]); break;
 | 
| +        case 7: path.cubicTo(finitePts[f], finitePts[g], finitePts[g]); break;
 | 
| +        case 8: path.cubicTo(finitePts[g], finitePts[f], finitePts[f]); break;
 | 
| +        case 9: path.cubicTo(finitePts[g], finitePts[f], finitePts[g]); break;
 | 
| +        case 10: path.moveTo(finitePts[f]); break;
 | 
| +    }
 | 
| +    SkPath result;
 | 
| +    result.setFillType(SkPath::kWinding_FillType);
 | 
| +    bool success = Simplify(path, &result);
 | 
| +    REPORTER_ASSERT(reporter, success);
 | 
| +    REPORTER_ASSERT(reporter, result.getFillType() != SkPath::kWinding_FillType);
 | 
| +    reporter->bumpTestCount();
 | 
| +}
 | 
| +
 | 
|  static void PathOpsSimplifyFailTest(skiatest::Reporter* reporter) {
 | 
|      for (int index = 0; index < (int) (13 * nonFinitePtsCount * finitePtsCount); ++index) {
 | 
| -        SkPath path;
 | 
| -        int i = (int) (index % nonFinitePtsCount);
 | 
| -        int f = (int) (index % finitePtsCount);
 | 
| -        int g = (int) ((f + 1) % finitePtsCount);
 | 
| -        switch (index % 13) {
 | 
| -            case 0: path.lineTo(nonFinitePts[i]); break;
 | 
| -            case 1: path.quadTo(nonFinitePts[i], nonFinitePts[i]); break;
 | 
| -            case 2: path.quadTo(nonFinitePts[i], finitePts[f]); break;
 | 
| -            case 3: path.quadTo(finitePts[f], nonFinitePts[i]); break;
 | 
| -            case 4: path.cubicTo(nonFinitePts[i], finitePts[f], finitePts[f]); break;
 | 
| -            case 5: path.cubicTo(finitePts[f], nonFinitePts[i], finitePts[f]); break;
 | 
| -            case 6: path.cubicTo(finitePts[f], finitePts[f], nonFinitePts[i]); break;
 | 
| -            case 7: path.cubicTo(nonFinitePts[i], nonFinitePts[i], finitePts[f]); break;
 | 
| -            case 8: path.cubicTo(nonFinitePts[i], finitePts[f], nonFinitePts[i]); break;
 | 
| -            case 9: path.cubicTo(finitePts[f], nonFinitePts[i], nonFinitePts[i]); break;
 | 
| -            case 10: path.cubicTo(nonFinitePts[i], nonFinitePts[i], nonFinitePts[i]); break;
 | 
| -            case 11: path.cubicTo(nonFinitePts[i], finitePts[f], finitePts[g]); break;
 | 
| -            case 12: path.moveTo(nonFinitePts[i]); break;
 | 
| -        }
 | 
| -        SkPath result;
 | 
| -        result.setFillType(SkPath::kWinding_FillType);
 | 
| -        bool success = Simplify(path, &result);
 | 
| -        REPORTER_ASSERT(reporter, !success);
 | 
| -        REPORTER_ASSERT(reporter, result.isEmpty());
 | 
| -        REPORTER_ASSERT(reporter, result.getFillType() == SkPath::kWinding_FillType);
 | 
| -        reporter->bumpTestCount();
 | 
| -    }
 | 
| -    if (sizeof(reporter) == 4) {
 | 
| -        return;
 | 
| +        failOne(reporter, index);
 | 
|      }
 | 
|      for (int index = 0; index < (int) (11 * finitePtsCount); ++index) {
 | 
| -        SkPath path;
 | 
| -        int f = (int) (index % finitePtsCount);
 | 
| -        int g = (int) ((f + 1) % finitePtsCount);
 | 
| -        switch (index % 11) {
 | 
| -            case 0: path.lineTo(finitePts[f]); break;
 | 
| -            case 1: path.quadTo(finitePts[f], finitePts[f]); break;
 | 
| -            case 2: path.quadTo(finitePts[f], finitePts[g]); break;
 | 
| -            case 3: path.quadTo(finitePts[g], finitePts[f]); break;
 | 
| -            case 4: path.cubicTo(finitePts[f], finitePts[f], finitePts[f]); break;
 | 
| -            case 5: path.cubicTo(finitePts[f], finitePts[f], finitePts[g]); break;
 | 
| -            case 6: path.cubicTo(finitePts[f], finitePts[g], finitePts[f]); break;
 | 
| -            case 7: path.cubicTo(finitePts[f], finitePts[g], finitePts[g]); break;
 | 
| -            case 8: path.cubicTo(finitePts[g], finitePts[f], finitePts[f]); break;
 | 
| -            case 9: path.cubicTo(finitePts[g], finitePts[f], finitePts[g]); break;
 | 
| -            case 10: path.moveTo(finitePts[f]); break;
 | 
| -        }
 | 
| -        SkPath result;
 | 
| -        result.setFillType(SkPath::kWinding_FillType);
 | 
| -        bool success = Simplify(path, &result);
 | 
| -        REPORTER_ASSERT(reporter, success);
 | 
| -        REPORTER_ASSERT(reporter, result.getFillType() != SkPath::kWinding_FillType);
 | 
| -        reporter->bumpTestCount();
 | 
| +        dontFailOne(reporter, index);
 | 
|      }
 | 
|  }
 | 
|  
 | 
| +static void PathOpsSimplifyFailOneTest(skiatest::Reporter* reporter) {
 | 
| +    int index = 0;
 | 
| +    failOne(reporter, index);
 | 
| +}
 | 
| +
 | 
| +static void PathOpsSimplifyDontFailOneTest(skiatest::Reporter* reporter) {
 | 
| +    int index = 6;
 | 
| +    dontFailOne(reporter, index);
 | 
| +}
 | 
| +
 | 
|  #include "TestClassDef.h"
 | 
|  DEFINE_TESTCLASS_SHORT(PathOpsSimplifyFailTest)
 | 
| +
 | 
| +DEFINE_TESTCLASS_SHORT(PathOpsSimplifyFailOneTest)
 | 
| +
 | 
| +DEFINE_TESTCLASS_SHORT(PathOpsSimplifyDontFailOneTest)
 | 
| 
 |