diff options
author | Mike Reed <reed@google.com> | 2016-12-09 13:14:25 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-09 13:14:41 +0000 |
commit | a129dfef2aaab0b5995cdf1ab7b2cdd41c29cf72 (patch) | |
tree | 9398e074cf0408ba24f739130c062b2bfbe4435e /tools | |
parent | 76073c11040204d51dceb49cbd0b05be8e7fd0a2 (diff) |
Revert "remove SK_SUPPORT_LEGACY_CLIP_REGIONOPS"
This reverts commit 8e7432b7f98dd592e529a0c8bb038d73ebfb0478.
Reason for revert: <INSERT REASONING HERE>
external/skia/bench/../tools/android/SkAndroidSDKCanvas.h:103:36: error: C++ requires a type specifier for all declarations
void onClipRect(const SkRect&, ClipOp, ClipEdgeStyle) override;
Original change's description:
> remove SK_SUPPORT_LEGACY_CLIP_REGIONOPS
>
>
> switch over to SkClipOps now that SK_SUPPORT_LEGACY_CLIP_REGIONOPS is gone
>
> BUG=skia:
>
> Change-Id: Ifdc8b3746d508348a40cc009a4e529a1cb3c405d
> Reviewed-on: https://skia-review.googlesource.com/5714
> Commit-Queue: Mike Reed <reed@google.com>
> Reviewed-by: Mike Reed <reed@google.com>
>
TBR=reed@google.com,reviews@skia.org
BUG=skia:
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: If26ea91d7464615e43c1d3d2f726e337ff56b55c
Reviewed-on: https://skia-review.googlesource.com/5721
Commit-Queue: Mike Reed <reed@google.com>
Reviewed-by: Mike Reed <reed@google.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/debugger/SkDebugCanvas.cpp | 30 | ||||
-rw-r--r-- | tools/debugger/SkDebugCanvas.h | 10 | ||||
-rw-r--r-- | tools/debugger/SkDrawCommand.cpp | 38 | ||||
-rw-r--r-- | tools/debugger/SkDrawCommand.h | 34 | ||||
-rw-r--r-- | tools/debugger/SkObjectParser.cpp | 14 | ||||
-rw-r--r-- | tools/debugger/SkObjectParser.h | 2 |
6 files changed, 64 insertions, 64 deletions
diff --git a/tools/debugger/SkDebugCanvas.cpp b/tools/debugger/SkDebugCanvas.cpp index b02ab59ee8..eaf72c3c35 100644 --- a/tools/debugger/SkDebugCanvas.cpp +++ b/tools/debugger/SkDebugCanvas.cpp @@ -105,7 +105,7 @@ SkDebugCanvas::SkDebugCanvas(int width, int height) SkASSERT(!large.roundOut().isEmpty()); #endif // call the base class' version to avoid adding a draw command - this->INHERITED::onClipRect(large, kReplace_SkClipOp, kHard_ClipEdgeStyle); + this->INHERITED::onClipRect(large, kReplace_Op, kHard_ClipEdgeStyle); } SkDebugCanvas::~SkDebugCanvas() { @@ -153,21 +153,21 @@ class SkDebugClipVisitor : public SkCanvas::ClipVisitor { public: SkDebugClipVisitor(SkCanvas* canvas) : fCanvas(canvas) {} - void clipRect(const SkRect& r, SkClipOp, bool doAA) override { + void clipRect(const SkRect& r, SkCanvas::ClipOp, bool doAA) override { SkPaint p; p.setColor(SK_ColorRED); p.setStyle(SkPaint::kStroke_Style); p.setAntiAlias(doAA); fCanvas->drawRect(r, p); } - void clipRRect(const SkRRect& rr, SkClipOp, bool doAA) override { + void clipRRect(const SkRRect& rr, SkCanvas::ClipOp, bool doAA) override { SkPaint p; p.setColor(SK_ColorGREEN); p.setStyle(SkPaint::kStroke_Style); p.setAntiAlias(doAA); fCanvas->drawRRect(rr, p); } - void clipPath(const SkPath& path, SkClipOp, bool doAA) override { + void clipPath(const SkPath& path, SkCanvas::ClipOp, bool doAA) override { SkPaint p; p.setColor(SK_ColorBLUE); p.setStyle(SkPaint::kStroke_Style); @@ -220,7 +220,7 @@ void SkDebugCanvas::drawTo(SkCanvas* canvas, int index, int m) { canvas->clear(SK_ColorWHITE); canvas->resetMatrix(); if (!windowRect.isEmpty()) { - canvas->clipRect(windowRect, kReplace_SkClipOp); + canvas->clipRect(windowRect, SkCanvas::kReplace_Op); } this->applyUserTransform(canvas); @@ -280,7 +280,7 @@ void SkDebugCanvas::drawTo(SkCanvas* canvas, int index, int m) { canvas->save(); #define LARGE_COORD 1000000000 canvas->clipRect(SkRect::MakeLTRB(-LARGE_COORD, -LARGE_COORD, LARGE_COORD, LARGE_COORD), - kReverseDifference_SkClipOp); + SkCanvas::kReverseDifference_Op); SkPaint clipPaint; clipPaint.setColor(fClipVizColor); canvas->drawPaint(clipPaint); @@ -295,7 +295,7 @@ void SkDebugCanvas::drawTo(SkCanvas* canvas, int index, int m) { if (!windowRect.isEmpty()) { SkRect r = windowRect; r.outset(SK_Scalar1, SK_Scalar1); - canvas->clipRect(r, kReplace_SkClipOp); + canvas->clipRect(r, SkCanvas::kReplace_Op); } // visualize existing clips SkDebugClipVisitor visitor(canvas); @@ -316,9 +316,9 @@ void SkDebugCanvas::drawTo(SkCanvas* canvas, int index, int m) { if (type != SkClipStack::Element::kEmpty_Type) { element->asPath(&operand); } - SkClipOp elementOp = element->getOp(); + SkCanvas::ClipOp elementOp = element->getOp(); this->addClipStackData(devPath, operand, elementOp); - if (elementOp == kReplace_SkClipOp) { + if (elementOp == SkCanvas::kReplace_Op) { devPath = operand; } else { Op(devPath, operand, (SkPathOp) elementOp, &devPath); @@ -515,19 +515,19 @@ void SkDebugCanvas::overrideTexFiltering(bool overrideTexFiltering, SkFilterQual fFilterQuality = quality; } -void SkDebugCanvas::onClipPath(const SkPath& path, SkClipOp op, ClipEdgeStyle edgeStyle) { +void SkDebugCanvas::onClipPath(const SkPath& path, ClipOp op, ClipEdgeStyle edgeStyle) { this->addDrawCommand(new SkClipPathCommand(path, op, kSoft_ClipEdgeStyle == edgeStyle)); } -void SkDebugCanvas::onClipRect(const SkRect& rect, SkClipOp op, ClipEdgeStyle edgeStyle) { +void SkDebugCanvas::onClipRect(const SkRect& rect, ClipOp op, ClipEdgeStyle edgeStyle) { this->addDrawCommand(new SkClipRectCommand(rect, op, kSoft_ClipEdgeStyle == edgeStyle)); } -void SkDebugCanvas::onClipRRect(const SkRRect& rrect, SkClipOp op, ClipEdgeStyle edgeStyle) { +void SkDebugCanvas::onClipRRect(const SkRRect& rrect, ClipOp op, ClipEdgeStyle edgeStyle) { this->addDrawCommand(new SkClipRRectCommand(rrect, op, kSoft_ClipEdgeStyle == edgeStyle)); } -void SkDebugCanvas::onClipRegion(const SkRegion& region, SkClipOp op) { +void SkDebugCanvas::onClipRegion(const SkRegion& region, ClipOp op) { this->addDrawCommand(new SkClipRegionCommand(region, op)); } @@ -800,8 +800,8 @@ void SkDebugCanvas::addPathData(const SkPath& path, const char* pathName) { } void SkDebugCanvas::addClipStackData(const SkPath& devPath, const SkPath& operand, - SkClipOp elementOp) { - if (elementOp == kReplace_SkClipOp) { + SkCanvas::ClipOp elementOp) { + if (elementOp == SkCanvas::kReplace_Op) { if (!lastClipStackData(devPath)) { fSaveDevPath = operand; } diff --git a/tools/debugger/SkDebugCanvas.h b/tools/debugger/SkDebugCanvas.h index b9ef693a0b..6e7086e6ce 100644 --- a/tools/debugger/SkDebugCanvas.h +++ b/tools/debugger/SkDebugCanvas.h @@ -248,10 +248,10 @@ protected: const SkPaint*, SrcRectConstraint) override; void onDrawBitmapNine(const SkBitmap&, const SkIRect& center, const SkRect& dst, const SkPaint*) override; - void onClipRect(const SkRect&, SkClipOp, ClipEdgeStyle) override; - void onClipRRect(const SkRRect&, SkClipOp, ClipEdgeStyle) override; - void onClipPath(const SkPath&, SkClipOp, ClipEdgeStyle) override; - void onClipRegion(const SkRegion& region, SkClipOp) override; + void onClipRect(const SkRect&, ClipOp, ClipEdgeStyle) override; + void onClipRRect(const SkRRect&, ClipOp, ClipEdgeStyle) override; + void onClipPath(const SkPath&, ClipOp, ClipEdgeStyle) override; + void onClipRegion(const SkRegion& region, ClipOp) override; void onDrawPicture(const SkPicture*, const SkMatrix*, const SkPaint*) override; @@ -308,7 +308,7 @@ private: void resetClipStackData() { fClipStackData.reset(); fCalledAddStackData = false; } - void addClipStackData(const SkPath& devPath, const SkPath& operand, SkClipOp elementOp); + void addClipStackData(const SkPath& devPath, const SkPath& operand, ClipOp elementOp); void addPathData(const SkPath& path, const char* pathName); bool lastClipStackData(const SkPath& devPath); void outputConicPoints(const SkPoint* pts, SkScalar weight); diff --git a/tools/debugger/SkDrawCommand.cpp b/tools/debugger/SkDrawCommand.cpp index 6cec6eb6b1..c8eaf98851 100644 --- a/tools/debugger/SkDrawCommand.cpp +++ b/tools/debugger/SkDrawCommand.cpp @@ -584,19 +584,19 @@ Json::Value SkDrawCommand::MakeJsonRegion(const SkRegion& region) { return Json::Value("<unimplemented>"); } -static Json::Value make_json_regionop(SkClipOp op) { +static Json::Value make_json_regionop(SkCanvas::ClipOp op) { switch (op) { - case kDifference_SkClipOp: + case SkCanvas::kDifference_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_DIFFERENCE); - case kIntersect_SkClipOp: + case SkCanvas::kIntersect_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_INTERSECT); - case kUnion_SkClipOp: + case SkCanvas::kUnion_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_UNION); - case kXOR_SkClipOp: + case SkCanvas::kXOR_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_XOR); - case kReverseDifference_SkClipOp: + case SkCanvas::kReverseDifference_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_REVERSE_DIFFERENCE); - case kReplace_SkClipOp: + case SkCanvas::kReplace_Op: return Json::Value(SKDEBUGCANVAS_REGIONOP_REPLACE); default: SkASSERT(false); @@ -1592,28 +1592,28 @@ static void extract_json_path(Json::Value& path, SkPath* result) { } } -SkClipOp get_json_clipop(Json::Value& jsonOp) { +SkCanvas::ClipOp get_json_clipop(Json::Value& jsonOp) { const char* op = jsonOp.asCString(); if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_DIFFERENCE)) { - return kDifference_SkClipOp; + return SkCanvas::kDifference_Op; } else if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_INTERSECT)) { - return kIntersect_SkClipOp; + return SkCanvas::kIntersect_Op; } else if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_UNION)) { - return kUnion_SkClipOp; + return SkCanvas::kUnion_Op; } else if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_XOR)) { - return kXOR_SkClipOp; + return SkCanvas::kXOR_Op; } else if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_REVERSE_DIFFERENCE)) { - return kReverseDifference_SkClipOp; + return SkCanvas::kReverseDifference_Op; } else if (!strcmp(op, SKDEBUGCANVAS_REGIONOP_REPLACE)) { - return kReplace_SkClipOp; + return SkCanvas::kReplace_Op; } SkASSERT(false); - return kIntersect_SkClipOp; + return SkCanvas::kIntersect_Op; } SkClearCommand::SkClearCommand(SkColor color) : INHERITED(kDrawClear_OpType) { @@ -1636,7 +1636,7 @@ Json::Value SkClearCommand::toJSON(UrlDataManager& urlDataManager) const { return new SkClearCommand(get_json_color(color)); } -SkClipPathCommand::SkClipPathCommand(const SkPath& path, SkClipOp op, bool doAA) +SkClipPathCommand::SkClipPathCommand(const SkPath& path, SkCanvas::ClipOp op, bool doAA) : INHERITED(kClipPath_OpType) { fPath = path; fOp = op; @@ -1672,7 +1672,7 @@ SkClipPathCommand* SkClipPathCommand::fromJSON(Json::Value& command, command[SKDEBUGCANVAS_ATTRIBUTE_ANTIALIAS].asBool()); } -SkClipRegionCommand::SkClipRegionCommand(const SkRegion& region, SkClipOp op) +SkClipRegionCommand::SkClipRegionCommand(const SkRegion& region, SkCanvas::ClipOp op) : INHERITED(kClipRegion_OpType) { fRegion = region; fOp = op; @@ -1698,7 +1698,7 @@ SkClipRegionCommand* SkClipRegionCommand::fromJSON(Json::Value& command, return nullptr; } -SkClipRectCommand::SkClipRectCommand(const SkRect& rect, SkClipOp op, bool doAA) +SkClipRectCommand::SkClipRectCommand(const SkRect& rect, SkCanvas::ClipOp op, bool doAA) : INHERITED(kClipRect_OpType) { fRect = rect; fOp = op; @@ -1733,7 +1733,7 @@ SkClipRectCommand* SkClipRectCommand::fromJSON(Json::Value& command, command[SKDEBUGCANVAS_ATTRIBUTE_ANTIALIAS].asBool()); } -SkClipRRectCommand::SkClipRRectCommand(const SkRRect& rrect, SkClipOp op, bool doAA) +SkClipRRectCommand::SkClipRRectCommand(const SkRRect& rrect, SkCanvas::ClipOp op, bool doAA) : INHERITED(kClipRRect_OpType) { fRRect = rrect; fOp = op; diff --git a/tools/debugger/SkDrawCommand.h b/tools/debugger/SkDrawCommand.h index 880c2aa086..144c6c560e 100644 --- a/tools/debugger/SkDrawCommand.h +++ b/tools/debugger/SkDrawCommand.h @@ -173,69 +173,69 @@ private: class SkClipPathCommand : public SkDrawCommand { public: - SkClipPathCommand(const SkPath& path, SkClipOp op, bool doAA); + SkClipPathCommand(const SkPath& path, SkCanvas::ClipOp op, bool doAA); void execute(SkCanvas* canvas) const override; bool render(SkCanvas* canvas) const override; Json::Value toJSON(UrlDataManager& urlDataManager) const override; static SkClipPathCommand* fromJSON(Json::Value& command, UrlDataManager& urlDataManager); private: - SkPath fPath; - SkClipOp fOp; - bool fDoAA; + SkPath fPath; + SkCanvas::ClipOp fOp; + bool fDoAA; typedef SkDrawCommand INHERITED; }; class SkClipRegionCommand : public SkDrawCommand { public: - SkClipRegionCommand(const SkRegion& region, SkClipOp op); + SkClipRegionCommand(const SkRegion& region, SkCanvas::ClipOp op); void execute(SkCanvas* canvas) const override; Json::Value toJSON(UrlDataManager& urlDataManager) const override; static SkClipRegionCommand* fromJSON(Json::Value& command, UrlDataManager& urlDataManager); private: - SkRegion fRegion; - SkClipOp fOp; + SkRegion fRegion; + SkCanvas::ClipOp fOp; typedef SkDrawCommand INHERITED; }; class SkClipRectCommand : public SkDrawCommand { public: - SkClipRectCommand(const SkRect& rect, SkClipOp op, bool doAA); + SkClipRectCommand(const SkRect& rect, SkCanvas::ClipOp op, bool doAA); void execute(SkCanvas* canvas) const override; Json::Value toJSON(UrlDataManager& urlDataManager) const override; static SkClipRectCommand* fromJSON(Json::Value& command, UrlDataManager& urlDataManager); const SkRect& rect() const { return fRect; } - SkClipOp op() const { return fOp; } + SkCanvas::ClipOp op() const { return fOp; } bool doAA() const { return fDoAA; } private: - SkRect fRect; - SkClipOp fOp; - bool fDoAA; + SkRect fRect; + SkCanvas::ClipOp fOp; + bool fDoAA; typedef SkDrawCommand INHERITED; }; class SkClipRRectCommand : public SkDrawCommand { public: - SkClipRRectCommand(const SkRRect& rrect, SkClipOp op, bool doAA); + SkClipRRectCommand(const SkRRect& rrect, SkCanvas::ClipOp op, bool doAA); void execute(SkCanvas* canvas) const override; bool render(SkCanvas* canvas) const override; Json::Value toJSON(UrlDataManager& urlDataManager) const override; static SkClipRRectCommand* fromJSON(Json::Value& command, UrlDataManager& urlDataManager); const SkRRect& rrect() const { return fRRect; } - SkClipOp op() const { return fOp; } + SkCanvas::ClipOp op() const { return fOp; } bool doAA() const { return fDoAA; } private: - SkRRect fRRect; - SkClipOp fOp; - bool fDoAA; + SkRRect fRRect; + SkCanvas::ClipOp fOp; + bool fDoAA; typedef SkDrawCommand INHERITED; }; diff --git a/tools/debugger/SkObjectParser.cpp b/tools/debugger/SkObjectParser.cpp index 8e3f2390c0..5f0e5f8aa9 100644 --- a/tools/debugger/SkObjectParser.cpp +++ b/tools/debugger/SkObjectParser.cpp @@ -313,19 +313,19 @@ SkString* SkObjectParser::RRectToString(const SkRRect& rrect, const char* title) return mRRect; } -SkString* SkObjectParser::ClipOpToString(SkClipOp op) { +SkString* SkObjectParser::ClipOpToString(SkCanvas::ClipOp op) { SkString* mOp = new SkString("SkRegion::Op: "); - if (op == kDifference_SkClipOp) { + if (op == SkCanvas::kDifference_Op) { mOp->append("kDifference_Op"); - } else if (op == kIntersect_SkClipOp) { + } else if (op == SkCanvas::kIntersect_Op) { mOp->append("kIntersect_Op"); - } else if (op == kUnion_SkClipOp) { + } else if (op == SkCanvas::kUnion_Op) { mOp->append("kUnion_Op"); - } else if (op == kXOR_SkClipOp) { + } else if (op == SkCanvas::kXOR_Op) { mOp->append("kXOR_Op"); - } else if (op == kReverseDifference_SkClipOp) { + } else if (op == SkCanvas::kReverseDifference_Op) { mOp->append("kReverseDifference_Op"); - } else if (op == kReplace_SkClipOp) { + } else if (op == SkCanvas::kReplace_Op) { mOp->append("kReplace_Op"); } else { mOp->append("Unknown Type"); diff --git a/tools/debugger/SkObjectParser.h b/tools/debugger/SkObjectParser.h index 48051792cb..11baf12ec0 100644 --- a/tools/debugger/SkObjectParser.h +++ b/tools/debugger/SkObjectParser.h @@ -101,7 +101,7 @@ public: Returns a string representation of the SkRegion enum. @param op SkRegion::op enum */ - static SkString* ClipOpToString(SkClipOp op); + static SkString* ClipOpToString(SkCanvas::ClipOp op); /** Returns a string representation of the SkRegion. |