diff options
author | Florin Malita <fmalita@chromium.org> | 2017-05-01 12:17:12 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-05-01 21:26:01 +0000 |
commit | a6b72cb5729306fdd676d739c1e2c53afa0786a7 (patch) | |
tree | ba4a30663e7cb1cb926b4f4fe895eaebefbcfb21 /src | |
parent | 5664e65eb1680a14eeaa6ca79ddf9e734518c822 (diff) |
SaveLayerRec::fClipMask -> raw pointer
Use raw pointers for optional clip mask plumbing, to match the backdrop
API.
Change-Id: I7eb0ee5896faf34cc05789ba0703f35a4ab6a4f2
Reviewed-on: https://skia-review.googlesource.com/14901
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkCanvas.cpp | 4 | ||||
-rw-r--r-- | src/core/SkColorSpaceXformCanvas.cpp | 4 | ||||
-rw-r--r-- | src/core/SkLiteDL.cpp | 12 | ||||
-rw-r--r-- | src/core/SkLiteDL.h | 2 | ||||
-rw-r--r-- | src/core/SkPicturePlayback.cpp | 2 | ||||
-rw-r--r-- | src/core/SkPictureRecord.cpp | 2 | ||||
-rw-r--r-- | src/core/SkRecordDraw.cpp | 2 | ||||
-rw-r--r-- | src/core/SkRecorder.cpp | 2 | ||||
-rw-r--r-- | src/pipe/SkPipeCanvas.cpp | 2 | ||||
-rw-r--r-- | src/pipe/SkPipeReader.cpp | 2 |
10 files changed, 17 insertions, 17 deletions
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp index c7e1d6e898..c09e6cbb7d 100644 --- a/src/core/SkCanvas.cpp +++ b/src/core/SkCanvas.cpp @@ -257,12 +257,12 @@ struct DeviceCM { SkMatrix fClipMatrix; DeviceCM(sk_sp<SkBaseDevice> device, const SkPaint* paint, const SkMatrix& stashed, - sk_sp<SkImage> clipImage, const SkMatrix* clipMatrix) + const SkImage* clipImage, const SkMatrix* clipMatrix) : fNext(nullptr) , fDevice(std::move(device)) , fPaint(paint ? skstd::make_unique<SkPaint>(*paint) : nullptr) , fStashedMatrix(stashed) - , fClipImage(std::move(clipImage)) + , fClipImage(sk_ref_sp(const_cast<SkImage*>(clipImage))) , fClipMatrix(clipMatrix ? *clipMatrix : SkMatrix::I()) {} diff --git a/src/core/SkColorSpaceXformCanvas.cpp b/src/core/SkColorSpaceXformCanvas.cpp index b4dd5bb1c4..2d62458b73 100644 --- a/src/core/SkColorSpaceXformCanvas.cpp +++ b/src/core/SkColorSpaceXformCanvas.cpp @@ -229,12 +229,12 @@ public: SaveLayerStrategy getSaveLayerStrategy(const SaveLayerRec& rec) override { sk_sp<SkImageFilter> backdrop = rec.fBackdrop ? fXformer->apply(rec.fBackdrop) : nullptr; - sk_sp<SkImage> clipMask = rec.fClipMask ? fXformer->apply(rec.fClipMask.get()) : nullptr; + sk_sp<SkImage> clipMask = rec.fClipMask ? fXformer->apply(rec.fClipMask) : nullptr; fTarget->saveLayer({ rec.fBounds, MaybePaint(rec.fPaint, fXformer.get()), backdrop.get(), - std::move(clipMask), + clipMask.get(), rec.fClipMatrix, rec.fSaveLayerFlags, }); diff --git a/src/core/SkLiteDL.cpp b/src/core/SkLiteDL.cpp index f84060751e..71cbf681eb 100644 --- a/src/core/SkLiteDL.cpp +++ b/src/core/SkLiteDL.cpp @@ -91,23 +91,23 @@ namespace { struct SaveLayer final : Op { static const auto kType = Type::SaveLayer; SaveLayer(const SkRect* bounds, const SkPaint* paint, - const SkImageFilter* backdrop, sk_sp<SkImage> clipMask, + const SkImageFilter* backdrop, const SkImage* clipMask, const SkMatrix* clipMatrix, SkCanvas::SaveLayerFlags flags) { if (bounds) { this->bounds = *bounds; } if (paint) { this->paint = *paint; } this->backdrop = sk_ref_sp(backdrop); - this->clipMask = std::move(clipMask); + this->clipMask = sk_ref_sp(clipMask); this->clipMatrix = clipMatrix ? *clipMatrix : SkMatrix::I(); this->flags = flags; } SkRect bounds = kUnset; SkPaint paint; sk_sp<const SkImageFilter> backdrop; - sk_sp<SkImage> clipMask; + sk_sp<const SkImage> clipMask; SkMatrix clipMatrix; SkCanvas::SaveLayerFlags flags; void draw(SkCanvas* c, const SkMatrix&) const { - c->saveLayer({ maybe_unset(bounds), &paint, backdrop.get(), clipMask, + c->saveLayer({ maybe_unset(bounds), &paint, backdrop.get(), clipMask.get(), clipMatrix.isIdentity() ? nullptr : &clipMatrix, flags }); } }; @@ -550,9 +550,9 @@ void SkLiteDL::setDrawFilter(SkDrawFilter* df) { void SkLiteDL:: save() { this->push <Save>(0); } void SkLiteDL::restore() { this->push<Restore>(0); } void SkLiteDL::saveLayer(const SkRect* bounds, const SkPaint* paint, - const SkImageFilter* backdrop, sk_sp<SkImage> clipMask, + const SkImageFilter* backdrop, const SkImage* clipMask, const SkMatrix* clipMatrix, SkCanvas::SaveLayerFlags flags) { - this->push<SaveLayer>(0, bounds, paint, backdrop, std::move(clipMask), clipMatrix, flags); + this->push<SaveLayer>(0, bounds, paint, backdrop, clipMask, clipMatrix, flags); } void SkLiteDL:: concat(const SkMatrix& matrix) { this->push <Concat>(0, matrix); } diff --git a/src/core/SkLiteDL.h b/src/core/SkLiteDL.h index 5219cd002c..f8e7910c82 100644 --- a/src/core/SkLiteDL.h +++ b/src/core/SkLiteDL.h @@ -29,7 +29,7 @@ public: #endif void save(); - void saveLayer(const SkRect*, const SkPaint*, const SkImageFilter*, sk_sp<SkImage>, + void saveLayer(const SkRect*, const SkPaint*, const SkImageFilter*, const SkImage*, const SkMatrix*, SkCanvas::SaveLayerFlags); void restore(); diff --git a/src/core/SkPicturePlayback.cpp b/src/core/SkPicturePlayback.cpp index 84228d0359..3af7909eff 100644 --- a/src/core/SkPicturePlayback.cpp +++ b/src/core/SkPicturePlayback.cpp @@ -744,7 +744,7 @@ void SkPicturePlayback::handleOp(SkReadBuffer* reader, rec.fSaveLayerFlags = reader->readInt(); } if (flatFlags & SAVELAYERREC_HAS_CLIPMASK) { - rec.fClipMask = sk_ref_sp(const_cast<SkImage*>(fPictureData->getImage(reader))); + rec.fClipMask = fPictureData->getImage(reader); } if (flatFlags & SAVELAYERREC_HAS_CLIPMATRIX) { reader->readMatrix(&clipMatrix); diff --git a/src/core/SkPictureRecord.cpp b/src/core/SkPictureRecord.cpp index ea4273f582..608c6b3997 100644 --- a/src/core/SkPictureRecord.cpp +++ b/src/core/SkPictureRecord.cpp @@ -125,7 +125,7 @@ void SkPictureRecord::recordSaveLayer(const SaveLayerRec& rec) { this->addInt(rec.fSaveLayerFlags); } if (flatFlags & SAVELAYERREC_HAS_CLIPMASK) { - this->addImage(rec.fClipMask.get()); + this->addImage(rec.fClipMask); } if (flatFlags & SAVELAYERREC_HAS_CLIPMATRIX) { this->addMatrix(*rec.fClipMatrix); diff --git a/src/core/SkRecordDraw.cpp b/src/core/SkRecordDraw.cpp index 02c8c69c8e..9e8d6fcea2 100644 --- a/src/core/SkRecordDraw.cpp +++ b/src/core/SkRecordDraw.cpp @@ -77,7 +77,7 @@ DRAW(Save, save()); DRAW(SaveLayer, saveLayer(SkCanvas::SaveLayerRec(r.bounds, r.paint, r.backdrop.get(), - r.clipMask, + r.clipMask.get(), r.clipMatrix, r.saveLayerFlags))); DRAW(SetMatrix, setMatrix(SkMatrix::Concat(fInitialCTM, r.matrix))); diff --git a/src/core/SkRecorder.cpp b/src/core/SkRecorder.cpp index c9d88fc503..3fdc17dfbf 100644 --- a/src/core/SkRecorder.cpp +++ b/src/core/SkRecorder.cpp @@ -366,7 +366,7 @@ SkCanvas::SaveLayerStrategy SkRecorder::getSaveLayerStrategy(const SaveLayerRec& APPEND(SaveLayer, this->copy(rec.fBounds) , this->copy(rec.fPaint) , sk_ref_sp(rec.fBackdrop) - , rec.fClipMask + , sk_ref_sp(rec.fClipMask) , this->copy(rec.fClipMatrix) , rec.fSaveLayerFlags); return SkCanvas::kNoLayer_SaveLayerStrategy; diff --git a/src/pipe/SkPipeCanvas.cpp b/src/pipe/SkPipeCanvas.cpp index 91e74eeca5..7b665d9968 100644 --- a/src/pipe/SkPipeCanvas.cpp +++ b/src/pipe/SkPipeCanvas.cpp @@ -258,7 +258,7 @@ SkCanvas::SaveLayerStrategy SkPipeCanvas::getSaveLayerStrategy(const SaveLayerRe writer.writeFlattenable(rec.fBackdrop); } if (rec.fClipMask) { - writer.writeImage(rec.fClipMask.get()); + writer.writeImage(rec.fClipMask); } if (rec.fClipMatrix) { writer.writeMatrix(*rec.fClipMatrix); diff --git a/src/pipe/SkPipeReader.cpp b/src/pipe/SkPipeReader.cpp index 73ef970c1b..07b360c28c 100644 --- a/src/pipe/SkPipeReader.cpp +++ b/src/pipe/SkPipeReader.cpp @@ -259,7 +259,7 @@ static void saveLayer_handler(SkPipeReader& reader, uint32_t packedVerb, SkCanva flags |= (1 << 31);//SkCanvas::kDontClipToLayer_PrivateSaveLayerFlag; } - canvas->saveLayer(SkCanvas::SaveLayerRec(bounds, paint, backdrop.get(), std::move(clipMask), + canvas->saveLayer(SkCanvas::SaveLayerRec(bounds, paint, backdrop.get(), clipMask.get(), (extra & kHasClipMatrix_SaveLayerMask) ? &clipMatrix : nullptr, flags)); } |