/* * Copyright 2016 Google Inc. * * Use of this source code is governed by a BSD-style license that can be * found in the LICENSE file. */ #include "include/core/SkCanvas.h" #include "include/core/SkPath.h" #include "include/core/SkSurface.h" #include "include/effects/SkDashPathEffect.h" #include "include/pathops/SkPathOps.h" #include "src/core/SkClipOpPriv.h" #include "src/core/SkRectPriv.h" #include "src/gpu/GrShape.h" #include "tests/Test.h" #include #include #include uint32_t GrShape::testingOnly_getOriginalGenerationID() const { if (const auto* lp = this->originalPathForListeners()) { return lp->getGenerationID(); } return SkPath().getGenerationID(); } bool GrShape::testingOnly_isPath() const { return Type::kPath == fType; } bool GrShape::testingOnly_isNonVolatilePath() const { return Type::kPath == fType && !fPathData.fPath.isVolatile(); } using Key = SkTArray; static bool make_key(Key* key, const GrShape& shape) { int size = shape.unstyledKeySize(); if (size <= 0) { key->reset(0); return false; } SkASSERT(size); key->reset(size); shape.writeUnstyledKey(key->begin()); return true; } static bool paths_fill_same(const SkPath& a, const SkPath& b) { SkPath pathXor; Op(a, b, SkPathOp::kXOR_SkPathOp, &pathXor); return pathXor.isEmpty(); } static bool test_bounds_by_rasterizing(const SkPath& path, const SkRect& bounds) { // We test the bounds by rasterizing the path into a kRes by kRes grid. The bounds is // mapped to the range kRes/4 to 3*kRes/4 in x and y. A difference clip is used to avoid // rendering within the bounds (with a tolerance). Then we render the path and check that // everything got clipped out. static constexpr int kRes = 2000; // This tolerance is in units of 1/kRes fractions of the bounds width/height. static constexpr int kTol = 2; GR_STATIC_ASSERT(kRes % 4 == 0); SkImageInfo info = SkImageInfo::MakeA8(kRes, kRes); sk_sp surface = SkSurface::MakeRaster(info); surface->getCanvas()->clear(0x0); SkRect clip = SkRect::MakeXYWH(kRes/4, kRes/4, kRes/2, kRes/2); SkMatrix matrix; matrix.setRectToRect(bounds, clip, SkMatrix::kFill_ScaleToFit); clip.outset(SkIntToScalar(kTol), SkIntToScalar(kTol)); surface->getCanvas()->clipRect(clip, kDifference_SkClipOp); surface->getCanvas()->concat(matrix); SkPaint whitePaint; whitePaint.setColor(SK_ColorWHITE); surface->getCanvas()->drawPath(path, whitePaint); SkPixmap pixmap; surface->getCanvas()->peekPixels(&pixmap); #if defined(SK_BUILD_FOR_WIN) // The static constexpr version in #else causes cl.exe to crash. const uint8_t* kZeros = reinterpret_cast(calloc(kRes, 1)); #else static constexpr uint8_t kZeros[kRes] = {0}; #endif for (int y = 0; y < kRes; ++y) { const uint8_t* row = pixmap.addr8(0, y); if (0 != memcmp(kZeros, row, kRes)) { return false; } } #ifdef SK_BUILD_FOR_WIN free(const_cast(kZeros)); #endif return true; } static bool can_interchange_winding_and_even_odd_fill(const GrShape& shape) { SkPath path; shape.asPath(&path); if (shape.style().hasNonDashPathEffect()) { return false; } const SkStrokeRec::Style strokeRecStyle = shape.style().strokeRec().getStyle(); return strokeRecStyle == SkStrokeRec::kStroke_Style || strokeRecStyle == SkStrokeRec::kHairline_Style || (shape.style().isSimpleFill() && path.isConvex()); } static void check_equivalence(skiatest::Reporter* r, const GrShape& a, const GrShape& b, const Key& keyA, const Key& keyB) { // GrShape only respects the input winding direction and start point for rrect shapes // when there is a path effect. Thus, if there are two GrShapes representing the same rrect // but one has a path effect in its style and the other doesn't then asPath() and the unstyled // key will differ. GrShape will have canonicalized the direction and start point for the shape // without the path effect. If *both* have path effects then they should have both preserved // the direction and starting point. // The asRRect() output params are all initialized just to silence compiler warnings about // uninitialized variables. SkRRect rrectA = SkRRect::MakeEmpty(), rrectB = SkRRect::MakeEmpty(); SkPath::Direction dirA = SkPath::kCW_Direction, dirB = SkPath::kCW_Direction; unsigned startA = ~0U, startB = ~0U; bool invertedA = true, invertedB = true; bool aIsRRect = a.asRRect(&rrectA, &dirA, &startA, &invertedA); bool bIsRRect = b.asRRect(&rrectB, &dirB, &startB, &invertedB); bool aHasPE = a.style().hasPathEffect(); bool bHasPE = b.style().hasPathEffect(); bool allowSameRRectButDiffStartAndDir = (aIsRRect && bIsRRect) && (aHasPE != bHasPE); // GrShape will close paths with simple fill style. bool allowedClosednessDiff = (a.style().isSimpleFill() != b.style().isSimpleFill()); SkPath pathA, pathB; a.asPath(&pathA); b.asPath(&pathB); // Having a dash path effect can allow 'a' but not 'b' to turn a inverse fill type into a // non-inverse fill type (or vice versa). bool ignoreInversenessDifference = false; if (pathA.isInverseFillType() != pathB.isInverseFillType()) { const GrShape* s1 = pathA.isInverseFillType() ? &a : &b; const GrShape* s2 = pathA.isInverseFillType() ? &b : &a; bool canDropInverse1 = s1->style().isDashed(); bool canDropInverse2 = s2->style().isDashed(); ignoreInversenessDifference = (canDropInverse1 != canDropInverse2); } bool ignoreWindingVsEvenOdd = false; if (SkPath::ConvertToNonInverseFillType(pathA.getFillType()) != SkPath::ConvertToNonInverseFillType(pathB.getFillType())) { bool aCanChange = can_interchange_winding_and_even_odd_fill(a); bool bCanChange = can_interchange_winding_and_even_odd_fill(b); if (aCanChange != bCanChange) { ignoreWindingVsEvenOdd = true; } } if (allowSameRRectButDiffStartAndDir) { REPORTER_ASSERT(r, rrectA == rrectB); REPORTER_ASSERT(r, paths_fill_same(pathA, pathB)); REPORTER_ASSERT(r, ignoreInversenessDifference || invertedA == invertedB); } else { SkPath pA = pathA; SkPath pB = pathB; REPORTER_ASSERT(r, a.inverseFilled() == pA.isInverseFillType()); REPORTER_ASSERT(r, b.inverseFilled() == pB.isInverseFillType()); if (ignoreInversenessDifference) { pA.setFillType(SkPath::ConvertToNonInverseFillType(pathA.getFillType())); pB.setFillType(SkPath::ConvertToNonInverseFillType(pathB.getFillType())); } if (ignoreWindingVsEvenOdd) { pA.setFillType(pA.isInverseFillType() ? SkPath::kInverseEvenOdd_FillType : SkPath::kEvenOdd_FillType); pB.setFillType(pB.isInverseFillType() ? SkPath::kInverseEvenOdd_FillType : SkPath::kEvenOdd_FillType); } if (!ignoreInversenessDifference && !ignoreWindingVsEvenOdd) { REPORTER_ASSERT(r, keyA == keyB); } else { REPORTER_ASSERT(r, keyA != keyB); } if (allowedClosednessDiff) { // GrShape will close paths with simple fill style. Make the non-filled path closed // so that the comparision will succeed. Make sure both are closed before comparing. pA.close(); pB.close(); } REPORTER_ASSERT(r, pA == pB); REPORTER_ASSERT(r, aIsRRect == bIsRRect); if (aIsRRect) { REPORTER_ASSERT(r, rrectA == rrectB); REPORTER_ASSERT(r, dirA == dirB); REPORTER_ASSERT(r, startA == startB); REPORTER_ASSERT(r, ignoreInversenessDifference || invertedA == invertedB); } } REPORTER_ASSERT(r, a.isEmpty() == b.isEmpty()); REPORTER_ASSERT(r, allowedClosednessDiff || a.knownToBeClosed() == b.knownToBeClosed()); // closedness can affect convexity. REPORTER_ASSERT(r, allowedClosednessDiff || a.knownToBeConvex() == b.knownToBeConvex()); if (a.knownToBeConvex()) { REPORTER_ASSERT(r, pathA.isConvex()); } if (b.knownToBeConvex()) { REPORTER_ASSERT(r, pathB.isConvex()); } REPORTER_ASSERT(r, a.bounds() == b.bounds()); REPORTER_ASSERT(r, a.segmentMask() == b.segmentMask()); // Init these to suppress warnings. SkPoint pts[4] {{0, 0,}, {0, 0}, {0, 0}, {0, 0}} ; bool invertedLine[2] {true, true}; REPORTER_ASSERT(r, a.asLine(pts, &invertedLine[0]) == b.asLine(pts + 2, &invertedLine[1])); // mayBeInverseFilledAfterStyling() is allowed to differ if one has a arbitrary PE and the other // doesn't (since the PE can set any fill type on its output path). // Moreover, dash style explicitly ignores inverseness. So if one is dashed but not the other // then they may disagree about inverseness. if (a.style().hasNonDashPathEffect() == b.style().hasNonDashPathEffect() && a.style().isDashed() == b.style().isDashed()) { REPORTER_ASSERT(r, a.mayBeInverseFilledAfterStyling() == b.mayBeInverseFilledAfterStyling()); } if (a.asLine(nullptr, nullptr)) { REPORTER_ASSERT(r, pts[2] == pts[0] && pts[3] == pts[1]); REPORTER_ASSERT(r, ignoreInversenessDifference || invertedLine[0] == invertedLine[1]); REPORTER_ASSERT(r, invertedLine[0] == a.inverseFilled()); REPORTER_ASSERT(r, invertedLine[1] == b.inverseFilled()); } REPORTER_ASSERT(r, ignoreInversenessDifference || a.inverseFilled() == b.inverseFilled()); } static void check_original_path_ids(skiatest::Reporter* r, const GrShape& base, const GrShape& pe, const GrShape& peStroke, const GrShape& full) { bool baseIsNonVolatilePath = base.testingOnly_isNonVolatilePath(); bool peIsPath = pe.testingOnly_isPath(); bool peStrokeIsPath = peStroke.testingOnly_isPath(); bool fullIsPath = full.testingOnly_isPath(); REPORTER_ASSERT(r, peStrokeIsPath == fullIsPath); uint32_t baseID = base.testingOnly_getOriginalGenerationID(); uint32_t peID = pe.testingOnly_getOriginalGenerationID(); uint32_t peStrokeID = peStroke.testingOnly_getOriginalGenerationID(); uint32_t fullID = full.testingOnly_getOriginalGenerationID(); // All empty paths have the same gen ID uint32_t emptyID = SkPath().getGenerationID(); // If we started with a real path, then our genID should match that path's gen ID (and not be // empty). If we started with a simple shape or a volatile path, our original path should have // been reset. REPORTER_ASSERT(r, baseIsNonVolatilePath == (baseID != emptyID)); // For the derived shapes, if they're simple types, their original paths should have been reset REPORTER_ASSERT(r, peIsPath || (peID == emptyID)); REPORTER_ASSERT(r, peStrokeIsPath || (peStrokeID == emptyID)); REPORTER_ASSERT(r, fullIsPath || (fullID == emptyID)); if (!peIsPath) { // If the path effect produces a simple shape, then there are no unbroken chains to test return; } // From here on, we know that the path effect produced a shape that was a "real" path if (baseIsNonVolatilePath) { REPORTER_ASSERT(r, baseID == peID); } if (peStrokeIsPath) { REPORTER_ASSERT(r, peID == peStrokeID); REPORTER_ASSERT(r, peStrokeID == fullID); } if (baseIsNonVolatilePath && peStrokeIsPath) { REPORTER_ASSERT(r, baseID == peStrokeID); REPORTER_ASSERT(r, baseID == fullID); } } void test_inversions(skiatest::Reporter* r, const GrShape& shape, const Key& shapeKey) { GrShape preserve = GrShape::MakeFilled(shape, GrShape::FillInversion::kPreserve); Key preserveKey; make_key(&preserveKey, preserve); GrShape flip = GrShape::MakeFilled(shape, GrShape::FillInversion::kFlip); Key flipKey; make_key(&flipKey, flip); GrShape inverted = GrShape::MakeFilled(shape, GrShape::FillInversion::kForceInverted); Key invertedKey; make_key(&invertedKey, inverted); GrShape noninverted = GrShape::MakeFilled(shape, GrShape::FillInversion::kForceNoninverted); Key noninvertedKey; make_key(&noninvertedKey, noninverted); if (invertedKey.count() || noninvertedKey.count()) { REPORTER_ASSERT(r, invertedKey != noninvertedKey); } if (shape.style().isSimpleFill()) { check_equivalence(r, shape, preserve, shapeKey, preserveKey); } if (shape.inverseFilled()) { check_equivalence(r, preserve, inverted, preserveKey, invertedKey); check_equivalence(r, flip, noninverted, flipKey, noninvertedKey); } else { check_equivalence(r, preserve, noninverted, preserveKey, noninvertedKey); check_equivalence(r, flip, inverted, flipKey, invertedKey); } GrShape doubleFlip = GrShape::MakeFilled(flip, GrShape::FillInversion::kFlip); Key doubleFlipKey; make_key(&doubleFlipKey, doubleFlip); // It can be the case that the double flip has no key but preserve does. This happens when the // original shape has an inherited style key. That gets dropped on the first inversion flip. if (preserveKey.count() && !doubleFlipKey.count()) { preserveKey.reset(); } check_equivalence(r, preserve, doubleFlip, preserveKey, doubleFlipKey); } namespace { /** * Geo is a factory for creating a GrShape from another representation. It also answers some * questions about expected behavior for GrShape given the inputs. */ class Geo { public: virtual ~Geo() {} virtual GrShape makeShape(const SkPaint&) const = 0; virtual SkPath path() const = 0; // These functions allow tests to check for special cases where style gets // applied by GrShape in its constructor (without calling GrShape::applyStyle). // These unfortunately rely on knowing details of GrShape's implementation. // These predicates are factored out here to avoid littering the rest of the // test code with GrShape implementation details. virtual bool fillChangesGeom() const { return false; } virtual bool strokeIsConvertedToFill() const { return false; } virtual bool strokeAndFillIsConvertedToFill(const SkPaint&) const { return false; } // Is this something we expect GrShape to recognize as something simpler than a path. virtual bool isNonPath(const SkPaint& paint) const { return true; } }; class RectGeo : public Geo { public: RectGeo(const SkRect& rect) : fRect(rect) {} SkPath path() const override { SkPath path; path.addRect(fRect); return path; } GrShape makeShape(const SkPaint& paint) const override { return GrShape(fRect, paint); } bool strokeAndFillIsConvertedToFill(const SkPaint& paint) const override { SkASSERT(paint.getStyle() == SkPaint::kStrokeAndFill_Style); // Converted to an outset rectangle. return paint.getStrokeJoin() == SkPaint::kMiter_Join && paint.getStrokeMiter() >= SK_ScalarSqrt2; } private: SkRect fRect; }; class RRectGeo : public Geo { public: RRectGeo(const SkRRect& rrect) : fRRect(rrect) {} GrShape makeShape(const SkPaint& paint) const override { return GrShape(fRRect, paint); } SkPath path() const override { SkPath path; path.addRRect(fRRect); return path; } bool strokeAndFillIsConvertedToFill(const SkPaint& paint) const override { SkASSERT(paint.getStyle() == SkPaint::kStrokeAndFill_Style); if (fRRect.isRect()) { return RectGeo(fRRect.rect()).strokeAndFillIsConvertedToFill(paint); } return false; } private: SkRRect fRRect; }; class ArcGeo : public Geo { public: ArcGeo(const SkRect& oval, SkScalar startAngle, SkScalar sweepAngle, bool useCenter) : fOval(oval) , fStartAngle(startAngle) , fSweepAngle(sweepAngle) , fUseCenter(useCenter) {} SkPath path() const override { SkPath path; SkPathPriv::CreateDrawArcPath(&path, fOval, fStartAngle, fSweepAngle, fUseCenter, false); return path; } GrShape makeShape(const SkPaint& paint) const override { return GrShape::MakeArc(fOval, fStartAngle, fSweepAngle, fUseCenter, GrStyle(paint)); } // GrShape specializes when created from arc params but it doesn't recognize arcs from SkPath. bool isNonPath(const SkPaint& paint) const override { return false; } private: SkRect fOval; SkScalar fStartAngle; SkScalar fSweepAngle; bool fUseCenter; }; class PathGeo : public Geo { public: enum class Invert { kNo, kYes }; PathGeo(const SkPath& path, Invert invert) : fPath(path) { SkASSERT(!path.isInverseFillType()); if (Invert::kYes == invert) { if (fPath.getFillType() == SkPath::kEvenOdd_FillType) { fPath.setFillType(SkPath::kInverseEvenOdd_FillType); } else { SkASSERT(fPath.getFillType() == SkPath::kWinding_FillType); fPath.setFillType(SkPath::kInverseWinding_FillType); } } } GrShape makeShape(const SkPaint& paint) const override { return GrShape(fPath, paint); } SkPath path() const override { return fPath; } bool fillChangesGeom() const override { // unclosed rects get closed. Lines get turned into empty geometry return this->isUnclosedRect() || fPath.isLine(nullptr); } bool strokeIsConvertedToFill() const override { return this->isAxisAlignedLine(); } bool strokeAndFillIsConvertedToFill(const SkPaint& paint) const override { SkASSERT(paint.getStyle() == SkPaint::kStrokeAndFill_Style); if (this->isAxisAlignedLine()) { // The fill is ignored (zero area) and the stroke is converted to a rrect. return true; } SkRect rect; unsigned start; SkPath::Direction dir; if (SkPathPriv::IsSimpleClosedRect(fPath, &rect, &dir, &start)) { return RectGeo(rect).strokeAndFillIsConvertedToFill(paint); } return false; } bool isNonPath(const SkPaint& paint) const override { return fPath.isLine(nullptr) || fPath.isEmpty(); } private: bool isAxisAlignedLine() const { SkPoint pts[2]; if (!fPath.isLine(pts)) { return false; } return pts[0].fX == pts[1].fX || pts[0].fY == pts[1].fY; } bool isUnclosedRect() const { bool closed; return fPath.isRect(nullptr, &closed, nullptr) && !closed; } SkPath fPath; }; class RRectPathGeo : public PathGeo { public: enum class RRectForStroke { kNo, kYes }; RRectPathGeo(const SkPath& path, const SkRRect& equivalentRRect, RRectForStroke rrectForStroke, Invert invert) : PathGeo(path, invert) , fRRect(equivalentRRect) , fRRectForStroke(rrectForStroke) {} RRectPathGeo(const SkPath& path, const SkRect& equivalentRect, RRectForStroke rrectForStroke, Invert invert) : RRectPathGeo(path, SkRRect::MakeRect(equivalentRect), rrectForStroke, invert) {} bool isNonPath(const SkPaint& paint) const override { if (SkPaint::kFill_Style == paint.getStyle() || RRectForStroke::kYes == fRRectForStroke) { return true; } return false; } const SkRRect& rrect() const { return fRRect; } private: SkRRect fRRect; RRectForStroke fRRectForStroke; }; class TestCase { public: TestCase(const Geo& geo, const SkPaint& paint, skiatest::Reporter* r, SkScalar scale = SK_Scalar1) : fBase(new GrShape(geo.makeShape(paint))) { this->init(r, scale); } template TestCase(skiatest::Reporter* r, ShapeArgs... shapeArgs) : fBase(new GrShape(shapeArgs...)) { this->init(r, SK_Scalar1); } TestCase(const GrShape& shape, skiatest::Reporter* r, SkScalar scale = SK_Scalar1) : fBase(new GrShape(shape)) { this->init(r, scale); } struct SelfExpectations { bool fPEHasEffect; bool fPEHasValidKey; bool fStrokeApplies; }; void testExpectations(skiatest::Reporter* reporter, SelfExpectations expectations) const; enum ComparisonExpecation { kAllDifferent_ComparisonExpecation, kSameUpToPE_ComparisonExpecation, kSameUpToStroke_ComparisonExpecation, kAllSame_ComparisonExpecation, }; void compare(skiatest::Reporter*, const TestCase& that, ComparisonExpecation) const; const GrShape& baseShape() const { return *fBase; } const GrShape& appliedPathEffectShape() const { return *fAppliedPE; } const GrShape& appliedFullStyleShape() const { return *fAppliedFull; } // The returned array's count will be 0 if the key shape has no key. const Key& baseKey() const { return fBaseKey; } const Key& appliedPathEffectKey() const { return fAppliedPEKey; } const Key& appliedFullStyleKey() const { return fAppliedFullKey; } const Key& appliedPathEffectThenStrokeKey() const { return fAppliedPEThenStrokeKey; } private: static void CheckBounds(skiatest::Reporter* r, const GrShape& shape, const SkRect& bounds) { SkPath path; shape.asPath(&path); // If the bounds are empty, the path ought to be as well. if (bounds.fLeft > bounds.fRight || bounds.fTop > bounds.fBottom) { REPORTER_ASSERT(r, path.isEmpty()); return; } if (path.isEmpty()) { return; } // The bounds API explicitly calls out that it does not consider inverseness. SkPath p = path; p.setFillType(SkPath::ConvertToNonInverseFillType(path.getFillType())); REPORTER_ASSERT(r, test_bounds_by_rasterizing(p, bounds)); } void init(skiatest::Reporter* r, SkScalar scale) { fAppliedPE.reset(new GrShape); fAppliedPEThenStroke.reset(new GrShape); fAppliedFull.reset(new GrShape); *fAppliedPE = fBase->applyStyle(GrStyle::Apply::kPathEffectOnly, scale); *fAppliedPEThenStroke = fAppliedPE->applyStyle(GrStyle::Apply::kPathEffectAndStrokeRec, scale); *fAppliedFull = fBase->applyStyle(GrStyle::Apply::kPathEffectAndStrokeRec, scale); make_key(&fBaseKey, *fBase); make_key(&fAppliedPEKey, *fAppliedPE); make_key(&fAppliedPEThenStrokeKey, *fAppliedPEThenStroke); make_key(&fAppliedFullKey, *fAppliedFull); // All shapes should report the same "original" path, so that path renderers can get to it // if necessary. check_original_path_ids(r, *fBase, *fAppliedPE, *fAppliedPEThenStroke, *fAppliedFull); // Applying the path effect and then the stroke should always be the same as applying // both in one go. REPORTER_ASSERT(r, fAppliedPEThenStrokeKey == fAppliedFullKey); SkPath a, b; fAppliedPEThenStroke->asPath(&a); fAppliedFull->asPath(&b); // If the output of the path effect is a rrect then it is possible for a and b to be // different paths that fill identically. The reason is that fAppliedFull will do this: // base -> apply path effect -> rrect_as_path -> stroke -> stroked_rrect_as_path // fAppliedPEThenStroke will have converted the rrect_as_path back to a rrect. However, // now that there is no longer a path effect, the direction and starting index get // canonicalized before the stroke. if (fAppliedPE->asRRect(nullptr, nullptr, nullptr, nullptr)) { REPORTER_ASSERT(r, paths_fill_same(a, b)); } else { REPORTER_ASSERT(r, a == b); } REPORTER_ASSERT(r, fAppliedFull->isEmpty() == fAppliedPEThenStroke->isEmpty()); SkPath path; fBase->asPath(&path); REPORTER_ASSERT(r, path.isEmpty() == fBase->isEmpty()); REPORTER_ASSERT(r, path.getSegmentMasks() == fBase->segmentMask()); fAppliedPE->asPath(&path); REPORTER_ASSERT(r, path.isEmpty() == fAppliedPE->isEmpty()); REPORTER_ASSERT(r, path.getSegmentMasks() == fAppliedPE->segmentMask()); fAppliedFull->asPath(&path); REPORTER_ASSERT(r, path.isEmpty() == fAppliedFull->isEmpty()); REPORTER_ASSERT(r, path.getSegmentMasks() == fAppliedFull->segmentMask()); CheckBounds(r, *fBase, fBase->bounds()); CheckBounds(r, *fAppliedPE, fAppliedPE->bounds()); CheckBounds(r, *fAppliedPEThenStroke, fAppliedPEThenStroke->bounds()); CheckBounds(r, *fAppliedFull, fAppliedFull->bounds()); SkRect styledBounds = fBase->styledBounds(); CheckBounds(r, *fAppliedFull, styledBounds); styledBounds = fAppliedPE->styledBounds(); CheckBounds(r, *fAppliedFull, styledBounds); // Check that the same path is produced when style is applied by GrShape and GrStyle. SkPath preStyle; SkPath postPathEffect; SkPath postAllStyle; fBase->asPath(&preStyle); SkStrokeRec postPEStrokeRec(SkStrokeRec::kFill_InitStyle); if (fBase->style().applyPathEffectToPath(&postPathEffect, &postPEStrokeRec, preStyle, scale)) { // run postPathEffect through GrShape to get any geometry reductions that would have // occurred to fAppliedPE. GrShape(postPathEffect, GrStyle(postPEStrokeRec, nullptr)).asPath(&postPathEffect); SkPath testPath; fAppliedPE->asPath(&testPath); REPORTER_ASSERT(r, testPath == postPathEffect); REPORTER_ASSERT(r, postPEStrokeRec.hasEqualEffect(fAppliedPE->style().strokeRec())); } SkStrokeRec::InitStyle fillOrHairline; if (fBase->style().applyToPath(&postAllStyle, &fillOrHairline, preStyle, scale)) { SkPath testPath; fAppliedFull->asPath(&testPath); if (fBase->style().hasPathEffect()) { // Because GrShape always does two-stage application when there is a path effect // there may be a reduction/canonicalization step between the path effect and // strokerec not reflected in postAllStyle since it applied both the path effect // and strokerec without analyzing the intermediate path. REPORTER_ASSERT(r, paths_fill_same(postAllStyle, testPath)); } else { // Make sure that postAllStyle sees any reductions/canonicalizations that GrShape // would apply. GrShape(postAllStyle, GrStyle(fillOrHairline)).asPath(&postAllStyle); REPORTER_ASSERT(r, testPath == postAllStyle); } if (fillOrHairline == SkStrokeRec::kFill_InitStyle) { REPORTER_ASSERT(r, fAppliedFull->style().isSimpleFill()); } else { REPORTER_ASSERT(r, fAppliedFull->style().isSimpleHairline()); } } test_inversions(r, *fBase, fBaseKey); test_inversions(r, *fAppliedPE, fAppliedPEKey); test_inversions(r, *fAppliedFull, fAppliedFullKey); } std::unique_ptr fBase; std::unique_ptr fAppliedPE; std::unique_ptr fAppliedPEThenStroke; std::unique_ptr fAppliedFull; Key fBaseKey; Key fAppliedPEKey; Key fAppliedPEThenStrokeKey; Key fAppliedFullKey; }; void TestCase::testExpectations(skiatest::Reporter* reporter, SelfExpectations expectations) const { // The base's key should always be valid (unless the path is volatile) REPORTER_ASSERT(reporter, fBaseKey.count()); if (expectations.fPEHasEffect) { REPORTER_ASSERT(reporter, fBaseKey != fAppliedPEKey); REPORTER_ASSERT(reporter, expectations.fPEHasValidKey == SkToBool(fAppliedPEKey.count())); REPORTER_ASSERT(reporter, fBaseKey != fAppliedFullKey); REPORTER_ASSERT(reporter, expectations.fPEHasValidKey == SkToBool(fAppliedFullKey.count())); if (expectations.fStrokeApplies && expectations.fPEHasValidKey) { REPORTER_ASSERT(reporter, fAppliedPEKey != fAppliedFullKey); REPORTER_ASSERT(reporter, SkToBool(fAppliedFullKey.count())); } } else { REPORTER_ASSERT(reporter, fBaseKey == fAppliedPEKey); SkPath a, b; fBase->asPath(&a); fAppliedPE->asPath(&b); REPORTER_ASSERT(reporter, a == b); if (expectations.fStrokeApplies) { REPORTER_ASSERT(reporter, fBaseKey != fAppliedFullKey); } else { REPORTER_ASSERT(reporter, fBaseKey == fAppliedFullKey); } } } void TestCase::compare(skiatest::Reporter* r, const TestCase& that, ComparisonExpecation expectation) const { SkPath a, b; switch (expectation) { case kAllDifferent_ComparisonExpecation: REPORTER_ASSERT(r, fBaseKey != that.fBaseKey); REPORTER_ASSERT(r, fAppliedPEKey != that.fAppliedPEKey); REPORTER_ASSERT(r, fAppliedFullKey != that.fAppliedFullKey); break; case kSameUpToPE_ComparisonExpecation: check_equivalence(r, *fBase, *that.fBase, fBaseKey, that.fBaseKey); REPORTER_ASSERT(r, fAppliedPEKey != that.fAppliedPEKey); REPORTER_ASSERT(r, fAppliedFullKey != that.fAppliedFullKey); break; case kSameUpToStroke_ComparisonExpecation: check_equivalence(r, *fBase, *that.fBase, fBaseKey, that.fBaseKey); check_equivalence(r, *fAppliedPE, *that.fAppliedPE, fAppliedPEKey, that.fAppliedPEKey); REPORTER_ASSERT(r, fAppliedFullKey != that.fAppliedFullKey); break; case kAllSame_ComparisonExpecation: check_equivalence(r, *fBase, *that.fBase, fBaseKey, that.fBaseKey); check_equivalence(r, *fAppliedPE, *that.fAppliedPE, fAppliedPEKey, that.fAppliedPEKey); check_equivalence(r, *fAppliedFull, *that.fAppliedFull, fAppliedFullKey, that.fAppliedFullKey); break; } } } // namespace static sk_sp make_dash() { static const SkScalar kIntervals[] = { 0.25, 3.f, 0.5, 2.f }; static const SkScalar kPhase = 0.75; return SkDashPathEffect::Make(kIntervals, SK_ARRAY_COUNT(kIntervals), kPhase); } static sk_sp make_null_dash() { static const SkScalar kNullIntervals[] = {0, 0, 0, 0, 0, 0}; return SkDashPathEffect::Make(kNullIntervals, SK_ARRAY_COUNT(kNullIntervals), 0.f); } // We make enough TestCases, and they're large enough, that on Google3 builds we exceed // the maximum stack frame limit. make_TestCase() moves those temporaries over to the heap. template static std::unique_ptr make_TestCase(Args&&... args) { return std::unique_ptr{ new TestCase(std::forward(args)...) }; } static void test_basic(skiatest::Reporter* reporter, const Geo& geo) { sk_sp dashPE = make_dash(); TestCase::SelfExpectations expectations; SkPaint fill; TestCase fillCase(geo, fill, reporter); expectations.fPEHasEffect = false; expectations.fPEHasValidKey = false; expectations.fStrokeApplies = false; fillCase.testExpectations(reporter, expectations); // Test that another GrShape instance built from the same primitive is the same. make_TestCase(geo, fill, reporter) ->compare(reporter, fillCase, TestCase::kAllSame_ComparisonExpecation); SkPaint stroke2RoundBevel; stroke2RoundBevel.setStyle(SkPaint::kStroke_Style); stroke2RoundBevel.setStrokeCap(SkPaint::kRound_Cap); stroke2RoundBevel.setStrokeJoin(SkPaint::kBevel_Join); stroke2RoundBevel.setStrokeWidth(2.f); TestCase stroke2RoundBevelCase(geo, stroke2RoundBevel, reporter); expectations.fPEHasValidKey = true; expectations.fPEHasEffect = false; expectations.fStrokeApplies = !geo.strokeIsConvertedToFill(); stroke2RoundBevelCase.testExpectations(reporter, expectations); make_TestCase(geo, stroke2RoundBevel, reporter) ->compare(reporter, stroke2RoundBevelCase, TestCase::kAllSame_ComparisonExpecation); SkPaint stroke2RoundBevelDash = stroke2RoundBevel; stroke2RoundBevelDash.setPathEffect(make_dash()); TestCase stroke2RoundBevelDashCase(geo, stroke2RoundBevelDash, reporter); expectations.fPEHasValidKey = true; expectations.fPEHasEffect = true; expectations.fStrokeApplies = true; stroke2RoundBevelDashCase.testExpectations(reporter, expectations); make_TestCase(geo, stroke2RoundBevelDash, reporter) ->compare(reporter, stroke2RoundBevelDashCase, TestCase::kAllSame_ComparisonExpecation); if (geo.fillChangesGeom() || geo.strokeIsConvertedToFill()) { fillCase.compare(reporter, stroke2RoundBevelCase, TestCase::kAllDifferent_ComparisonExpecation); fillCase.compare(reporter, stroke2RoundBevelDashCase, TestCase::kAllDifferent_ComparisonExpecation); } else { fillCase.compare(reporter, stroke2RoundBevelCase, TestCase::kSameUpToStroke_ComparisonExpecation); fillCase.compare(reporter, stroke2RoundBevelDashCase, TestCase::kSameUpToPE_ComparisonExpecation); } if (geo.strokeIsConvertedToFill()) { stroke2RoundBevelCase.compare(reporter, stroke2RoundBevelDashCase, TestCase::kAllDifferent_ComparisonExpecation); } else { stroke2RoundBevelCase.compare(reporter, stroke2RoundBevelDashCase, TestCase::kSameUpToPE_ComparisonExpecation); } // Stroke and fill cases SkPaint stroke2RoundBevelAndFill = stroke2RoundBevel; stroke2RoundBevelAndFill.setStyle(SkPaint::kStrokeAndFill_Style); TestCase stroke2RoundBevelAndFillCase(geo, stroke2RoundBevelAndFill, reporter); expectations.fPEHasValidKey = true; expectations.fPEHasEffect = false; expectations.fStrokeApplies = !geo.strokeIsConvertedToFill(); stroke2RoundBevelAndFillCase.testExpectations(reporter, expectations); make_TestCase(geo, stroke2RoundBevelAndFill, reporter)->compare( reporter, stroke2RoundBevelAndFillCase, TestCase::kAllSame_ComparisonExpecation); SkPaint stroke2RoundBevelAndFillDash = stroke2RoundBevelDash; stroke2RoundBevelAndFillDash.setStyle(SkPaint::kStrokeAndFill_Style); TestCase stroke2RoundBevelAndFillDashCase(geo, stroke2RoundBevelAndFillDash, reporter); expectations.fPEHasValidKey = true; expectations.fPEHasEffect = false; expectations.fStrokeApplies = !geo.strokeIsConvertedToFill(); stroke2RoundBevelAndFillDashCase.testExpectations(reporter, expectations); make_TestCase(geo, stroke2RoundBevelAndFillDash, reporter)->compare( reporter, stroke2RoundBevelAndFillDashCase, TestCase::kAllSame_ComparisonExpecation); stroke2RoundBevelAndFillDashCase.compare(reporter, stroke2RoundBevelAndFillCase, TestCase::kAllSame_ComparisonExpecation); SkPaint hairline; hairline.setStyle(SkPaint::kStroke_Style); hairline.setStrokeWidth(0.f); TestCase hairlineCase(geo, hairline, reporter); // Since hairline style doesn't change the SkPath data, it is keyed identically to fill (except // in the line and unclosed rect cases). if (geo.fillChangesGeom()) { hairlineCase.compare(reporter, fillCase, TestCase::kAllDifferent_ComparisonExpecation); } else { hairlineCase.compare(reporter, fillCase, TestCase::kAllSame_ComparisonExpecation); } REPORTER_ASSERT(reporter, hairlineCase.baseShape().style().isSimpleHairline()); REPORTER_ASSERT(reporter, hairlineCase.appliedFullStyleShape().style().isSimpleHairline()); REPORTER_ASSERT(reporter, hairlineCase.appliedPathEffectShape().style().isSimpleHairline()); } static void test_scale(skiatest::Reporter* reporter, const Geo& geo) { sk_sp dashPE = make_dash(); static const SkScalar kS1 = 1.f; static const SkScalar kS2 = 2.f; SkPaint fill; TestCase fillCase1(geo, fill, reporter, kS1); TestCase fillCase2(geo, fill, reporter, kS2); // Scale doesn't affect fills. fillCase1.compare(reporter, fillCase2, TestCase::kAllSame_ComparisonExpecation); SkPaint hairline; hairline.setStyle(SkPaint::kStroke_Style); hairline.setStrokeWidth(0.f); TestCase hairlineCase1(geo, hairline, reporter, kS1); TestCase hairlineCase2(geo, hairline, reporter, kS2); // Scale doesn't affect hairlines. hairlineCase1.compare(reporter, hairlineCase2, TestCase::kAllSame_ComparisonExpecation); SkPaint stroke; stroke.setStyle(SkPaint::kStroke_Style); stroke.setStrokeWidth(2.f); TestCase strokeCase1(geo, stroke, reporter, kS1); TestCase strokeCase2(geo, stroke, reporter, kS2); // Scale affects the stroke if (geo.strokeIsConvertedToFill()) { REPORTER_ASSERT(reporter, !strokeCase1.baseShape().style().applies()); strokeCase1.compare(reporter, strokeCase2, TestCase::kAllSame_ComparisonExpecation); } else { strokeCase1.compare(reporter, strokeCase2, TestCase::kSameUpToStroke_ComparisonExpecation); } SkPaint strokeDash = stroke; strokeDash.setPathEffect(make_dash()); TestCase strokeDashCase1(geo, strokeDash, reporter, kS1); TestCase strokeDashCase2(geo, strokeDash, reporter, kS2); // Scale affects the dash and the stroke. strokeDashCase1.compare(reporter, strokeDashCase2, TestCase::kSameUpToPE_ComparisonExpecation); // Stroke and fill cases SkPaint strokeAndFill = stroke; strokeAndFill.setStyle(SkPaint::kStrokeAndFill_Style); TestCase strokeAndFillCase1(geo, strokeAndFill, reporter, kS1); TestCase strokeAndFillCase2(geo, strokeAndFill, reporter, kS2); SkPaint strokeAndFillDash = strokeDash; strokeAndFillDash.setStyle(SkPaint::kStrokeAndFill_Style); // Dash is ignored for stroke and fill TestCase strokeAndFillDashCase1(geo, strokeAndFillDash, reporter, kS1); TestCase strokeAndFillDashCase2(geo, strokeAndFillDash, reporter, kS2); // Scale affects the stroke, but check to make sure this didn't become a simpler shape (e.g. // stroke-and-filled rect can become a rect), in which case the scale shouldn't matter and the // geometries should agree. if (geo.strokeAndFillIsConvertedToFill(strokeAndFillDash)) { REPORTER_ASSERT(reporter, !strokeAndFillCase1.baseShape().style().applies()); strokeAndFillCase1.compare(reporter, strokeAndFillCase2, TestCase::kAllSame_ComparisonExpecation); strokeAndFillDashCase1.compare(reporter, strokeAndFillDashCase2, TestCase::kAllSame_ComparisonExpecation); } else { strokeAndFillCase1.compare(reporter, strokeAndFillCase2, TestCase::kSameUpToStroke_ComparisonExpecation); } strokeAndFillDashCase1.compare(reporter, strokeAndFillCase1, TestCase::kAllSame_ComparisonExpecation); strokeAndFillDashCase2.compare(reporter, strokeAndFillCase2, TestCase::kAllSame_ComparisonExpecation); } template static void test_stroke_param_impl(skiatest::Reporter* reporter, const Geo& geo, std::function setter, T a, T b, bool paramAffectsStroke, bool paramAffectsDashAndStroke) { // Set the stroke width so that we don't get hairline. However, call the setter afterward so // that it can override the stroke width. SkPaint strokeA; strokeA.setStyle(SkPaint::kStroke_Style); strokeA.setStrokeWidth(2.f); setter(&strokeA, a); SkPaint strokeB; strokeB.setStyle(SkPaint::kStroke_Style); strokeB.setStrokeWidth(2.f); setter(&strokeB, b); TestCase strokeACase(geo, strokeA, reporter); TestCase strokeBCase(geo, strokeB, reporter); if (paramAffectsStroke) { // If stroking is immediately incorporated into a geometric transformation then the base // shapes will differ. if (geo.strokeIsConvertedToFill()) { strokeACase.compare(reporter, strokeBCase, TestCase::kAllDifferent_ComparisonExpecation); } else { strokeACase.compare(reporter, strokeBCase, TestCase::kSameUpToStroke_ComparisonExpecation); } } else { strokeACase.compare(reporter, strokeBCase, TestCase::kAllSame_ComparisonExpecation); } SkPaint strokeAndFillA = strokeA; SkPaint strokeAndFillB = strokeB; strokeAndFillA.setStyle(SkPaint::kStrokeAndFill_Style); strokeAndFillB.setStyle(SkPaint::kStrokeAndFill_Style); TestCase strokeAndFillACase(geo, strokeAndFillA, reporter); TestCase strokeAndFillBCase(geo, strokeAndFillB, reporter); if (paramAffectsStroke) { // If stroking is immediately incorporated into a geometric transformation then the base // shapes will differ. if (geo.strokeAndFillIsConvertedToFill(strokeAndFillA) || geo.strokeAndFillIsConvertedToFill(strokeAndFillB)) { strokeAndFillACase.compare(reporter, strokeAndFillBCase, TestCase::kAllDifferent_ComparisonExpecation); } else { strokeAndFillACase.compare(reporter, strokeAndFillBCase, TestCase::kSameUpToStroke_ComparisonExpecation); } } else { strokeAndFillACase.compare(reporter, strokeAndFillBCase, TestCase::kAllSame_ComparisonExpecation); } // Make sure stroking params don't affect fill style. SkPaint fillA = strokeA, fillB = strokeB; fillA.setStyle(SkPaint::kFill_Style); fillB.setStyle(SkPaint::kFill_Style); TestCase fillACase(geo, fillA, reporter); TestCase fillBCase(geo, fillB, reporter); fillACase.compare(reporter, fillBCase, TestCase::kAllSame_ComparisonExpecation); // Make sure just applying the dash but not stroke gives the same key for both stroking // variations. SkPaint dashA = strokeA, dashB = strokeB; dashA.setPathEffect(make_dash()); dashB.setPathEffect(make_dash()); TestCase dashACase(geo, dashA, reporter); TestCase dashBCase(geo, dashB, reporter); if (paramAffectsDashAndStroke) { dashACase.compare(reporter, dashBCase, TestCase::kSameUpToStroke_ComparisonExpecation); } else { dashACase.compare(reporter, dashBCase, TestCase::kAllSame_ComparisonExpecation); } } template static void test_stroke_param(skiatest::Reporter* reporter, const Geo& geo, std::function setter, T a, T b) { test_stroke_param_impl(reporter, geo, setter, a, b, true, true); }; static void test_stroke_cap(skiatest::Reporter* reporter, const Geo& geo) { SkPaint hairline; hairline.setStrokeWidth(0); hairline.setStyle(SkPaint::kStroke_Style); GrShape shape = geo.makeShape(hairline); // The cap should only affect shapes that may be open. bool affectsStroke = !shape.knownToBeClosed(); // Dashing adds ends that need caps. bool affectsDashAndStroke = true; test_stroke_param_impl( reporter, geo, [](SkPaint* p, SkPaint::Cap c) { p->setStrokeCap(c);}, SkPaint::kButt_Cap, SkPaint::kRound_Cap, affectsStroke, affectsDashAndStroke); }; static bool shape_known_not_to_have_joins(const GrShape& shape) { return shape.asLine(nullptr, nullptr) || shape.isEmpty(); } static void test_stroke_join(skiatest::Reporter* reporter, const Geo& geo) { SkPaint hairline; hairline.setStrokeWidth(0); hairline.setStyle(SkPaint::kStroke_Style); GrShape shape = geo.makeShape(hairline); // GrShape recognizes certain types don't have joins and will prevent the join type from // affecting the style key. // Dashing doesn't add additional joins. However, GrShape currently loses track of this // after applying the dash. bool affectsStroke = !shape_known_not_to_have_joins(shape); test_stroke_param_impl( reporter, geo, [](SkPaint* p, SkPaint::Join j) { p->setStrokeJoin(j);}, SkPaint::kRound_Join, SkPaint::kBevel_Join, affectsStroke, true); }; static void test_miter_limit(skiatest::Reporter* reporter, const Geo& geo) { auto setMiterJoinAndLimit = [](SkPaint* p, SkScalar miter) { p->setStrokeJoin(SkPaint::kMiter_Join); p->setStrokeMiter(miter); }; auto setOtherJoinAndLimit = [](SkPaint* p, SkScalar miter) { p->setStrokeJoin(SkPaint::kRound_Join); p->setStrokeMiter(miter); }; SkPaint hairline; hairline.setStrokeWidth(0); hairline.setStyle(SkPaint::kStroke_Style); GrShape shape = geo.makeShape(hairline); bool mayHaveJoins = !shape_known_not_to_have_joins(shape); // The miter limit should affect stroked and dashed-stroked cases when the join type is // miter. test_stroke_param_impl( reporter, geo, setMiterJoinAndLimit, 0.5f, 0.75f, mayHaveJoins, true); // The miter limit should not affect stroked and dashed-stroked cases when the join type is // not miter. test_stroke_param_impl( reporter, geo, setOtherJoinAndLimit, 0.5f, 0.75f, false, false); } static void test_dash_fill(skiatest::Reporter* reporter, const Geo& geo) { // A dash with no stroke should have no effect using DashFactoryFn = sk_sp(*)(); for (DashFactoryFn md : {&make_dash, &make_null_dash}) { SkPaint dashFill; dashFill.setPathEffect((*md)()); TestCase dashFillCase(geo, dashFill, reporter); TestCase fillCase(geo, SkPaint(), reporter); dashFillCase.compare(reporter, fillCase, TestCase::kAllSame_ComparisonExpecation); } } void test_null_dash(skiatest::Reporter* reporter, const Geo& geo) { SkPaint fill; SkPaint stroke; stroke.setStyle(SkPaint::kStroke_Style); stroke.setStrokeWidth(1.f); SkPaint dash; dash.setStyle(SkPaint::kStroke_Style); dash.setStrokeWidth(1.f); dash.setPathEffect(make_dash()); SkPaint nullDash; nullDash.setStyle(SkPaint::kStroke_Style); nullDash.setStrokeWidth(1.f); nullDash.setPathEffect(make_null_dash()); TestCase fillCase(geo, fill, reporter); TestCase strokeCase(geo, stroke, reporter); TestCase dashCase(geo, dash, reporter); TestCase nullDashCase(geo, nullDash, reporter); // We expect the null dash to be ignored so nullDashCase should match strokeCase, always. nullDashCase.compare(reporter, strokeCase, TestCase::kAllSame_ComparisonExpecation); // Check whether the fillCase or strokeCase/nullDashCase would undergo a geometric tranformation // on construction in order to determine how to compare the fill and stroke. if (geo.fillChangesGeom() || geo.strokeIsConvertedToFill()) { nullDashCase.compare(reporter, fillCase, TestCase::kAllDifferent_ComparisonExpecation); } else { nullDashCase.compare(reporter, fillCase, TestCase::kSameUpToStroke_ComparisonExpecation); } // In the null dash case we may immediately convert to a fill, but not for the normal dash case. if (geo.strokeIsConvertedToFill()) { nullDashCase.compare(reporter, dashCase, TestCase::kAllDifferent_ComparisonExpecation); } else { nullDashCase.compare(reporter, dashCase, TestCase::kSameUpToPE_ComparisonExpecation); } } void test_path_effect_makes_rrect(skiatest::Reporter* reporter, const Geo& geo) { /** * This path effect takes any input path and turns it into a rrect. It passes through stroke * info. */ class RRectPathEffect : SkPathEffect { public: static const SkRRect& RRect() { static const SkRRect kRRect = SkRRect::MakeRectXY(SkRect::MakeWH(12, 12), 3, 5); return kRRect; } static sk_sp Make() { return sk_sp(new RRectPathEffect); } Factory getFactory() const override { return nullptr; } const char* getTypeName() const override { return nullptr; } protected: bool onFilterPath(SkPath* dst, const SkPath& src, SkStrokeRec*, const SkRect* cullR) const override { dst->reset(); dst->addRRect(RRect()); return true; } SkRect onComputeFastBounds(const SkRect& src) const override { return RRect().getBounds(); } private: RRectPathEffect() {} }; SkPaint fill; TestCase fillGeoCase(geo, fill, reporter); SkPaint pe; pe.setPathEffect(RRectPathEffect::Make()); TestCase geoPECase(geo, pe, reporter); SkPaint peStroke; peStroke.setPathEffect(RRectPathEffect::Make()); peStroke.setStrokeWidth(2.f); peStroke.setStyle(SkPaint::kStroke_Style); TestCase geoPEStrokeCase(geo, peStroke, reporter); // Check whether constructing the filled case would cause the base shape to have a different // geometry (because of a geometric transformation upon initial GrShape construction). if (geo.fillChangesGeom()) { fillGeoCase.compare(reporter, geoPECase, TestCase::kAllDifferent_ComparisonExpecation); fillGeoCase.compare(reporter, geoPEStrokeCase, TestCase::kAllDifferent_ComparisonExpecation); } else { fillGeoCase.compare(reporter, geoPECase, TestCase::kSameUpToPE_ComparisonExpecation); fillGeoCase.compare(reporter, geoPEStrokeCase, TestCase::kSameUpToPE_ComparisonExpecation); } geoPECase.compare(reporter, geoPEStrokeCase, TestCase::kSameUpToStroke_ComparisonExpecation); TestCase rrectFillCase(reporter, RRectPathEffect::RRect(), fill); SkPaint stroke = peStroke; stroke.setPathEffect(nullptr); TestCase rrectStrokeCase(reporter, RRectPathEffect::RRect(), stroke); SkRRect rrect; // Applying the path effect should make a SkRRect shape. There is no further stroking in the // geoPECase, so the full style should be the same as just the PE. REPORTER_ASSERT(reporter, geoPECase.appliedPathEffectShape().asRRect(&rrect, nullptr, nullptr, nullptr)); REPORTER_ASSERT(reporter, rrect == RRectPathEffect::RRect()); REPORTER_ASSERT(reporter, geoPECase.appliedPathEffectKey() == rrectFillCase.baseKey()); REPORTER_ASSERT(reporter, geoPECase.appliedFullStyleShape().asRRect(&rrect, nullptr, nullptr, nullptr)); REPORTER_ASSERT(reporter, rrect == RRectPathEffect::RRect()); REPORTER_ASSERT(reporter, geoPECase.appliedFullStyleKey() == rrectFillCase.baseKey()); // In the PE+stroke case applying the full style should be the same as just stroking the rrect. REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedPathEffectShape().asRRect(&rrect, nullptr, nullptr, nullptr)); REPORTER_ASSERT(reporter, rrect == RRectPathEffect::RRect()); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedPathEffectKey() == rrectFillCase.baseKey()); REPORTER_ASSERT(reporter, !geoPEStrokeCase.appliedFullStyleShape().asRRect(&rrect, nullptr, nullptr, nullptr)); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedFullStyleKey() == rrectStrokeCase.appliedFullStyleKey()); } void test_unknown_path_effect(skiatest::Reporter* reporter, const Geo& geo) { /** * This path effect just adds two lineTos to the input path. */ class AddLineTosPathEffect : SkPathEffect { public: static sk_sp Make() { return sk_sp(new AddLineTosPathEffect); } Factory getFactory() const override { return nullptr; } const char* getTypeName() const override { return nullptr; } protected: bool onFilterPath(SkPath* dst, const SkPath& src, SkStrokeRec*, const SkRect* cullR) const override { *dst = src; // To avoid triggering data-based keying of paths with few verbs we add many segments. for (int i = 0; i < 100; ++i) { dst->lineTo(SkIntToScalar(i), SkIntToScalar(i)); } return true; } SkRect onComputeFastBounds(const SkRect& src) const override { SkRect dst = src; SkRectPriv::GrowToInclude(&dst, {0, 0}); SkRectPriv::GrowToInclude(&dst, {100, 100}); return dst; } private: AddLineTosPathEffect() {} }; // This path effect should make the keys invalid when it is applied. We only produce a path // effect key for dash path effects. So the only way another arbitrary path effect can produce // a styled result with a key is to produce a non-path shape that has a purely geometric key. SkPaint peStroke; peStroke.setPathEffect(AddLineTosPathEffect::Make()); peStroke.setStrokeWidth(2.f); peStroke.setStyle(SkPaint::kStroke_Style); TestCase geoPEStrokeCase(geo, peStroke, reporter); TestCase::SelfExpectations expectations; expectations.fPEHasEffect = true; expectations.fPEHasValidKey = false; expectations.fStrokeApplies = true; geoPEStrokeCase.testExpectations(reporter, expectations); } void test_make_hairline_path_effect(skiatest::Reporter* reporter, const Geo& geo) { /** * This path effect just changes the stroke rec to hairline. */ class MakeHairlinePathEffect : SkPathEffect { public: static sk_sp Make() { return sk_sp(new MakeHairlinePathEffect); } Factory getFactory() const override { return nullptr; } const char* getTypeName() const override { return nullptr; } protected: bool onFilterPath(SkPath* dst, const SkPath& src, SkStrokeRec* strokeRec, const SkRect* cullR) const override { *dst = src; strokeRec->setHairlineStyle(); return true; } private: MakeHairlinePathEffect() {} }; SkPaint fill; SkPaint pe; pe.setPathEffect(MakeHairlinePathEffect::Make()); TestCase peCase(geo, pe, reporter); SkPath a, b, c; peCase.baseShape().asPath(&a); peCase.appliedPathEffectShape().asPath(&b); peCase.appliedFullStyleShape().asPath(&c); if (geo.isNonPath(pe)) { // RRect types can have a change in start index or direction after the PE is applied. This // is because once the PE is applied, GrShape may canonicalize the dir and index since it // is not germane to the styling any longer. // Instead we just check that the paths would fill the same both before and after styling. REPORTER_ASSERT(reporter, paths_fill_same(a, b)); REPORTER_ASSERT(reporter, paths_fill_same(a, c)); } else { // The base shape cannot perform canonicalization on the path's fill type because of an // unknown path effect. However, after the path effect is applied the resulting hairline // shape will canonicalize the path fill type since hairlines (and stroking in general) // don't distinguish between even/odd and non-zero winding. a.setFillType(b.getFillType()); REPORTER_ASSERT(reporter, a == b); REPORTER_ASSERT(reporter, a == c); // If the resulting path is small enough then it will have a key. REPORTER_ASSERT(reporter, paths_fill_same(a, b)); REPORTER_ASSERT(reporter, paths_fill_same(a, c)); REPORTER_ASSERT(reporter, peCase.appliedPathEffectKey().empty()); REPORTER_ASSERT(reporter, peCase.appliedFullStyleKey().empty()); } REPORTER_ASSERT(reporter, peCase.appliedPathEffectShape().style().isSimpleHairline()); REPORTER_ASSERT(reporter, peCase.appliedFullStyleShape().style().isSimpleHairline()); } void test_volatile_path(skiatest::Reporter* reporter, const Geo& geo) { SkPath vPath = geo.path(); vPath.setIsVolatile(true); SkPaint dashAndStroke; dashAndStroke.setPathEffect(make_dash()); dashAndStroke.setStrokeWidth(2.f); dashAndStroke.setStyle(SkPaint::kStroke_Style); TestCase volatileCase(reporter, vPath, dashAndStroke); // We expect a shape made from a volatile path to have a key iff the shape is recognized // as a specialized geometry. if (geo.isNonPath(dashAndStroke)) { REPORTER_ASSERT(reporter, SkToBool(volatileCase.baseKey().count())); // In this case all the keys should be identical to the non-volatile case. TestCase nonVolatileCase(reporter, geo.path(), dashAndStroke); volatileCase.compare(reporter, nonVolatileCase, TestCase::kAllSame_ComparisonExpecation); } else { // None of the keys should be valid. REPORTER_ASSERT(reporter, !SkToBool(volatileCase.baseKey().count())); REPORTER_ASSERT(reporter, !SkToBool(volatileCase.appliedPathEffectKey().count())); REPORTER_ASSERT(reporter, !SkToBool(volatileCase.appliedFullStyleKey().count())); REPORTER_ASSERT(reporter, !SkToBool(volatileCase.appliedPathEffectThenStrokeKey().count())); } } void test_path_effect_makes_empty_shape(skiatest::Reporter* reporter, const Geo& geo) { /** * This path effect returns an empty path (possibly inverted) */ class EmptyPathEffect : SkPathEffect { public: static sk_sp Make(bool invert) { return sk_sp(new EmptyPathEffect(invert)); } Factory getFactory() const override { return nullptr; } const char* getTypeName() const override { return nullptr; } protected: bool onFilterPath(SkPath* dst, const SkPath& src, SkStrokeRec*, const SkRect* cullR) const override { dst->reset(); if (fInvert) { dst->toggleInverseFillType(); } return true; } SkRect onComputeFastBounds(const SkRect& src) const override { return { 0, 0, 0, 0 }; } private: bool fInvert; EmptyPathEffect(bool invert) : fInvert(invert) {} }; SkPath emptyPath; GrShape emptyShape(emptyPath); Key emptyKey; make_key(&emptyKey, emptyShape); REPORTER_ASSERT(reporter, emptyShape.isEmpty()); emptyPath.toggleInverseFillType(); GrShape invertedEmptyShape(emptyPath); Key invertedEmptyKey; make_key(&invertedEmptyKey, invertedEmptyShape); REPORTER_ASSERT(reporter, invertedEmptyShape.isEmpty()); REPORTER_ASSERT(reporter, invertedEmptyKey != emptyKey); SkPaint pe; pe.setPathEffect(EmptyPathEffect::Make(false)); TestCase geoPECase(geo, pe, reporter); REPORTER_ASSERT(reporter, geoPECase.appliedFullStyleKey() == emptyKey); REPORTER_ASSERT(reporter, geoPECase.appliedPathEffectKey() == emptyKey); REPORTER_ASSERT(reporter, geoPECase.appliedPathEffectThenStrokeKey() == emptyKey); REPORTER_ASSERT(reporter, geoPECase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, geoPECase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, !geoPECase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, !geoPECase.appliedFullStyleShape().inverseFilled()); SkPaint peStroke; peStroke.setPathEffect(EmptyPathEffect::Make(false)); peStroke.setStrokeWidth(2.f); peStroke.setStyle(SkPaint::kStroke_Style); TestCase geoPEStrokeCase(geo, peStroke, reporter); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedFullStyleKey() == emptyKey); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedPathEffectKey() == emptyKey); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedPathEffectThenStrokeKey() == emptyKey); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, geoPEStrokeCase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, !geoPEStrokeCase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, !geoPEStrokeCase.appliedFullStyleShape().inverseFilled()); pe.setPathEffect(EmptyPathEffect::Make(true)); TestCase geoPEInvertCase(geo, pe, reporter); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedFullStyleKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedPathEffectKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedPathEffectThenStrokeKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, geoPEInvertCase.appliedFullStyleShape().inverseFilled()); peStroke.setPathEffect(EmptyPathEffect::Make(true)); TestCase geoPEInvertStrokeCase(geo, peStroke, reporter); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedFullStyleKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedPathEffectKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedPathEffectThenStrokeKey() == invertedEmptyKey); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, geoPEInvertStrokeCase.appliedFullStyleShape().inverseFilled()); } void test_path_effect_fails(skiatest::Reporter* reporter, const Geo& geo) { /** * This path effect always fails to apply. */ class FailurePathEffect : SkPathEffect { public: static sk_sp Make() { return sk_sp(new FailurePathEffect); } Factory getFactory() const override { return nullptr; } const char* getTypeName() const override { return nullptr; } protected: bool onFilterPath(SkPath* dst, const SkPath& src, SkStrokeRec*, const SkRect* cullR) const override { return false; } private: FailurePathEffect() {} }; SkPaint fill; TestCase fillCase(geo, fill, reporter); SkPaint pe; pe.setPathEffect(FailurePathEffect::Make()); TestCase peCase(geo, pe, reporter); SkPaint stroke; stroke.setStrokeWidth(2.f); stroke.setStyle(SkPaint::kStroke_Style); TestCase strokeCase(geo, stroke, reporter); SkPaint peStroke = stroke; peStroke.setPathEffect(FailurePathEffect::Make()); TestCase peStrokeCase(geo, peStroke, reporter); // In general the path effect failure can cause some of the TestCase::compare() tests to fail // for at least two reasons: 1) We will initially treat the shape as unkeyable because of the // path effect, but then when the path effect fails we can key it. 2) GrShape will change its // mind about whether a unclosed rect is actually rect. The path effect initially bars us from // closing it but after the effect fails we can (for the fill+pe case). This causes different // routes through GrShape to have equivalent but different representations of the path (closed // or not) but that fill the same. SkPath a; SkPath b; fillCase.appliedPathEffectShape().asPath(&a); peCase.appliedPathEffectShape().asPath(&b); REPORTER_ASSERT(reporter, paths_fill_same(a, b)); fillCase.appliedFullStyleShape().asPath(&a); peCase.appliedFullStyleShape().asPath(&b); REPORTER_ASSERT(reporter, paths_fill_same(a, b)); strokeCase.appliedPathEffectShape().asPath(&a); peStrokeCase.appliedPathEffectShape().asPath(&b); REPORTER_ASSERT(reporter, paths_fill_same(a, b)); strokeCase.appliedFullStyleShape().asPath(&a); peStrokeCase.appliedFullStyleShape().asPath(&b); REPORTER_ASSERT(reporter, paths_fill_same(a, b)); } DEF_TEST(GrShape_empty_shape, reporter) { SkPath emptyPath; SkPath invertedEmptyPath; invertedEmptyPath.toggleInverseFillType(); SkPaint fill; TestCase fillEmptyCase(reporter, emptyPath, fill); REPORTER_ASSERT(reporter, fillEmptyCase.baseShape().isEmpty()); REPORTER_ASSERT(reporter, fillEmptyCase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, fillEmptyCase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, !fillEmptyCase.baseShape().inverseFilled()); REPORTER_ASSERT(reporter, !fillEmptyCase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, !fillEmptyCase.appliedFullStyleShape().inverseFilled()); TestCase fillInvertedEmptyCase(reporter, invertedEmptyPath, fill); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.baseShape().isEmpty()); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.appliedPathEffectShape().isEmpty()); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.appliedFullStyleShape().isEmpty()); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.baseShape().inverseFilled()); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.appliedPathEffectShape().inverseFilled()); REPORTER_ASSERT(reporter, fillInvertedEmptyCase.appliedFullStyleShape().inverseFilled()); Key emptyKey(fillEmptyCase.baseKey()); REPORTER_ASSERT(reporter, emptyKey.count()); Key inverseEmptyKey(fillInvertedEmptyCase.baseKey()); REPORTER_ASSERT(reporter, inverseEmptyKey.count()); TestCase::SelfExpectations expectations; expectations.fStrokeApplies = false; expectations.fPEHasEffect = false; // This will test whether applying style preserves emptiness fillEmptyCase.testExpectations(reporter, expectations); fillInvertedEmptyCase.testExpectations(reporter, expectations); // Stroking an empty path should have no effect SkPaint stroke; stroke.setStrokeWidth(2.f); stroke.setStyle(SkPaint::kStroke_Style); stroke.setStrokeJoin(SkPaint::kRound_Join); stroke.setStrokeCap(SkPaint::kRound_Cap); TestCase strokeEmptyCase(reporter, emptyPath, stroke); strokeEmptyCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase strokeInvertedEmptyCase(reporter, invertedEmptyPath, stroke); strokeInvertedEmptyCase.compare(reporter, fillInvertedEmptyCase, TestCase::kAllSame_ComparisonExpecation); // Dashing and stroking an empty path should have no effect SkPaint dashAndStroke; dashAndStroke.setPathEffect(make_dash()); dashAndStroke.setStrokeWidth(2.f); dashAndStroke.setStyle(SkPaint::kStroke_Style); TestCase dashAndStrokeEmptyCase(reporter, emptyPath, dashAndStroke); dashAndStrokeEmptyCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase dashAndStrokeInvertexEmptyCase(reporter, invertedEmptyPath, dashAndStroke); // Dashing ignores inverseness so this is equivalent to the non-inverted empty fill. dashAndStrokeInvertexEmptyCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); // A shape made from an empty rrect should behave the same as an empty path when filled but not // when stroked. However, dashing an empty rrect produces an empty path leaving nothing to // stroke - so equivalent to filling an empty path. SkRRect emptyRRect = SkRRect::MakeEmpty(); REPORTER_ASSERT(reporter, emptyRRect.getType() == SkRRect::kEmpty_Type); TestCase fillEmptyRRectCase(reporter, emptyRRect, fill); fillEmptyRRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase strokeEmptyRRectCase(reporter, emptyRRect, stroke); strokeEmptyRRectCase.compare(reporter, strokeEmptyCase, TestCase::kAllDifferent_ComparisonExpecation); TestCase dashAndStrokeEmptyRRectCase(reporter, emptyRRect, dashAndStroke); dashAndStrokeEmptyRRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); static constexpr SkPath::Direction kDir = SkPath::kCCW_Direction; static constexpr int kStart = 0; TestCase fillInvertedEmptyRRectCase(reporter, emptyRRect, kDir, kStart, true, GrStyle(fill)); fillInvertedEmptyRRectCase.compare(reporter, fillInvertedEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase strokeInvertedEmptyRRectCase(reporter, emptyRRect, kDir, kStart, true, GrStyle(stroke)); strokeInvertedEmptyRRectCase.compare(reporter, strokeInvertedEmptyCase, TestCase::kAllDifferent_ComparisonExpecation); TestCase dashAndStrokeEmptyInvertedRRectCase(reporter, emptyRRect, kDir, kStart, true, GrStyle(dashAndStroke)); dashAndStrokeEmptyInvertedRRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); // Same for a rect. SkRect emptyRect = SkRect::MakeEmpty(); TestCase fillEmptyRectCase(reporter, emptyRect, fill); fillEmptyRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase dashAndStrokeEmptyRectCase(reporter, emptyRect, dashAndStroke); dashAndStrokeEmptyRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); TestCase dashAndStrokeEmptyInvertedRectCase(reporter, SkRRect::MakeRect(emptyRect), kDir, kStart, true, GrStyle(dashAndStroke)); // Dashing ignores inverseness so this is equivalent to the non-inverted empty fill. dashAndStrokeEmptyInvertedRectCase.compare(reporter, fillEmptyCase, TestCase::kAllSame_ComparisonExpecation); } // rect and oval types have rrect start indices that collapse to the same point. Here we select the // canonical point in these cases. unsigned canonicalize_rrect_start(int s, const SkRRect& rrect) { switch (rrect.getType()) { case SkRRect::kRect_Type: return (s + 1) & 0b110; case SkRRect::kOval_Type: return s & 0b110; default: return s; } } void test_rrect(skiatest::Reporter* r, const SkRRect& rrect) { enum Style { kFill, kStroke, kHairline, kStrokeAndFill }; // SkStrokeRec has no default cons., so init with kFill before calling the setters below. SkStrokeRec strokeRecs[4] { SkStrokeRec::kFill_InitStyle, SkStrokeRec::kFill_InitStyle, SkStrokeRec::kFill_InitStyle, SkStrokeRec::kFill_InitStyle}; strokeRecs[kFill].setFillStyle(); strokeRecs[kStroke].setStrokeStyle(2.f); strokeRecs[kHairline].setHairlineStyle(); strokeRecs[kStrokeAndFill].setStrokeStyle(3.f, true); // Use a bevel join to avoid complications of stroke+filled rects becoming filled rects before // applyStyle() is called. strokeRecs[kStrokeAndFill].setStrokeParams(SkPaint::kButt_Cap, SkPaint::kBevel_Join, 1.f); sk_sp dashEffect = make_dash(); static constexpr Style kStyleCnt = static_cast