diff options
author | reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-31 16:31:11 +0000 |
---|---|---|
committer | reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-31 16:31:11 +0000 |
commit | 6fcbfcead5dc1b61fa5b4c139a1a3714e8c58091 (patch) | |
tree | c7bfb4b07deb28223c78af4de69e2440f34c3427 | |
parent | 081560e3abe25c4821b79ca1465f4dbd371c4b5c (diff) |
Revert "add SK_ATTR_DEPRECATED -- will need to disable for chrome, since it triggers a warning"
This reverts commit 1d22c4aaf9d8f053f25194a1ed74b137bfb19497.
git-svn-id: http://skia.googlecode.com/svn/trunk@12056 2bbb7eff-a529-9590-31e7-b0007b416f81
94 files changed, 303 insertions, 364 deletions
diff --git a/bench/BitmapRectBench.cpp b/bench/BitmapRectBench.cpp index 6029aebda2..104cb24bd9 100644 --- a/bench/BitmapRectBench.cpp +++ b/bench/BitmapRectBench.cpp @@ -39,20 +39,18 @@ static void draw_into_bitmap(const SkBitmap& bm) { */ class BitmapRectBench : public SkBenchmark { - SkBitmap fBitmap; - bool fSlightMatrix; - uint8_t fAlpha; - SkPaint::FilterLevel fFilterLevel; - SkString fName; - SkRect fSrcR, fDstR; - + SkBitmap fBitmap; + bool fDoFilter; + bool fSlightMatrix; + uint8_t fAlpha; + SkString fName; + SkRect fSrcR, fDstR; static const int kWidth = 128; static const int kHeight = 128; public: - BitmapRectBench(U8CPU alpha, SkPaint::FilterLevel filterLevel, - bool slightMatrix) { + BitmapRectBench(U8CPU alpha, bool doFilter, bool slightMatrix) { fAlpha = SkToU8(alpha); - fFilterLevel = filterLevel; + fDoFilter = doFilter; fSlightMatrix = slightMatrix; fBitmap.setConfig(SkBitmap::kARGB_8888_Config, kWidth, kHeight); @@ -61,8 +59,7 @@ public: protected: virtual const char* onGetName() SK_OVERRIDE { fName.printf("bitmaprect_%02X_%sfilter_%s", - fAlpha, - SkPaint::kNone_FilterLevel == fFilterLevel ? "no" : "", + fAlpha, fDoFilter ? "" : "no", fSlightMatrix ? "trans" : "identity"); return fName.c_str(); } @@ -92,7 +89,7 @@ protected: SkPaint paint; this->setupPaint(&paint); - paint.setFilterLevel(fFilterLevel); + paint.setFilterBitmap(fDoFilter); paint.setAlpha(fAlpha); for (int i = 0; i < this->getLoops(); i++) { @@ -104,10 +101,10 @@ private: typedef SkBenchmark INHERITED; }; -DEF_BENCH(return new BitmapRectBench(0xFF, SkPaint::kNone_FilterLevel, false)) -DEF_BENCH(return new BitmapRectBench(0x80, SkPaint::kNone_FilterLevel, false)) -DEF_BENCH(return new BitmapRectBench(0xFF, SkPaint::kLow_FilterLevel, false)) -DEF_BENCH(return new BitmapRectBench(0x80, SkPaint::kLow_FilterLevel, false)) +DEF_BENCH(return new BitmapRectBench(0xFF, false, false)) +DEF_BENCH(return new BitmapRectBench(0x80, false, false)) +DEF_BENCH(return new BitmapRectBench(0xFF, true, false)) +DEF_BENCH(return new BitmapRectBench(0x80, true, false)) -DEF_BENCH(return new BitmapRectBench(0xFF, SkPaint::kNone_FilterLevel, true)) -DEF_BENCH(return new BitmapRectBench(0xFF, SkPaint::kLow_FilterLevel, true)) +DEF_BENCH(return new BitmapRectBench(0xFF, false, true)) +DEF_BENCH(return new BitmapRectBench(0xFF, true, true)) diff --git a/bench/GameBench.cpp b/bench/GameBench.cpp index 3d371bee2b..7812285344 100644 --- a/bench/GameBench.cpp +++ b/bench/GameBench.cpp @@ -134,11 +134,11 @@ protected: SkPaint p; p.setColor(0xFF000000); - p.setFilterLevel(SkPaint::kLow_FilterLevel); + p.setFilterBitmap(true); SkPaint p2; // for drawVertices path p2.setColor(0xFF000000); - p2.setFilterLevel(SkPaint::kLow_FilterLevel); + p2.setFilterBitmap(true); p2.setShader(SkShader::CreateBitmapShader(fAtlas, SkShader::kClamp_TileMode, SkShader::kClamp_TileMode))->unref(); diff --git a/bench/Matrix44Bench.cpp b/bench/Matrix44Bench.cpp index 13169971c4..a4b36d8945 100644 --- a/bench/Matrix44Bench.cpp +++ b/bench/Matrix44Bench.cpp @@ -13,7 +13,7 @@ class Matrix44Bench : public SkBenchmark { SkString fName; public: - Matrix44Bench(const char name[]) { + Matrix44Bench(const char name[]) { fName.printf("matrix44_%s", name); fIsRendering = false; } @@ -39,12 +39,7 @@ private: class EqualsMatrix44Bench : public Matrix44Bench { public: - EqualsMatrix44Bench() - : INHERITED("equals") - , fM0(SkMatrix44::kIdentity_Constructor) - , fM1(SkMatrix44::kIdentity_Constructor) - , fM2(SkMatrix44::kIdentity_Constructor) - { + EqualsMatrix44Bench() : INHERITED("equals") { fM1.set(0, 0, 0); fM2.set(3, 3, 0); } @@ -63,10 +58,7 @@ private: class SetIdentityMatrix44Bench : public Matrix44Bench { public: - SetIdentityMatrix44Bench() - : INHERITED("setidentity") - , mat(SkMatrix44::kIdentity_Constructor) - { + SetIdentityMatrix44Bench() : INHERITED("setidentity") { double rowMajor[16] = { 1, 2, 3, 4, 5, 6, 7, 8, @@ -87,10 +79,7 @@ private: class PreScaleMatrix44Bench : public Matrix44Bench { public: - PreScaleMatrix44Bench() - : INHERITED("prescale") - , fM0(SkMatrix44::kUninitialized_Constructor) - { + PreScaleMatrix44Bench() : INHERITED("prescale") { fX = fY = fZ = SkDoubleToMScalar(1.5); } protected: @@ -108,11 +97,7 @@ private: class InvertMatrix44Bench : public Matrix44Bench { public: - InvertMatrix44Bench() - : INHERITED("invert") - , fM0(SkMatrix44::kUninitialized_Constructor) - , fM1(SkMatrix44::kUninitialized_Constructor) - { + InvertMatrix44Bench() : INHERITED("invert") { fM0.set(0, 0, -1.1); fM0.set(0, 1, 2.1); fM0.set(0, 2, -3.1); @@ -143,11 +128,7 @@ private: class InvertAffineMatrix44Bench : public Matrix44Bench { public: - InvertAffineMatrix44Bench() - : INHERITED("invertaffine") - , fM0(SkMatrix44::kIdentity_Constructor) - , fM1(SkMatrix44::kUninitialized_Constructor) - { + InvertAffineMatrix44Bench() : INHERITED("invertaffine") { fM0.set(0, 0, -1.1); fM0.set(0, 1, 2.1); fM0.set(0, 2, -3.1); @@ -175,11 +156,7 @@ private: class InvertScaleTranslateMatrix44Bench : public Matrix44Bench { public: - InvertScaleTranslateMatrix44Bench() - : INHERITED("invertscaletranslate") - , fM0(SkMatrix44::kIdentity_Constructor) - , fM1(SkMatrix44::kUninitialized_Constructor) - { + InvertScaleTranslateMatrix44Bench() : INHERITED("invertscaletranslate") { fM0.set(0, 0, -1.1); fM0.set(0, 3, 4.1); @@ -202,11 +179,7 @@ private: class InvertTranslateMatrix44Bench : public Matrix44Bench { public: - InvertTranslateMatrix44Bench() - : INHERITED("inverttranslate") - , fM0(SkMatrix44::kIdentity_Constructor) - , fM1(SkMatrix44::kUninitialized_Constructor) - { + InvertTranslateMatrix44Bench() : INHERITED("inverttranslate") { fM0.set(0, 3, 4.1); fM0.set(1, 3, 8.1); fM0.set(2, 3, -12.1); @@ -224,10 +197,7 @@ private: class PostScaleMatrix44Bench : public Matrix44Bench { public: - PostScaleMatrix44Bench() - : INHERITED("postscale") - , fM0(SkMatrix44::kUninitialized_Constructor) - { + PostScaleMatrix44Bench() : INHERITED("postscale") { fX = fY = fZ = SkDoubleToMScalar(1.5); } protected: @@ -245,12 +215,7 @@ private: class SetConcatMatrix44Bench : public Matrix44Bench { public: - SetConcatMatrix44Bench() - : INHERITED("setconcat") - , fM0(SkMatrix44::kUninitialized_Constructor) - , fM1(SkMatrix44::kUninitialized_Constructor) - , fM2(SkMatrix44::kUninitialized_Constructor) -{ + SetConcatMatrix44Bench() : INHERITED("setconcat") { fX = fY = fZ = SkDoubleToMScalar(1.5); fM1.setScale(fX, fY, fZ); fM2.setTranslate(fX, fY, fZ); @@ -270,10 +235,7 @@ private: class GetTypeMatrix44Bench : public Matrix44Bench { public: - GetTypeMatrix44Bench() - : INHERITED("gettype") - , fMatrix(SkMatrix44::kIdentity_Constructor) - {} + GetTypeMatrix44Bench() : INHERITED("gettype") {} protected: // Putting random generation of the matrix inside performTest() // would help us avoid anomalous runs, but takes up 25% or diff --git a/bench/SkBenchmark.cpp b/bench/SkBenchmark.cpp index 432d5be643..e11ebcf328 100644 --- a/bench/SkBenchmark.cpp +++ b/bench/SkBenchmark.cpp @@ -19,7 +19,6 @@ SkString SkBenchmark::gResourcePath; SkBenchmark::SkBenchmark() { fForceAlpha = 0xFF; fForceAA = true; - fForceFilter = false; fDither = SkTriState::kDefault; fIsRendering = true; fOrMask = fClearMask = 0; @@ -49,8 +48,7 @@ void SkBenchmark::postDraw() { void SkBenchmark::setupPaint(SkPaint* paint) { paint->setAlpha(fForceAlpha); paint->setAntiAlias(fForceAA); - paint->setFilterLevel(fForceFilter ? SkPaint::kLow_FilterLevel - : SkPaint::kNone_FilterLevel); + paint->setFilterBitmap(fForceFilter); paint->setFlags((paint->getFlags() & ~fClearMask) | fOrMask); diff --git a/bench/TileBench.cpp b/bench/TileBench.cpp index 4619d8b1ef..03a758cdea 100644 --- a/bench/TileBench.cpp +++ b/bench/TileBench.cpp @@ -85,8 +85,7 @@ protected: virtual void onDraw(SkCanvas* canvas) { SkPaint paint(fPaint); this->setupPaint(&paint); - paint.setFilterLevel(fDoFilter ? SkPaint::kLow_FilterLevel - : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(fDoFilter); if (fDoTrans) { paint.setColor(SkColorSetARGBMacro(0x80, 0xFF, 0xFF, 0xFF)); } diff --git a/gm/bicubicfilter.cpp b/gm/bicubicfilter.cpp index 7eb0aeebb2..c3f80df118 100644 --- a/gm/bicubicfilter.cpp +++ b/gm/bicubicfilter.cpp @@ -57,7 +57,7 @@ protected: SkSize scale = SkSize::Make(sk32, sk32); canvas->save(); canvas->scale(sk32, sk32); - bilinearPaint.setFilterLevel(SkPaint::kLow_FilterLevel); + bilinearPaint.setFilterBitmap(true); canvas->drawBitmap(fCheckerboard, 0, 0, &bilinearPaint); canvas->restore(); SkAutoTUnref<SkImageFilter> bicubic(SkBicubicImageFilter::CreateMitchell(scale)); diff --git a/gm/bigmatrix.cpp b/gm/bigmatrix.cpp index 9b49a64ea2..d4144862b8 100644 --- a/gm/bigmatrix.cpp +++ b/gm/bigmatrix.cpp @@ -80,7 +80,7 @@ protected: shader->setLocalMatrix(s); paint.setShader(shader)->unref(); paint.setAntiAlias(false); - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); rect.setLTRB(pt.fX - small, pt.fY - small, pt.fX + small, pt.fY + small); canvas->drawRect(rect, paint); diff --git a/gm/bitmapfilters.cpp b/gm/bitmapfilters.cpp index f0153e482b..d74dbf509c 100644 --- a/gm/bitmapfilters.cpp +++ b/gm/bitmapfilters.cpp @@ -39,7 +39,7 @@ static SkScalar draw_bm(SkCanvas* canvas, const SkBitmap& bm, static SkScalar draw_set(SkCanvas* c, const SkBitmap& bm, SkScalar x, SkPaint* p) { x += draw_bm(c, bm, x, 0, p); - p->setFilterLevel(SkPaint::kLow_FilterLevel); + p->setFilterBitmap(true); x += draw_bm(c, bm, x, 0, p); p->setDither(true); return x + draw_bm(c, bm, x, 0, p); diff --git a/gm/bitmapmatrix.cpp b/gm/bitmapmatrix.cpp index 46098919f3..f89cc97b2e 100644 --- a/gm/bitmapmatrix.cpp +++ b/gm/bitmapmatrix.cpp @@ -81,7 +81,7 @@ protected: // SkGpuDevice::drawPath() -> SkGpuDevice::drawWithMaskFilter() SkPaint paint; - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); SkMaskFilter* mf = SkBlurMaskFilter::Create( SkBlurMaskFilter::kNormal_BlurStyle, diff --git a/gm/bleed.cpp b/gm/bleed.cpp index aaca62bd7e..0d093940de 100644 --- a/gm/bleed.cpp +++ b/gm/bleed.cpp @@ -91,7 +91,7 @@ protected: SkRect dst = SkRect::MakeXYWH(0, 0, SkIntToScalar(kBlockSize), SkIntToScalar(kBlockSize)); SkPaint paint; - paint.setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(filter); canvas->save(); canvas->translate(SkIntToScalar(transX), SkIntToScalar(transY)); @@ -108,7 +108,7 @@ protected: SkRect dst = SkRect::MakeXYWH(0, 0, SkIntToScalar(kBlockSize), SkIntToScalar(kBlockSize)); SkPaint paint; - paint.setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(filter); canvas->save(); canvas->translate(SkIntToScalar(transX), SkIntToScalar(transY)); @@ -125,7 +125,7 @@ protected: SkRect dst = SkRect::MakeXYWH(0, 0, SkIntToScalar(kBlockSize), SkIntToScalar(kBlockSize)); SkPaint paint; - paint.setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(filter); canvas->save(); canvas->translate(SkIntToScalar(transX), SkIntToScalar(transY)); @@ -142,7 +142,7 @@ protected: SkRect dst = SkRect::MakeXYWH(0, 0, SkIntToScalar(kBlockSize), SkIntToScalar(kBlockSize)); SkPaint paint; - paint.setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(filter); SkMaskFilter* mf = SkBlurMaskFilter::Create(SkBlurMaskFilter::kNormal_BlurStyle, SkBlurMask::ConvertRadiusToSigma(SkIntToScalar(3))); paint.setMaskFilter(mf)->unref(); diff --git a/gm/blurquickreject.cpp b/gm/blurquickreject.cpp index 431f022dc3..070a59c219 100644 --- a/gm/blurquickreject.cpp +++ b/gm/blurquickreject.cpp @@ -55,7 +55,7 @@ protected: hairlinePaint.setStrokeWidth(0); SkPaint blurPaint; - blurPaint.setFilterLevel(SkPaint::kLow_FilterLevel); + blurPaint.setFilterBitmap(true); SkMaskFilter* mf = SkBlurMaskFilter::Create(SkBlurMaskFilter::kNormal_BlurStyle, SkBlurMask::ConvertRadiusToSigma(kBlurRadius)); blurPaint.setMaskFilter(mf)->unref(); diff --git a/gm/drawbitmaprect.cpp b/gm/drawbitmaprect.cpp index c88006312a..106f6d393e 100644 --- a/gm/drawbitmaprect.cpp +++ b/gm/drawbitmaprect.cpp @@ -163,7 +163,7 @@ protected: SkBitmap bm; bm = make_chessbm(5, 5); - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); srcRect.setXYWH(1, 1, 3, 3); SkMaskFilter* mf = SkBlurMaskFilter::Create( diff --git a/gm/giantbitmap.cpp b/gm/giantbitmap.cpp index ab75dc7306..0d342b8ff7 100644 --- a/gm/giantbitmap.cpp +++ b/gm/giantbitmap.cpp @@ -110,7 +110,7 @@ protected: s->setLocalMatrix(m); paint.setShader(s)->unref(); - paint.setFilterLevel(fDoFilter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(fDoFilter); canvas->translate(SkIntToScalar(50), SkIntToScalar(50)); diff --git a/gm/ninepatchstretch.cpp b/gm/ninepatchstretch.cpp index bc6f13da25..551f020625 100644 --- a/gm/ninepatchstretch.cpp +++ b/gm/ninepatchstretch.cpp @@ -93,7 +93,7 @@ protected: SkScalar y = SkIntToScalar(100); SkPaint paint; - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); for (int iy = 0; iy < 2; ++iy) { for (int ix = 0; ix < 2; ++ix) { diff --git a/gm/shadertext2.cpp b/gm/shadertext2.cpp index a7da436823..e760e548d5 100644 --- a/gm/shadertext2.cpp +++ b/gm/shadertext2.cpp @@ -102,7 +102,7 @@ protected: SkPaint fillPaint; fillPaint.setAntiAlias(true); fillPaint.setTextSize(SkIntToScalar(kPointSize)); - fillPaint.setFilterLevel(SkPaint::kLow_FilterLevel); + fillPaint.setFilterBitmap(true); fillPaint.setShader(shader); SkPaint outlinePaint; diff --git a/gm/shadertext3.cpp b/gm/shadertext3.cpp index 36abf9f8e5..fa327c8bff 100644 --- a/gm/shadertext3.cpp +++ b/gm/shadertext3.cpp @@ -74,7 +74,7 @@ protected: SkPaint bmpPaint; bmpPaint.setAntiAlias(true); - bmpPaint.setFilterLevel(SkPaint::kLow_FilterLevel); + bmpPaint.setFilterBitmap(true); bmpPaint.setAlpha(0x80); canvas->drawBitmap(bmp, 5.f, 5.f, &bmpPaint); @@ -113,7 +113,7 @@ protected: SkPaint fillPaint; fillPaint.setAntiAlias(true); fillPaint.setTextSize(SkIntToScalar(kPointSize)); - fillPaint.setFilterLevel(SkPaint::kLow_FilterLevel); + fillPaint.setFilterBitmap(true); fillPaint.setShader(shader); canvas->drawText(kText, kTextLen, 0, 0, fillPaint); diff --git a/gm/tilemodes.cpp b/gm/tilemodes.cpp index dcfc133d37..5a5fbbe529 100644 --- a/gm/tilemodes.cpp +++ b/gm/tilemodes.cpp @@ -47,7 +47,7 @@ static void setup(SkPaint* paint, const SkBitmap& bm, bool filter, SkShader::TileMode tmx, SkShader::TileMode tmy) { SkShader* shader = SkShader::CreateBitmapShader(bm, tmx, tmy); paint->setShader(shader)->unref(); - paint->setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint->setFilterBitmap(filter); } static const SkBitmap::Config gConfigs[] = { diff --git a/include/core/SkBitmap.h b/include/core/SkBitmap.h index 59b6482735..3e22069e60 100644 --- a/include/core/SkBitmap.h +++ b/include/core/SkBitmap.h @@ -96,19 +96,20 @@ public: */ bool isNull() const { return NULL == fPixelRef; } - /** Return the config for the bitmap. */ + /** Return the config for the bitmap. + */ Config config() const { return (Config)fConfig; } - - SK_ATTR_DEPRECATED("use config()") + /** DEPRECATED, use config() + */ Config getConfig() const { return this->config(); } - - /** Return the bitmap's width, in pixels. */ + /** Return the bitmap's width, in pixels. + */ int width() const { return fWidth; } - - /** Return the bitmap's height, in pixels. */ + /** Return the bitmap's height, in pixels. + */ int height() const { return fHeight; } - - /** Return the number of bytes between subsequent rows of the bitmap. */ + /** Return the number of bytes between subsequent rows of the bitmap. + */ size_t rowBytes() const { return fRowBytes; } /** Return the shift amount per pixel (i.e. 0 for 1-byte per pixel, 1 for @@ -188,7 +189,9 @@ public: return SkAlphaTypeIsOpaque(this->alphaType()); } - SK_ATTR_DEPRECATED("use setAlphaType") + /** + * DEPRECATED: use setAlpahType() instead. + */ void setIsOpaque(bool opaque) { this->setAlphaType(opaque ? kOpaque_SkAlphaType : kPremul_SkAlphaType); } @@ -416,7 +419,7 @@ public: */ void eraseARGB(U8CPU a, U8CPU r, U8CPU g, U8CPU b) const; - SK_ATTR_DEPRECATED("use eraseARGB or eraseColor") + // DEPRECATED -- call eraseColor or eraseARGB void eraseRGB(U8CPU r, U8CPU g, U8CPU b) const { this->eraseARGB(0xFF, r, g, b); } @@ -550,7 +553,9 @@ public: */ bool canCopyTo(Config newConfig) const; - SK_ATTR_DEPRECATED("use setFilterLevel on SkPaint") + /** + * DEPRECATED -- will be replaced with API on SkPaint + */ void buildMipMap(bool forceRebuild = false); #ifdef SK_BUILD_FOR_ANDROID diff --git a/include/core/SkBitmapDevice.h b/include/core/SkBitmapDevice.h index 530b85c9f8..a288f8a3e2 100644 --- a/include/core/SkBitmapDevice.h +++ b/include/core/SkBitmapDevice.h @@ -87,7 +87,7 @@ public: /** Return the bitmap config of the device's pixels */ - virtual SkBitmap::Config config() const SK_OVERRIDE { return fBitmap.config(); } + virtual SkBitmap::Config config() const SK_OVERRIDE { return fBitmap.getConfig(); } /** * DEPRECATED: This will be made protected once WebKit stops using it. diff --git a/include/core/SkCanvas.h b/include/core/SkCanvas.h index d0b8fc94be..dc3953ed4b 100644 --- a/include/core/SkCanvas.h +++ b/include/core/SkCanvas.h @@ -57,7 +57,7 @@ public: */ explicit SkCanvas(SkBaseDevice* device); - /** Construct a canvas with the specified bitmap to draw into. + /** Deprecated - Construct a canvas with the specified bitmap to draw into. @param bitmap Specifies a bitmap for the canvas to draw into. Its structure are copied to the canvas. */ @@ -942,10 +942,11 @@ public: */ ClipType getClipType() const; - /** DEPRECATED -- need to move this guy to private/friend - * Return the current device clip (concatenation of all clip calls). + /** Return the current device clip (concatenation of all clip calls). * This does not account for the translate in any of the devices. * @return the current device clip (concatenation of all clip calls). + * + * DEPRECATED -- call getClipDeviceBounds() instead. */ const SkRegion& getTotalClip() const; @@ -1033,8 +1034,8 @@ protected: // can perform copy-on-write or invalidate any cached images void predrawNotify(); - /** - DEPRECATED -- need to remove when subclass stop relying on it. + /** DEPRECATED -- use constructor(device) + Marked as 'protected' to avoid new clients using this before we can completely remove it. diff --git a/include/core/SkDevice.h b/include/core/SkDevice.h index e38e2aa428..ed1c8fd63b 100644 --- a/include/core/SkDevice.h +++ b/include/core/SkDevice.h @@ -90,9 +90,8 @@ public: */ virtual bool isOpaque() const = 0; - /** Return the bitmap config of the device's pixels + /** DEPRECATED - Return the bitmap config of the device's pixels */ - SK_ATTR_DEPRECATED("want to hide configness of the device -- don't use") virtual SkBitmap::Config config() const = 0; /** Return the bitmap associated with this device. Call this each time you need @@ -118,7 +117,6 @@ public: * will determines how the pixel valuess are intepreted. If the bitmap is * not kARGB_8888_Config then this parameter is ignored. */ - SK_ATTR_DEPRECATED("use SkCanvas version of same call") virtual void writePixels(const SkBitmap& bitmap, int x, int y, SkCanvas::Config8888 config8888 = SkCanvas::kNative_Premul_Config8888) = 0; diff --git a/include/core/SkPaint.h b/include/core/SkPaint.h index 1244795819..6c3a10f9ef 100644 --- a/include/core/SkPaint.h +++ b/include/core/SkPaint.h @@ -304,18 +304,18 @@ public: void setFilterLevel(FilterLevel); /** + * DEPRECATED: use setFilterLevel instead. * If the predicate is true, set the filterLevel to Low, else set it to * None. */ - SK_ATTR_DEPRECATED("use setFilterLevel") void setFilterBitmap(bool doFilter) { this->setFilterLevel(doFilter ? kLow_FilterLevel : kNone_FilterLevel); } /** + * DEPRECATED: call getFilterLevel() instead. * Returns true if getFilterLevel() returns anything other than None. */ - SK_ATTR_DEPRECATED("use getFilterLevel") bool isFilterBitmap() const { return kNone_FilterLevel != this->getFilterLevel(); } @@ -638,8 +638,9 @@ public: /** * Returns true if there is an annotation installed on this paint, and * the annotation specifics no-drawing. + * + * Deprecated: all annotations are no-draw. Please just check getAnnotation(). */ - SK_ATTR_DEPRECATED("use getAnnotation and check for non-null") bool isNoDrawAnnotation() const { return this->getAnnotation() != NULL; } /** diff --git a/include/core/SkPath.h b/include/core/SkPath.h index 785eb7793f..02e3423550 100644 --- a/include/core/SkPath.h +++ b/include/core/SkPath.h @@ -123,6 +123,7 @@ public: void setConvexity(Convexity); /** + * DEPRECATED: use getConvexity() * Returns true if the path is flagged as being convex. This is not a * confirmed by any analysis, it is just the value set earlier. */ @@ -131,12 +132,12 @@ public: } /** + * DEPRECATED: use setConvexity() * Set the isConvex flag to true or false. Convex paths may draw faster if * this flag is set, though setting this to true on a path that is in fact * not convex can give undefined results when drawn. Paths default to * isConvex == false */ - SK_ATTR_DEPRECATED("use setConvexity") void setIsConvex(bool isConvex) { this->setConvexity(isConvex ? kConvex_Convexity : kConcave_Convexity); } diff --git a/include/core/SkPoint.h b/include/core/SkPoint.h index b94f730ec2..5eca72f6b6 100644 --- a/include/core/SkPoint.h +++ b/include/core/SkPoint.h @@ -353,7 +353,7 @@ struct SK_API SkPoint { /** Return true if this point and the given point are far enough apart such that a vector between them would be non-degenerate. - WARNING: Unlike the explicit tolerance version, + WARNING: Unlike the deprecated version of equalsWithinTolerance(), this method does not use componentwise comparison. Instead, it uses a comparison designed to match judgments elsewhere regarding degeneracy ("points A and B are so close that the vector between them @@ -363,7 +363,10 @@ struct SK_API SkPoint { return !CanNormalize(fX - p.fX, fY - p.fY); } - /** WARNING: There is no guarantee that the result will reflect judgments + /** DEPRECATED: Return true if this and the given point are componentwise + within tolerance "tol". + + WARNING: There is no guarantee that the result will reflect judgments elsewhere regarding degeneracy ("points A and B are so close that the vector between them is essentially zero"). */ diff --git a/include/core/SkPostConfig.h b/include/core/SkPostConfig.h index 61998f621a..0c2358aeb8 100644 --- a/include/core/SkPostConfig.h +++ b/include/core/SkPostConfig.h @@ -374,11 +374,6 @@ #define SK_UNUSED SK_ATTRIBUTE(unused) #endif -#if !defined(SK_ATTR_DEPRECATED) - // we ignore msg for now... - #define SK_ATTR_DEPRECATED(msg) SK_ATTRIBUTE(deprecated) -#endif - ////////////////////////////////////////////////////////////////////// #ifndef SK_PRINTF_LIKE diff --git a/include/core/SkPreConfig.h b/include/core/SkPreConfig.h index ab995d7aa8..db40f695b0 100644 --- a/include/core/SkPreConfig.h +++ b/include/core/SkPreConfig.h @@ -14,13 +14,6 @@ #include "config.h" #endif -// Allows embedders that want to disable macros that take arguments to just -// define that symbol to be one of these -// -#define SK_NOTHING_ARG1(arg1) -#define SK_NOTHING_ARG2(arg1, arg2) -#define SK_NOTHING_ARG3(arg1, arg2, arg3) - ////////////////////////////////////////////////////////////////////// #if !defined(SK_BUILD_FOR_ANDROID) && !defined(SK_BUILD_FOR_IOS) && !defined(SK_BUILD_FOR_PALM) && !defined(SK_BUILD_FOR_WINCE) && !defined(SK_BUILD_FOR_WIN32) && !defined(SK_BUILD_FOR_UNIX) && !defined(SK_BUILD_FOR_MAC) && !defined(SK_BUILD_FOR_SDL) && !defined(SK_BUILD_FOR_BREW) && !defined(SK_BUILD_FOR_NACL) diff --git a/include/core/SkRect.h b/include/core/SkRect.h index 98acb1cda3..d9ac3a6910 100644 --- a/include/core/SkRect.h +++ b/include/core/SkRect.h @@ -401,7 +401,7 @@ struct SK_API SkRect { return r; } - SK_ATTR_DEPRECATED("use Make()") + // DEPRECATED: call Make(r) static SkRect SK_WARN_UNUSED_RESULT MakeFromIRect(const SkIRect& irect) { SkRect r; r.set(SkIntToScalar(irect.fLeft), diff --git a/include/core/SkWriter32.h b/include/core/SkWriter32.h index 9fb1f7b85b..ba8893eab9 100644 --- a/include/core/SkWriter32.h +++ b/include/core/SkWriter32.h @@ -46,8 +46,7 @@ public: // return the current offset (will always be a multiple of 4) size_t bytesWritten() const { return fSize; } - - SK_ATTR_DEPRECATED("use bytesWritten") + // DEPRECATED: use bytesWritten instead TODO(mtklein): clean up size_t size() const { return this->bytesWritten(); } // Returns true if we've written only into the storage passed into constructor or reset. diff --git a/include/core/SkXfermode.h b/include/core/SkXfermode.h index 3ef2241c7d..5bf8a22517 100644 --- a/include/core/SkXfermode.h +++ b/include/core/SkXfermode.h @@ -186,7 +186,7 @@ public: */ static bool ModeAsCoeff(Mode mode, Coeff* src, Coeff* dst); - SK_ATTR_DEPRECATED("use AsMode(...)") + // DEPRECATED: call AsMode(...) static bool IsMode(const SkXfermode* xfer, Mode* mode) { return AsMode(xfer, mode); } diff --git a/include/effects/SkBlurDrawLooper.h b/include/effects/SkBlurDrawLooper.h index 013e80714f..8e0c2dd001 100644 --- a/include/effects/SkBlurDrawLooper.h +++ b/include/effects/SkBlurDrawLooper.h @@ -38,7 +38,7 @@ public: SkBlurDrawLooper(SkColor color, SkScalar sigma, SkScalar dx, SkScalar dy, uint32_t flags = kNone_BlurFlag); -// SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based SkBlurDrawLooper(SkScalar radius, SkScalar dx, SkScalar dy, SkColor color, uint32_t flags = kNone_BlurFlag); virtual ~SkBlurDrawLooper(); diff --git a/include/effects/SkBlurMaskFilter.h b/include/effects/SkBlurMaskFilter.h index 9b352dc1e4..d590b275d8 100644 --- a/include/effects/SkBlurMaskFilter.h +++ b/include/effects/SkBlurMaskFilter.h @@ -33,7 +33,9 @@ public: kAll_BlurFlag = 0x03 }; - SK_ATTR_DEPRECATED("use sigma version") + /** + * DEPRECATED - radius-based + */ static SkMaskFilter* Create(SkScalar radius, BlurStyle style, uint32_t flags = kNone_BlurFlag); @@ -57,7 +59,7 @@ public: static SkMaskFilter* CreateEmboss(SkScalar blurSigma, const SkScalar direction[3], SkScalar ambient, SkScalar specular); - SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based static SkMaskFilter* CreateEmboss(const SkScalar direction[3], SkScalar ambient, SkScalar specular, SkScalar blurRadius); diff --git a/include/effects/SkEmbossMaskFilter.h b/include/effects/SkEmbossMaskFilter.h index 7210b78c5c..4b5bd6f983 100644 --- a/include/effects/SkEmbossMaskFilter.h +++ b/include/effects/SkEmbossMaskFilter.h @@ -25,7 +25,7 @@ public: SkEmbossMaskFilter(SkScalar blurSigma, const Light& light); - SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based SkEmbossMaskFilter(const Light& light, SkScalar blurRadius); // overrides from SkMaskFilter diff --git a/include/effects/SkPorterDuff.h b/include/effects/SkPorterDuff.h index e984e8e9b4..c5f549211d 100644 --- a/include/effects/SkPorterDuff.h +++ b/include/effects/SkPorterDuff.h @@ -13,6 +13,8 @@ class SkXfermode; +/** DEPRECATED - use SkXfermode::Mode instead + */ class SK_API SkPorterDuff { public: /** List of predefined xfermodes. In general, the algebra for the modes @@ -74,6 +76,6 @@ public: /** Return the corersponding SkXfermode::Mode */ static SkXfermode::Mode ToXfermodeMode(Mode); -} SK_ATTR_DEPRECATED("use SkXfermode::Mode"); +}; #endif diff --git a/include/effects/SkRectShaderImageFilter.h b/include/effects/SkRectShaderImageFilter.h index ea974785a3..650a5833ad 100644 --- a/include/effects/SkRectShaderImageFilter.h +++ b/include/effects/SkRectShaderImageFilter.h @@ -25,9 +25,7 @@ public: * If NULL or a given crop edge is not specified, the source * primitive's bounds are used instead. */ - SK_ATTR_DEPRECATED("use Create(SkShader*, const CropRect*)") - static SkRectShaderImageFilter* Create(SkShader* s, const SkRect& rect); - + /* DEPRECATED */ static SkRectShaderImageFilter* Create(SkShader* s, const SkRect& rect); static SkRectShaderImageFilter* Create(SkShader* s, const CropRect* rect = NULL); virtual ~SkRectShaderImageFilter(); diff --git a/include/gpu/SkGpuDevice.h b/include/gpu/SkGpuDevice.h index 29f59f28df..0862514e78 100644 --- a/include/gpu/SkGpuDevice.h +++ b/include/gpu/SkGpuDevice.h @@ -43,16 +43,16 @@ public: SkGpuDevice(GrContext*, SkBitmap::Config, int width, int height, int sampleCount = 0); /** - * DEPRECATED -- need to make this private, call Create(surface) * New device that will render to the specified renderTarget. + * DEPRECATED: Use Create(surface) */ SkGpuDevice(GrContext*, GrRenderTarget*); /** - * DEPRECATED -- need to make this private, call Create(surface) * New device that will render to the texture (as a rendertarget). * The GrTexture's asRenderTarget() must be non-NULL or device will not * function. + * DEPRECATED: Use Create(surface) */ SkGpuDevice(GrContext*, GrTexture*); diff --git a/include/utils/SkMatrix44.h b/include/utils/SkMatrix44.h index 26247a01e9..9282770588 100644 --- a/include/utils/SkMatrix44.h +++ b/include/utils/SkMatrix44.h @@ -112,7 +112,7 @@ public: SkMatrix44(Uninitialized_Constructor) { } SkMatrix44(Identity_Constructor) { this->setIdentity(); } - SK_ATTR_DEPRECATED("use the constructors that take an enum") + // DEPRECATED: use the constructors that take an enum SkMatrix44() { this->setIdentity(); } SkMatrix44(const SkMatrix44& src) { @@ -335,12 +335,11 @@ public: this->mapScalars(vec, vec); } - SK_ATTR_DEPRECATED("use mapScalars") + // DEPRECATED: call mapScalars() void map(const SkScalar src[4], SkScalar dst[4]) const { this->mapScalars(src, dst); } - - SK_ATTR_DEPRECATED("use mapScalars") + // DEPRECATED: call mapScalars() void map(SkScalar vec[4]) const { this->mapScalars(vec, vec); } @@ -358,7 +357,7 @@ public: friend SkVector4 operator*(const SkMatrix44& m, const SkVector4& src) { SkVector4 dst; - m.mapScalars(src.fData, dst.fData); + m.map(src.fData, dst.fData); return dst; } diff --git a/samplecode/SampleApp.cpp b/samplecode/SampleApp.cpp index 04d2cdd785..f044e70f3d 100644 --- a/samplecode/SampleApp.cpp +++ b/samplecode/SampleApp.cpp @@ -505,8 +505,7 @@ public: paint->setAntiAlias(SkOSMenu::kOnState == fAAState); } if (SkOSMenu::kMixedState != fFilterState) { - paint->setFilterLevel(SkOSMenu::kOnState == fFilterState ? - SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint->setFilterBitmap(SkOSMenu::kOnState == fFilterState); } if (SkOSMenu::kMixedState != fSubpixelState) { paint->setSubpixelText(SkOSMenu::kOnState == fSubpixelState); diff --git a/samplecode/SampleBitmapRect.cpp b/samplecode/SampleBitmapRect.cpp index 422ed583ef..04297e05e7 100644 --- a/samplecode/SampleBitmapRect.cpp +++ b/samplecode/SampleBitmapRect.cpp @@ -133,7 +133,7 @@ protected: canvas->drawRect(srcR, paint); for (int i = 0; i < 2; ++i) { - paint.setFilterLevel(1 == i ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(1 == i); canvas->drawBitmapRectToRect(bitmap, &srcR, fDstR[i], &paint); canvas->drawRect(fDstR[i], paint); } @@ -226,7 +226,7 @@ protected: paint.setColor(SK_ColorYELLOW); for (int i = 0; i < 2; ++i) { - paint.setFilterLevel(1 == i ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(1 == i); canvas->drawBitmapRectToRect(fBitmap, &fSrcR, fDstR[i], &paint); canvas->drawRect(fDstR[i], paint); } diff --git a/samplecode/SampleFilter.cpp b/samplecode/SampleFilter.cpp index 71ab62fd55..3eb9cdc92c 100644 --- a/samplecode/SampleFilter.cpp +++ b/samplecode/SampleFilter.cpp @@ -59,7 +59,7 @@ static SkScalar draw_bm(SkCanvas* canvas, const SkBitmap& bm, static SkScalar draw_set(SkCanvas* c, const SkBitmap& bm, SkScalar x, SkPaint* p) { x += draw_bm(c, bm, x, 0, p); - p->setFilterLevel(SkPaint::kLow_FilterLevel); + p->setFilterBitmap(true); x += draw_bm(c, bm, x, 0, p); p->setDither(true); return x + draw_bm(c, bm, x, 0, p); diff --git a/samplecode/SampleFilter2.cpp b/samplecode/SampleFilter2.cpp index e982778234..f3a8b469ab 100644 --- a/samplecode/SampleFilter2.cpp +++ b/samplecode/SampleFilter2.cpp @@ -76,7 +76,7 @@ protected: canvas->scale(SK_Scalar1, scale); for (int k = 0; k < 2; k++) { - paint.setFilterLevel(k == 1 ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint.setFilterBitmap(k == 1); for (int j = 0; j < 2; j++) { paint.setDither(j == 1); for (int i = 0; i < fBitmapCount; i++) { @@ -93,7 +93,7 @@ protected: SkString s("dither="); s.appendS32(paint.isDither()); s.append(" filter="); - s.appendS32(paint.getFilterLevel() != SkPaint::kNone_FilterLevel); + s.appendS32(paint.isFilterBitmap()); canvas->drawText(s.c_str(), s.size(), x + W/2, y - p.getTextSize(), p); } diff --git a/samplecode/SampleFilterFuzz.cpp b/samplecode/SampleFilterFuzz.cpp index b10fb31834..c04292dadf 100644 --- a/samplecode/SampleFilterFuzz.cpp +++ b/samplecode/SampleFilterFuzz.cpp @@ -231,9 +231,9 @@ static SkImageFilter* make_image_filter(bool canBeNull = true) { make_scalar(true), make_scalar(true), R(10.0f), make_scalar()) : SkPerlinNoiseShader::CreateTubulence( make_scalar(true), make_scalar(true), R(10.0f), make_scalar())); - SkImageFilter::CropRect cropR(SkRect::MakeWH(SkIntToScalar(kBitmapSize), - SkIntToScalar(kBitmapSize))); - filter = SkRectShaderImageFilter::Create(shader, &cropR); + filter = SkRectShaderImageFilter::Create(shader, + SkRect::MakeWH(SkIntToScalar(kBitmapSize), + SkIntToScalar(kBitmapSize))); } break; case DROP_SHADOW: diff --git a/samplecode/SamplePatch.cpp b/samplecode/SamplePatch.cpp index c630e8956a..3da4b0c20e 100644 --- a/samplecode/SamplePatch.cpp +++ b/samplecode/SamplePatch.cpp @@ -269,7 +269,7 @@ protected: virtual void onDrawContent(SkCanvas* canvas) { SkPaint paint; paint.setDither(true); - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); canvas->translate(SkIntToScalar(20), 0); diff --git a/samplecode/SampleSlides.cpp b/samplecode/SampleSlides.cpp index 3dba782331..98e4e72618 100644 --- a/samplecode/SampleSlides.cpp +++ b/samplecode/SampleSlides.cpp @@ -444,7 +444,7 @@ static void mesh_slide(SkCanvas* canvas) { SkPaint paint; paint.setDither(true); - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); for (size_t i = 0; i < SK_ARRAY_COUNT(fRecs); i++) { canvas->save(); diff --git a/samplecode/SampleTextureDomain.cpp b/samplecode/SampleTextureDomain.cpp index 16e7a16cdd..154eaa9fd8 100644 --- a/samplecode/SampleTextureDomain.cpp +++ b/samplecode/SampleTextureDomain.cpp @@ -48,7 +48,7 @@ protected: SkIRect srcRect; SkRect dstRect; SkPaint paint; - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); // Test that bitmap draws from malloc-backed bitmaps respect // the constrained texture domain. diff --git a/samplecode/SampleTiling.cpp b/samplecode/SampleTiling.cpp index c35cf88846..eb791c5b39 100644 --- a/samplecode/SampleTiling.cpp +++ b/samplecode/SampleTiling.cpp @@ -52,7 +52,7 @@ static void setup(SkPaint* paint, const SkBitmap& bm, bool filter, SkShader::TileMode tmx, SkShader::TileMode tmy) { SkShader* shader = SkShader::CreateBitmapShader(bm, tmx, tmy); paint->setShader(shader)->unref(); - paint->setFilterLevel(filter ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint->setFilterBitmap(filter); } static const SkBitmap::Config gConfigs[] = { diff --git a/samplecode/SampleVertices.cpp b/samplecode/SampleVertices.cpp index 323c4b58cb..53db2ea3fa 100644 --- a/samplecode/SampleVertices.cpp +++ b/samplecode/SampleVertices.cpp @@ -91,7 +91,7 @@ protected: virtual void onDrawContent(SkCanvas* canvas) { SkPaint paint; paint.setDither(true); - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); for (size_t i = 0; i < SK_ARRAY_COUNT(fRecs); i++) { canvas->save(); diff --git a/src/animator/SkDrawPaint.cpp b/src/animator/SkDrawPaint.cpp index 825d0486a7..22d99e4f21 100644 --- a/src/animator/SkDrawPaint.cpp +++ b/src/animator/SkDrawPaint.cpp @@ -220,7 +220,7 @@ void SkDrawPaint::setupPaint(SkPaint* paint) const { if (fakeBold != -1) paint->setFakeBoldText(SkToBool(fakeBold)); if (filterBitmap != -1) - paint->setFilterLevel(filterBitmap ? SkPaint::kLow_FilterLevel : SkPaint::kNone_FilterLevel); + paint->setFilterBitmap(SkToBool(filterBitmap)); // stroke is legacy; style setting if present overrides stroke if (stroke != -1) paint->setStyle(SkToBool(stroke) ? SkPaint::kStroke_Style : SkPaint::kFill_Style); diff --git a/src/core/SkBBoxHierarchyRecord.cpp b/src/core/SkBBoxHierarchyRecord.cpp index 8e5861bb21..61a82cef3d 100644 --- a/src/core/SkBBoxHierarchyRecord.cpp +++ b/src/core/SkBBoxHierarchyRecord.cpp @@ -22,7 +22,7 @@ SkBBoxHierarchyRecord::SkBBoxHierarchyRecord(uint32_t recordFlags, void SkBBoxHierarchyRecord::handleBBox(const SkRect& bounds) { SkIRect r; bounds.roundOut(&r); - SkPictureStateTree::Draw* draw = fStateTree->appendDraw(this->writeStream().bytesWritten()); + SkPictureStateTree::Draw* draw = fStateTree->appendDraw(this->writeStream().size()); fBoundingHierarchy->insert(draw, r, true); } @@ -33,7 +33,7 @@ int SkBBoxHierarchyRecord::save(SaveFlags flags) { int SkBBoxHierarchyRecord::saveLayer(const SkRect* bounds, const SkPaint* paint, SaveFlags flags) { - fStateTree->appendSaveLayer(this->writeStream().bytesWritten()); + fStateTree->appendSaveLayer(this->writeStream().size()); return INHERITED::saveLayer(bounds, paint, flags); } @@ -80,27 +80,27 @@ void SkBBoxHierarchyRecord::setMatrix(const SkMatrix& matrix) { bool SkBBoxHierarchyRecord::clipRect(const SkRect& rect, SkRegion::Op op, bool doAntiAlias) { - fStateTree->appendClip(this->writeStream().bytesWritten()); + fStateTree->appendClip(this->writeStream().size()); return INHERITED::clipRect(rect, op, doAntiAlias); } bool SkBBoxHierarchyRecord::clipRegion(const SkRegion& region, SkRegion::Op op) { - fStateTree->appendClip(this->writeStream().bytesWritten()); + fStateTree->appendClip(this->writeStream().size()); return INHERITED::clipRegion(region, op); } bool SkBBoxHierarchyRecord::clipPath(const SkPath& path, SkRegion::Op op, bool doAntiAlias) { - fStateTree->appendClip(this->writeStream().bytesWritten()); + fStateTree->appendClip(this->writeStream().size()); return INHERITED::clipPath(path, op, doAntiAlias); } bool SkBBoxHierarchyRecord::clipRRect(const SkRRect& rrect, SkRegion::Op op, bool doAntiAlias) { - fStateTree->appendClip(this->writeStream().bytesWritten()); + fStateTree->appendClip(this->writeStream().size()); return INHERITED::clipRRect(rrect, op, doAntiAlias); } @@ -109,7 +109,7 @@ bool SkBBoxHierarchyRecord::shouldRewind(void* data) { // SkPicture has rewound its command stream. To match that rewind in the // BBH, we rewind all draws that reference commands that were recorded // past the point to which the SkPicture has rewound, which is given by - // writeStream().bytesWritten(). + // writeStream().size(). SkPictureStateTree::Draw* draw = static_cast<SkPictureStateTree::Draw*>(data); - return draw->fOffset >= writeStream().bytesWritten(); + return draw->fOffset >= writeStream().size(); } diff --git a/src/core/SkBBoxRecord.cpp b/src/core/SkBBoxRecord.cpp index 2349b1c3e1..1e6c69ba09 100644 --- a/src/core/SkBBoxRecord.cpp +++ b/src/core/SkBBoxRecord.cpp @@ -32,7 +32,7 @@ void SkBBoxRecord::drawPath(const SkPath& path, const SkPaint& paint) { // path's device-space bounding box. SkIRect clipBounds; if (this->getClipDeviceBounds(&clipBounds)) { - this->handleBBox(SkRect::Make(clipBounds)); + this->handleBBox(SkRect::MakeFromIRect(clipBounds)); INHERITED::drawPath(path, paint); } } else if (this->transformBounds(path.getBounds(), &paint)) { diff --git a/src/core/SkBitmap.cpp b/src/core/SkBitmap.cpp index bff08801fc..2f1c9a49ad 100644 --- a/src/core/SkBitmap.cpp +++ b/src/core/SkBitmap.cpp @@ -502,11 +502,11 @@ size_t SkBitmap::getSafeSize() const { // This is intended to be a size_t version of ComputeSafeSize64(), just // faster. The computation is meant to be identical. return (fHeight ? ((fHeight - 1) * fRowBytes) + - ComputeRowBytes(this->config(), fWidth): 0); + ComputeRowBytes(getConfig(), fWidth): 0); } Sk64 SkBitmap::getSafeSize64() const { - return ComputeSafeSize64(this->config(), fWidth, fHeight, fRowBytes); + return ComputeSafeSize64(getConfig(), fWidth, fHeight, fRowBytes); } bool SkBitmap::copyPixelsTo(void* const dst, size_t dstSize, @@ -516,12 +516,12 @@ bool SkBitmap::copyPixelsTo(void* const dst, size_t dstSize, dstRowBytes = fRowBytes; } - if (dstRowBytes < ComputeRowBytes(this->config(), fWidth) || + if (dstRowBytes < ComputeRowBytes(getConfig(), fWidth) || dst == NULL || (getPixels() == NULL && pixelRef() == NULL)) return false; if (!preserveDstPad && static_cast<uint32_t>(dstRowBytes) == fRowBytes) { - size_t safeSize = this->getSafeSize(); + size_t safeSize = getSafeSize(); if (safeSize > dstSize || safeSize == 0) return false; else { @@ -535,12 +535,12 @@ bool SkBitmap::copyPixelsTo(void* const dst, size_t dstSize, } } else { // If destination has different stride than us, then copy line by line. - if (ComputeSafeSize(this->config(), fWidth, fHeight, dstRowBytes) > + if (ComputeSafeSize(getConfig(), fWidth, fHeight, dstRowBytes) > dstSize) return false; else { // Just copy what we need on each line. - size_t rowBytes = ComputeRowBytes(this->config(), fWidth); + size_t rowBytes = ComputeRowBytes(getConfig(), fWidth); SkAutoLockPixels lock(*this); const uint8_t* srcP = reinterpret_cast<const uint8_t*>(getPixels()); uint8_t* dstP = reinterpret_cast<uint8_t*>(dst); @@ -874,7 +874,7 @@ void SkBitmap::eraseArea(const SkIRect& rect, SkColor c) const { * within the bounds of the SkPixelRef being used. */ static size_t get_sub_offset(const SkBitmap& bm, int x, int y) { - switch (bm.config()) { + switch (bm.getConfig()) { case SkBitmap::kA8_Config: case SkBitmap:: kIndex8_Config: // x is fine as is for the calculation @@ -1005,7 +1005,7 @@ bool SkBitmap::extractSubset(SkBitmap* result, const SkIRect& subset) const { #include "SkPaint.h" bool SkBitmap::canCopyTo(Config dstConfig) const { - if (this->config() == kNo_Config) { + if (this->getConfig() == kNo_Config) { return false; } @@ -1028,7 +1028,7 @@ bool SkBitmap::canCopyTo(Config dstConfig) const { } // do not copy src if srcConfig == kA1_Config while dstConfig != kA1_Config - if (this->config() == kA1_Config && !sameConfigs) { + if (this->getConfig() == kA1_Config && !sameConfigs) { return false; } @@ -1305,7 +1305,7 @@ void SkBitmap::buildMipMap(bool forceRebuild) { void (*proc)(SkBitmap* dst, int x, int y, const SkBitmap& src); - const SkBitmap::Config config = this->config(); + const SkBitmap::Config config = this->getConfig(); switch (config) { case kARGB_8888_Config: @@ -1444,7 +1444,7 @@ static bool GetBitmapAlpha(const SkBitmap& src, uint8_t* SK_RESTRICT alpha, SkASSERT(alpha != NULL); SkASSERT(alphaRowBytes >= src.width()); - SkBitmap::Config config = src.config(); + SkBitmap::Config config = src.getConfig(); int w = src.width(); int h = src.height(); size_t rb = src.rowBytes(); diff --git a/src/core/SkBitmapDevice.cpp b/src/core/SkBitmapDevice.cpp index eac21e2d48..ff68481538 100644 --- a/src/core/SkBitmapDevice.cpp +++ b/src/core/SkBitmapDevice.cpp @@ -13,8 +13,8 @@ SK_DEFINE_INST_COUNT(SkBitmapDevice) -#define CHECK_FOR_ANNOTATION(paint) \ - do { if (paint.getAnnotation()) { return; } } while (0) +#define CHECK_FOR_NODRAW_ANNOTATION(paint) \ + do { if (paint.isNoDrawAnnotation()) { return; } } while (0) SkBitmapDevice::SkBitmapDevice(const SkBitmap& bitmap) : fBitmap(bitmap) { @@ -210,17 +210,17 @@ void SkBitmapDevice::drawPaint(const SkDraw& draw, const SkPaint& paint) { void SkBitmapDevice::drawPoints(const SkDraw& draw, SkCanvas::PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); draw.drawPoints(mode, count, pts, paint); } void SkBitmapDevice::drawRect(const SkDraw& draw, const SkRect& r, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); draw.drawRect(r, paint); } void SkBitmapDevice::drawOval(const SkDraw& draw, const SkRect& oval, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); SkPath path; path.addOval(oval); @@ -230,7 +230,7 @@ void SkBitmapDevice::drawOval(const SkDraw& draw, const SkRect& oval, const SkPa } void SkBitmapDevice::drawRRect(const SkDraw& draw, const SkRRect& rrect, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); SkPath path; path.addRRect(rrect); @@ -242,7 +242,7 @@ void SkBitmapDevice::drawRRect(const SkDraw& draw, const SkRRect& rrect, const S void SkBitmapDevice::drawPath(const SkDraw& draw, const SkPath& path, const SkPaint& paint, const SkMatrix* prePathMatrix, bool pathIsMutable) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); draw.drawPath(path, paint, prePathMatrix, pathIsMutable); } diff --git a/src/core/SkBitmapHeap.cpp b/src/core/SkBitmapHeap.cpp index a17113842f..f3428db64f 100644 --- a/src/core/SkBitmapHeap.cpp +++ b/src/core/SkBitmapHeap.cpp @@ -259,7 +259,7 @@ bool SkBitmapHeap::copyBitmap(const SkBitmap& originalBitmap, SkBitmap& copiedBi // copiedBitmap.setPixelRef(sharedPixelRef, originalBitmap.pixelRefOffset()); } else if (originalBitmap.empty()) { copiedBitmap.reset(); - } else if (!originalBitmap.deepCopyTo(&copiedBitmap, originalBitmap.config())) { + } else if (!originalBitmap.deepCopyTo(&copiedBitmap, originalBitmap.getConfig())) { return false; } copiedBitmap.setImmutable(); diff --git a/src/core/SkBlitter.cpp b/src/core/SkBlitter.cpp index f4e1a37c80..6ba82bf077 100644 --- a/src/core/SkBlitter.cpp +++ b/src/core/SkBlitter.cpp @@ -857,7 +857,7 @@ SkBlitter* SkBlitter::Choose(const SkBitmap& device, // which check, in case we're being called by a client with a dummy device // (e.g. they have a bounder that always aborts the draw) - if (SkBitmap::kNo_Config == device.config()) { + if (SkBitmap::kNo_Config == device.getConfig()) { SK_PLACEMENT_NEW(blitter, SkNullBlitter, storage, storageSize); return blitter; } @@ -940,7 +940,7 @@ SkBlitter* SkBlitter::Choose(const SkBitmap& device, return blitter; } - switch (device.config()) { + switch (device.getConfig()) { case SkBitmap::kA1_Config: SK_PLACEMENT_NEW_ARGS(blitter, SkA1_Blitter, storage, storageSize, (device, *paint)); diff --git a/src/core/SkBlitter_Sprite.cpp b/src/core/SkBlitter_Sprite.cpp index 9322e20396..db7cc69a78 100644 --- a/src/core/SkBlitter_Sprite.cpp +++ b/src/core/SkBlitter_Sprite.cpp @@ -66,7 +66,7 @@ SkBlitter* SkBlitter::ChooseSprite( const SkBitmap& device, SkSpriteBlitter* blitter; - switch (device.config()) { + switch (device.getConfig()) { case SkBitmap::kRGB_565_Config: blitter = SkSpriteBlitter::ChooseD16(source, paint, storage, storageSize); diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp index 32a6e09ba8..6036186fc2 100644 --- a/src/core/SkCanvas.cpp +++ b/src/core/SkCanvas.cpp @@ -1307,7 +1307,7 @@ bool SkCanvas::updateClipConservativelyUsingBounds(const SkRect& bounds, SkRegio SkRect deviceBounds; SkIRect deviceIBounds; this->getDevice()->getGlobalBounds(&deviceIBounds); - deviceBounds = SkRect::Make(deviceIBounds); + deviceBounds = SkRect::MakeFromIRect(deviceIBounds); this->SkCanvas::save(SkCanvas::kMatrix_SaveFlag); // set the clip in device space this->SkCanvas::setMatrix(SkMatrix::I()); diff --git a/src/core/SkDraw.cpp b/src/core/SkDraw.cpp index df73f7ae37..d2d618090e 100644 --- a/src/core/SkDraw.cpp +++ b/src/core/SkDraw.cpp @@ -1156,7 +1156,7 @@ static bool just_translate(const SkMatrix& matrix, const SkBitmap& bitmap) { void SkDraw::drawBitmapAsMask(const SkBitmap& bitmap, const SkPaint& paint) const { - SkASSERT(bitmap.config() == SkBitmap::kA8_Config); + SkASSERT(bitmap.getConfig() == SkBitmap::kA8_Config); if (just_translate(*fMatrix, bitmap)) { int ix = SkScalarRound(fMatrix->getTranslateX()); @@ -1264,7 +1264,7 @@ void SkDraw::drawBitmap(const SkBitmap& bitmap, const SkMatrix& prematrix, // nothing to draw if (fRC->isEmpty() || bitmap.width() == 0 || bitmap.height() == 0 || - bitmap.config() == SkBitmap::kNo_Config) { + bitmap.getConfig() == SkBitmap::kNo_Config) { return; } @@ -1290,7 +1290,7 @@ void SkDraw::drawBitmap(const SkBitmap& bitmap, const SkMatrix& prematrix, } } - if (bitmap.config() != SkBitmap::kA8_Config && + if (bitmap.getConfig() != SkBitmap::kA8_Config && just_translate(matrix, bitmap)) { // // It is safe to call lock pixels now, since we know the matrix is @@ -1323,7 +1323,7 @@ void SkDraw::drawBitmap(const SkBitmap& bitmap, const SkMatrix& prematrix, SkDraw draw(*this); draw.fMatrix = &matrix; - if (bitmap.config() == SkBitmap::kA8_Config) { + if (bitmap.getConfig() == SkBitmap::kA8_Config) { draw.drawBitmapAsMask(bitmap, paint); } else { SkAutoBitmapShaderInstall install(bitmap, paint); @@ -1343,7 +1343,7 @@ void SkDraw::drawSprite(const SkBitmap& bitmap, int x, int y, // nothing to draw if (fRC->isEmpty() || bitmap.width() == 0 || bitmap.height() == 0 || - bitmap.config() == SkBitmap::kNo_Config) { + bitmap.getConfig() == SkBitmap::kNo_Config) { return; } diff --git a/src/core/SkMipMap.cpp b/src/core/SkMipMap.cpp index 4888b574e4..0673c7e093 100644 --- a/src/core/SkMipMap.cpp +++ b/src/core/SkMipMap.cpp @@ -129,7 +129,7 @@ SkMipMap::Level* SkMipMap::AllocLevels(int levelCount, size_t pixelSize) { SkMipMap* SkMipMap::Build(const SkBitmap& src) { void (*proc)(SkBitmap* dst, int x, int y, const SkBitmap& src); - const SkBitmap::Config config = src.config(); + const SkBitmap::Config config = src.getConfig(); switch (config) { case SkBitmap::kARGB_8888_Config: proc = downsampleby2_proc32; diff --git a/src/core/SkOrderedWriteBuffer.cpp b/src/core/SkOrderedWriteBuffer.cpp index 25ca769168..1c15e43ef0 100644 --- a/src/core/SkOrderedWriteBuffer.cpp +++ b/src/core/SkOrderedWriteBuffer.cpp @@ -310,10 +310,10 @@ void SkOrderedWriteBuffer::writeFlattenable(const SkFlattenable* flattenable) { // make room for the size of the flattened object (void)fWriter.reserve(sizeof(uint32_t)); // record the current size, so we can subtract after the object writes. - uint32_t offset = fWriter.bytesWritten(); + uint32_t offset = fWriter.size(); // now flatten the object flattenObject(flattenable, *this); - uint32_t objSize = fWriter.bytesWritten() - offset; + uint32_t objSize = fWriter.size() - offset; // record the obj's size *fWriter.peek32(offset - sizeof(uint32_t)) = objSize; } diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index 94dd6518af..83f481a91b 100644 --- a/src/core/SkPath.cpp +++ b/src/core/SkPath.cpp @@ -97,8 +97,7 @@ public: } ~SkAutoPathBoundsUpdate() { - fPath->setConvexity(fDegenerate ? SkPath::kConvex_Convexity - : SkPath::kUnknown_Convexity); + fPath->setIsConvex(fDegenerate); if (fEmpty || fHasValidBounds) { fPath->setBounds(fRect); } diff --git a/src/core/SkPicturePlayback.cpp b/src/core/SkPicturePlayback.cpp index f2d959d3d6..4ce29c8dbb 100644 --- a/src/core/SkPicturePlayback.cpp +++ b/src/core/SkPicturePlayback.cpp @@ -66,10 +66,10 @@ SkPicturePlayback::SkPicturePlayback(const SkPictureRecord& record, bool deepCop record.dumpPaints(); #endif - record.validate(record.writeStream().bytesWritten(), 0); + record.validate(record.writeStream().size(), 0); const SkWriter32& writer = record.writeStream(); init(); - if (writer.bytesWritten() == 0) { + if (writer.size() == 0) { fOpData = SkData::NewEmpty(); return; } @@ -85,7 +85,7 @@ SkPicturePlayback::SkPicturePlayback(const SkPictureRecord& record, bool deepCop } { - size_t size = writer.bytesWritten(); + size_t size = writer.size(); void* buffer = sk_malloc_throw(size); writer.flatten(buffer); SkASSERT(!fOpData); diff --git a/src/core/SkPictureRecord.cpp b/src/core/SkPictureRecord.cpp index c59dfe3d71..f7a7633faa 100644 --- a/src/core/SkPictureRecord.cpp +++ b/src/core/SkPictureRecord.cpp @@ -146,7 +146,7 @@ SkBaseDevice* SkPictureRecord::setDevice(SkBaseDevice* device) { int SkPictureRecord::save(SaveFlags flags) { // record the offset to us, making it non-positive to distinguish a save // from a clip entry. - fRestoreOffsetStack.push(-(int32_t)fWriter.bytesWritten()); + fRestoreOffsetStack.push(-(int32_t)fWriter.size()); // op + flags uint32_t size = kSaveSize; @@ -161,7 +161,7 @@ int SkPictureRecord::saveLayer(const SkRect* bounds, const SkPaint* paint, SaveFlags flags) { // record the offset to us, making it non-positive to distinguish a save // from a clip entry. - fRestoreOffsetStack.push(-(int32_t)fWriter.bytesWritten()); + fRestoreOffsetStack.push(-(int32_t)fWriter.size()); // op + bool for 'bounds' uint32_t size = 2 * kUInt32Size; @@ -175,7 +175,7 @@ int SkPictureRecord::saveLayer(const SkRect* bounds, const SkPaint* paint, size_t initialOffset = this->addDraw(SAVE_LAYER, &size); addRectPtr(bounds); - SkASSERT(initialOffset+getPaintOffset(SAVE_LAYER, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(SAVE_LAYER, size) == fWriter.size()); addPaintPtr(paint); addInt(flags); @@ -248,19 +248,19 @@ struct CommandInfo { */ static bool match(SkWriter32* writer, uint32_t offset, int* pattern, CommandInfo* result, int numCommands) { - SkASSERT(offset < writer->bytesWritten()); + SkASSERT(offset < writer->size()); uint32_t curOffset = offset; uint32_t curSize = 0; int numMatched; - for (numMatched = 0; numMatched < numCommands && curOffset < writer->bytesWritten(); ++numMatched) { + for (numMatched = 0; numMatched < numCommands && curOffset < writer->size(); ++numMatched) { DrawType op = peek_op_and_size(writer, curOffset, &curSize); - while (NOOP == op && curOffset < writer->bytesWritten()) { + while (NOOP == op && curOffset < writer->size()) { curOffset += curSize; op = peek_op_and_size(writer, curOffset, &curSize); } - if (curOffset >= writer->bytesWritten()) { + if (curOffset >= writer->size()) { return false; // ran out of byte stream } @@ -285,7 +285,7 @@ static bool match(SkWriter32* writer, uint32_t offset, } curOffset += curSize; - if (curOffset < writer->bytesWritten()) { + if (curOffset < writer->size()) { // Something else between the last command and the end of the stream return false; } @@ -464,7 +464,7 @@ static bool collapse_save_clip_restore(SkWriter32* writer, int32_t offset, gCollapseCalls += 1; #endif - int32_t restoreOffset = (int32_t)writer->bytesWritten(); + int32_t restoreOffset = (int32_t)writer->size(); // back up to the save block while (offset > 0) { @@ -585,7 +585,7 @@ void SkPictureRecord::restore() { if ((*gPictureRecordOpts[opt].fProc)(&fWriter, fRestoreOffsetStack.top(), &fPaints)) { // Some optimization fired so don't add the RESTORE size = 0; - initialOffset = fWriter.bytesWritten(); + initialOffset = fWriter.size(); apply_optimization_to_bbh(gPictureRecordOpts[opt].fType, fStateTree, fBoundingHierarchy); break; @@ -596,7 +596,7 @@ void SkPictureRecord::restore() { if ((fRecordFlags & SkPicture::kDisableRecordOptimizations_RecordingFlag) || SK_ARRAY_COUNT(gPictureRecordOpts) == opt) { // No optimization fired so add the RESTORE - fillRestoreOffsetPlaceholdersForCurrentStackLevel((uint32_t)fWriter.bytesWritten()); + fillRestoreOffsetPlaceholdersForCurrentStackLevel((uint32_t)fWriter.size()); size = 1 * kUInt32Size; // RESTORE consists solely of 1 op code initialOffset = this->addDraw(RESTORE, &size); } @@ -647,7 +647,7 @@ bool SkPictureRecord::skew(SkScalar sx, SkScalar sy) { } bool SkPictureRecord::concat(const SkMatrix& matrix) { - this->validate(fWriter.bytesWritten(), 0); + this->validate(fWriter.size(), 0); // op + matrix index uint32_t size = 2 * kUInt32Size; size_t initialOffset = this->addDraw(CONCAT, &size); @@ -657,7 +657,7 @@ bool SkPictureRecord::concat(const SkMatrix& matrix) { } void SkPictureRecord::setMatrix(const SkMatrix& matrix) { - this->validate(fWriter.bytesWritten(), 0); + this->validate(fWriter.size(), 0); // op + matrix index uint32_t size = 2 * kUInt32Size; size_t initialOffset = this->addDraw(SET_MATRIX, &size); @@ -734,7 +734,7 @@ void SkPictureRecord::recordRestoreOffsetPlaceholder(SkRegion::Op op) { prevOffset = 0; } - size_t offset = fWriter.bytesWritten(); + size_t offset = fWriter.size(); addInt(prevOffset); fRestoreOffsetStack.top() = offset; } @@ -840,7 +840,7 @@ void SkPictureRecord::drawPaint(const SkPaint& paint) { // op + paint index uint32_t size = 2 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_PAINT, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_PAINT, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_PAINT, size) == fWriter.size()); addPaint(paint); this->validate(initialOffset, size); } @@ -850,7 +850,7 @@ void SkPictureRecord::drawPoints(PointMode mode, size_t count, const SkPoint pts // op + paint index + mode + count + point data uint32_t size = 4 * kUInt32Size + count * sizeof(SkPoint); size_t initialOffset = this->addDraw(DRAW_POINTS, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_POINTS, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_POINTS, size) == fWriter.size()); addPaint(paint); addInt(mode); addInt(count); @@ -862,7 +862,7 @@ void SkPictureRecord::drawOval(const SkRect& oval, const SkPaint& paint) { // op + paint index + rect uint32_t size = 2 * kUInt32Size + sizeof(oval); size_t initialOffset = this->addDraw(DRAW_OVAL, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_OVAL, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_OVAL, size) == fWriter.size()); addPaint(paint); addRect(oval); this->validate(initialOffset, size); @@ -872,7 +872,7 @@ void SkPictureRecord::drawRect(const SkRect& rect, const SkPaint& paint) { // op + paint index + rect uint32_t size = 2 * kUInt32Size + sizeof(rect); size_t initialOffset = this->addDraw(DRAW_RECT, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_RECT, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_RECT, size) == fWriter.size()); addPaint(paint); addRect(rect); this->validate(initialOffset, size); @@ -888,7 +888,7 @@ void SkPictureRecord::drawRRect(const SkRRect& rrect, const SkPaint& paint) { uint32_t initialOffset, size; size = 2 * kUInt32Size + SkRRect::kSizeInMemory; initialOffset = this->addDraw(DRAW_RRECT, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_RRECT, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_RRECT, size) == fWriter.size()); addPaint(paint); addRRect(rrect); this->validate(initialOffset, size); @@ -899,7 +899,7 @@ void SkPictureRecord::drawPath(const SkPath& path, const SkPaint& paint) { // op + paint index + path index uint32_t size = 3 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_PATH, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_PATH, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_PATH, size) == fWriter.size()); addPaint(paint); addPath(path); this->validate(initialOffset, size); @@ -910,7 +910,7 @@ void SkPictureRecord::drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar // op + paint index + bitmap index + left + top uint32_t size = 3 * kUInt32Size + 2 * sizeof(SkScalar); size_t initialOffset = this->addDraw(DRAW_BITMAP, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP, size) == fWriter.size()); addPaintPtr(paint); addBitmap(bitmap); addScalar(left); @@ -929,7 +929,7 @@ void SkPictureRecord::drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* size += sizeof(dst); // + rect size_t initialOffset = this->addDraw(DRAW_BITMAP_RECT_TO_RECT, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_RECT_TO_RECT, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_RECT_TO_RECT, size) == fWriter.size()); addPaintPtr(paint); addBitmap(bitmap); addRectPtr(src); // may be null @@ -943,7 +943,7 @@ void SkPictureRecord::drawBitmapMatrix(const SkBitmap& bitmap, const SkMatrix& m // id + paint index + bitmap index + matrix index uint32_t size = 4 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_BITMAP_MATRIX, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_MATRIX, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_MATRIX, size) == fWriter.size()); addPaintPtr(paint); addBitmap(bitmap); addMatrix(matrix); @@ -955,7 +955,7 @@ void SkPictureRecord::drawBitmapNine(const SkBitmap& bitmap, const SkIRect& cent // op + paint index + bitmap id + center + dst rect uint32_t size = 3 * kUInt32Size + sizeof(center) + sizeof(dst); size_t initialOffset = this->addDraw(DRAW_BITMAP_NINE, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_NINE, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_BITMAP_NINE, size) == fWriter.size()); addPaintPtr(paint); addBitmap(bitmap); addIRect(center); @@ -968,7 +968,7 @@ void SkPictureRecord::drawSprite(const SkBitmap& bitmap, int left, int top, // op + paint index + bitmap index + left + top uint32_t size = 5 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_SPRITE, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_SPRITE, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_SPRITE, size) == fWriter.size()); addPaintPtr(paint); addBitmap(bitmap); addInt(left); @@ -1007,7 +1007,7 @@ void SkPictureRecord::drawText(const void* text, size_t byteLength, SkScalar x, DrawType op = fast ? DRAW_TEXT_TOP_BOTTOM : DRAW_TEXT; size_t initialOffset = this->addDraw(op, &size); - SkASSERT(initialOffset+getPaintOffset(op, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(op, size) == fWriter.size()); const SkFlatData* flatPaintData = addPaint(paint); SkASSERT(flatPaintData); addText(text, byteLength); @@ -1073,14 +1073,14 @@ void SkPictureRecord::drawPosText(const void* text, size_t byteLength, op = DRAW_POS_TEXT; } size_t initialOffset = this->addDraw(op, &size); - SkASSERT(initialOffset+getPaintOffset(op, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(op, size) == fWriter.size()); const SkFlatData* flatPaintData = addPaint(paint); SkASSERT(flatPaintData); addText(text, byteLength); addInt(points); #ifdef SK_DEBUG_SIZE - size_t start = fWriter.bytesWritten(); + size_t start = fWriter.size(); #endif if (canUseDrawH) { if (fast) { @@ -1097,7 +1097,7 @@ void SkPictureRecord::drawPosText(const void* text, size_t byteLength, } } #ifdef SK_DEBUG_SIZE - fPointBytes += fWriter.bytesWritten() - start; + fPointBytes += fWriter.size() - start; fPointWrites += points; #endif this->validate(initialOffset, size); @@ -1136,7 +1136,7 @@ void SkPictureRecord::drawPosTextHImpl(const void* text, size_t byteLength, addInt(points); #ifdef SK_DEBUG_SIZE - size_t start = fWriter.bytesWritten(); + size_t start = fWriter.size(); #endif if (fast) { addFontMetricsTopBottom(paint, *flatPaintData, constY, constY); @@ -1144,7 +1144,7 @@ void SkPictureRecord::drawPosTextHImpl(const void* text, size_t byteLength, addScalar(constY); fWriter.writeMul4(xpos, points * sizeof(SkScalar)); #ifdef SK_DEBUG_SIZE - fPointBytes += fWriter.bytesWritten() - start; + fPointBytes += fWriter.size() - start; fPointWrites += points; #endif this->validate(initialOffset, size); @@ -1156,7 +1156,7 @@ void SkPictureRecord::drawTextOnPath(const void* text, size_t byteLength, // op + paint index + length + 'length' worth of data + path index + matrix index uint32_t size = 3 * kUInt32Size + SkAlign4(byteLength) + 2 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_TEXT_ON_PATH, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_TEXT_ON_PATH, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_TEXT_ON_PATH, size) == fWriter.size()); addPaint(paint); addText(text, byteLength); addPath(path); @@ -1202,7 +1202,7 @@ void SkPictureRecord::drawVertices(VertexMode vmode, int vertexCount, } size_t initialOffset = this->addDraw(DRAW_VERTICES, &size); - SkASSERT(initialOffset+getPaintOffset(DRAW_VERTICES, size) == fWriter.bytesWritten()); + SkASSERT(initialOffset+getPaintOffset(DRAW_VERTICES, size) == fWriter.size()); addPaint(paint); addInt(flags); addInt(vmode); @@ -1311,11 +1311,11 @@ void SkPictureRecord::addPicture(SkPicture& picture) { void SkPictureRecord::addPoint(const SkPoint& point) { #ifdef SK_DEBUG_SIZE - size_t start = fWriter.bytesWritten(); + size_t start = fWriter.size(); #endif fWriter.writePoint(point); #ifdef SK_DEBUG_SIZE - fPointBytes += fWriter.bytesWritten() - start; + fPointBytes += fWriter.size() - start; fPointWrites++; #endif } @@ -1330,11 +1330,11 @@ void SkPictureRecord::addPoints(const SkPoint pts[], int count) { void SkPictureRecord::addRect(const SkRect& rect) { #ifdef SK_DEBUG_SIZE - size_t start = fWriter.bytesWritten(); + size_t start = fWriter.size(); #endif fWriter.writeRect(rect); #ifdef SK_DEBUG_SIZE - fRectBytes += fWriter.bytesWritten() - start; + fRectBytes += fWriter.size() - start; fRectWrites++; #endif } @@ -1365,12 +1365,12 @@ void SkPictureRecord::addRegion(const SkRegion& region) { void SkPictureRecord::addText(const void* text, size_t byteLength) { #ifdef SK_DEBUG_SIZE - size_t start = fWriter.bytesWritten(); + size_t start = fWriter.size(); #endif addInt(byteLength); fWriter.writePad(text, byteLength); #ifdef SK_DEBUG_SIZE - fTextBytes += fWriter.bytesWritten() - start; + fTextBytes += fWriter.size() - start; fTextWrites++; #endif } diff --git a/src/core/SkPictureRecord.h b/src/core/SkPictureRecord.h index da79fc1bbb..d80182cf8c 100644 --- a/src/core/SkPictureRecord.h +++ b/src/core/SkPictureRecord.h @@ -131,7 +131,7 @@ private: * operates in this manner. */ size_t addDraw(DrawType drawType, uint32_t* size) { - size_t offset = fWriter.bytesWritten(); + size_t offset = fWriter.size(); this->predrawNotify(); @@ -212,7 +212,7 @@ private: #else public: void validate(size_t initialOffset, uint32_t size) const { - SkASSERT(fWriter.bytesWritten() == initialOffset + size); + SkASSERT(fWriter.size() == initialOffset + size); } #endif diff --git a/src/core/SkShader.cpp b/src/core/SkShader.cpp index 8f6bfb5d80..32f3df4d3c 100644 --- a/src/core/SkShader.cpp +++ b/src/core/SkShader.cpp @@ -51,7 +51,7 @@ bool SkShader::setContext(const SkBitmap& device, const SkMatrix* m = &matrix; SkMatrix total; - fDeviceConfig = SkToU8(device.config()); + fDeviceConfig = SkToU8(device.getConfig()); fPaintAlpha = paint.getAlpha(); if (this->hasLocalMatrix()) { total.setConcat(matrix, this->getLocalMatrix()); diff --git a/src/core/SkSpriteBlitter_ARGB32.cpp b/src/core/SkSpriteBlitter_ARGB32.cpp index 8255336c65..255ef26d40 100644 --- a/src/core/SkSpriteBlitter_ARGB32.cpp +++ b/src/core/SkSpriteBlitter_ARGB32.cpp @@ -277,7 +277,7 @@ SkSpriteBlitter* SkSpriteBlitter::ChooseD32(const SkBitmap& source, SkColorFilter* filter = paint.getColorFilter(); SkSpriteBlitter* blitter = NULL; - switch (source.config()) { + switch (source.getConfig()) { case SkBitmap::kARGB_4444_Config: if (alpha != 0xFF) { return NULL; // we only have opaque sprites diff --git a/src/core/SkSpriteBlitter_RGB16.cpp b/src/core/SkSpriteBlitter_RGB16.cpp index 2bce41e993..8cef76719e 100644 --- a/src/core/SkSpriteBlitter_RGB16.cpp +++ b/src/core/SkSpriteBlitter_RGB16.cpp @@ -324,7 +324,7 @@ SkSpriteBlitter* SkSpriteBlitter::ChooseD16(const SkBitmap& source, SkSpriteBlitter* blitter = NULL; unsigned alpha = paint.getAlpha(); - switch (source.config()) { + switch (source.getConfig()) { case SkBitmap::kARGB_8888_Config: SK_PLACEMENT_NEW_ARGS(blitter, Sprite_D16_S32_BlitRowProc, storage, storageSize, (source)); diff --git a/src/effects/SkBitmapAlphaThresholdShader.cpp b/src/effects/SkBitmapAlphaThresholdShader.cpp index 226b3173e1..1b54425548 100644 --- a/src/effects/SkBitmapAlphaThresholdShader.cpp +++ b/src/effects/SkBitmapAlphaThresholdShader.cpp @@ -249,7 +249,7 @@ GrEffectRef* BATShader::asNewEffect(GrContext* context, const SkPaint& paint) co context->setMatrix(matrix); while (!iter.done()) { - SkRect rect = SkRect::Make(iter.rect()); + SkRect rect = SkRect::MakeFromIRect(iter.rect()); context->drawRect(grPaint, rect); iter.next(); } diff --git a/src/effects/SkBlurMask.h b/src/effects/SkBlurMask.h index e0b8d54ce1..061cfdec52 100644 --- a/src/effects/SkBlurMask.h +++ b/src/effects/SkBlurMask.h @@ -44,19 +44,17 @@ public: Style style, SkIPoint* margin = NULL); - SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based static bool BlurRect(SkMask *dst, const SkRect &src, SkScalar radius, Style style, SkIPoint *margin = NULL, SkMask::CreateMode createMode = SkMask::kComputeBoundsAndRenderImage_CreateMode); - - SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based static bool Blur(SkMask* dst, const SkMask& src, SkScalar radius, Style style, Quality quality, SkIPoint* margin = NULL); - - SK_ATTR_DEPRECATED("use sigma version") + // DEPRECATED - radius-based static bool BlurGroundTruth(SkMask* dst, const SkMask& src, SkScalar radius, Style style, SkIPoint* margin = NULL); diff --git a/src/effects/SkBlurMaskFilter.cpp b/src/effects/SkBlurMaskFilter.cpp index 5b72a6c800..6a5a39778c 100644 --- a/src/effects/SkBlurMaskFilter.cpp +++ b/src/effects/SkBlurMaskFilter.cpp @@ -386,7 +386,7 @@ bool SkBlurMaskFilterImpl::canFilterMaskGPU(const SkRect& srcBounds, float sigma3 = 3 * SkScalarToFloat(xformedSigma); - SkRect clipRect = SkRect::Make(clipBounds); + SkRect clipRect = SkRect::MakeFromIRect(clipBounds); SkRect srcRect(srcBounds); // Outset srcRect and clipRect by 3 * sigma, to compute affected blur area. diff --git a/src/effects/SkMorphologyImageFilter.cpp b/src/effects/SkMorphologyImageFilter.cpp index 152ec560bd..52c01fe0d6 100644 --- a/src/effects/SkMorphologyImageFilter.cpp +++ b/src/effects/SkMorphologyImageFilter.cpp @@ -501,7 +501,7 @@ void apply_morphology_pass(GrContext* context, direction, radius, morphType))->unref(); - context->drawRectToRect(paint, SkRect::Make(dstRect), SkRect::Make(srcRect)); + context->drawRectToRect(paint, SkRect::MakeFromIRect(dstRect), SkRect::MakeFromIRect(srcRect)); } bool apply_morphology(const SkBitmap& input, diff --git a/src/effects/SkTestImageFilters.cpp b/src/effects/SkTestImageFilters.cpp index 289113133c..d0e21a3920 100755 --- a/src/effects/SkTestImageFilters.cpp +++ b/src/effects/SkTestImageFilters.cpp @@ -48,7 +48,7 @@ bool SkDownSampleImageFilter::onFilterImage(Proxy* proxy, const SkBitmap& src, OwnDeviceCanvas canvas(dev); SkPaint paint; - paint.setFilterLevel(SkPaint::kLow_FilterLevel); + paint.setFilterBitmap(true); canvas.scale(scale, scale); canvas.drawBitmap(src, 0, 0, &paint); tmp = dev->accessBitmap(false); diff --git a/src/effects/SkTransparentShader.cpp b/src/effects/SkTransparentShader.cpp index 970e74faa8..0274481623 100644 --- a/src/effects/SkTransparentShader.cpp +++ b/src/effects/SkTransparentShader.cpp @@ -23,7 +23,7 @@ bool SkTransparentShader::setContext(const SkBitmap& device, uint32_t SkTransparentShader::getFlags() { uint32_t flags = this->INHERITED::getFlags(); - switch (fDevice->config()) { + switch (fDevice->getConfig()) { case SkBitmap::kRGB_565_Config: flags |= kHasSpan16_Flag; if (fAlpha == 255) @@ -42,7 +42,7 @@ uint32_t SkTransparentShader::getFlags() { void SkTransparentShader::shadeSpan(int x, int y, SkPMColor span[], int count) { unsigned scale = SkAlpha255To256(fAlpha); - switch (fDevice->config()) { + switch (fDevice->getConfig()) { case SkBitmap::kARGB_8888_Config: if (scale == 256) { SkPMColor* src = fDevice->getAddr32(x, y); @@ -103,7 +103,7 @@ void SkTransparentShader::shadeSpan(int x, int y, SkPMColor span[], int count) { } void SkTransparentShader::shadeSpan16(int x, int y, uint16_t span[], int count) { - SkASSERT(fDevice->config() == SkBitmap::kRGB_565_Config); + SkASSERT(fDevice->getConfig() == SkBitmap::kRGB_565_Config); uint16_t* src = fDevice->getAddr16(x, y); if (src != span) { diff --git a/src/gpu/GrClipMaskManager.cpp b/src/gpu/GrClipMaskManager.cpp index e2597dfad8..c8713b63ad 100644 --- a/src/gpu/GrClipMaskManager.cpp +++ b/src/gpu/GrClipMaskManager.cpp @@ -366,7 +366,7 @@ void GrClipMaskManager::mergeMask(GrTexture* dstMask, GrTextureDomainEffect::MakeTexelDomain(srcMask, srcBound), GrTextureDomainEffect::kDecal_WrapMode, GrTextureParams::kNone_FilterMode))->unref(); - fGpu->drawSimpleRect(SkRect::Make(dstBound), NULL); + fGpu->drawSimpleRect(SkRect::MakeFromIRect(dstBound), NULL); } // get a texture to act as a temporary buffer for AA clip boolean operations @@ -722,7 +722,7 @@ bool GrClipMaskManager::createStencilClipMask(InitialState initialState, SET_RANDOM_COLOR // The view matrix is setup to do clip space -> stencil space translation, so // draw rect in clip space. - fGpu->drawSimpleRect(SkRect::Make(clipSpaceIBounds), NULL); + fGpu->drawSimpleRect(SkRect::MakeFromIRect(clipSpaceIBounds), NULL); } } } @@ -961,7 +961,7 @@ GrTexture* GrClipMaskManager::createSoftwareClipMask(int32_t clipStackGenID, // but leave the pixels inside the geometry alone. For reverse difference we invert all // the pixels before clearing the ones outside the geometry. if (SkRegion::kReverseDifference_Op == op) { - SkRect temp = SkRect::Make(clipSpaceIBounds); + SkRect temp = SkRect::MakeFromIRect(clipSpaceIBounds); // invert the entire scene helper.draw(temp, SkRegion::kXOR_Op, false, 0xFF); } diff --git a/src/gpu/GrDrawTarget.h b/src/gpu/GrDrawTarget.h index 6a1cc02133..c75bba2a10 100644 --- a/src/gpu/GrDrawTarget.h +++ b/src/gpu/GrDrawTarget.h @@ -362,7 +362,7 @@ public: this->drawRect(rect, matrix, NULL, NULL); } void drawSimpleRect(const SkIRect& irect, const SkMatrix* matrix = NULL) { - SkRect rect = SkRect::Make(irect); + SkRect rect = SkRect::MakeFromIRect(irect); this->drawRect(rect, matrix, NULL, NULL); } diff --git a/src/gpu/GrInOrderDrawBuffer.cpp b/src/gpu/GrInOrderDrawBuffer.cpp index a299512538..e426de358a 100644 --- a/src/gpu/GrInOrderDrawBuffer.cpp +++ b/src/gpu/GrInOrderDrawBuffer.cpp @@ -213,7 +213,7 @@ bool GrInOrderDrawBuffer::quickInsideClip(const SkRect& devBounds) { // free via the viewport. We don't want to think that clipping must be enabled in this // case. So we extend the clip outward from the edge to avoid these false negatives. fClipProxyState = kValid_ClipProxyState; - fClipProxy = SkRect::Make(rect); + fClipProxy = SkRect::MakeFromIRect(rect); if (fClipProxy.fLeft <= 0) { fClipProxy.fLeft = SK_ScalarMin; diff --git a/src/gpu/GrReducedClip.cpp b/src/gpu/GrReducedClip.cpp index a5f4519ea7..2b6583e3e3 100644 --- a/src/gpu/GrReducedClip.cpp +++ b/src/gpu/GrReducedClip.cpp @@ -51,7 +51,7 @@ void ReduceClipStack(const SkClipStack& stack, const SkIRect* bounds = &queryBounds; - SkRect scalarQueryBounds = SkRect::Make(queryBounds); + SkRect scalarQueryBounds = SkRect::MakeFromIRect(queryBounds); if (iior) { SkASSERT(SkClipStack::kNormal_BoundsType == stackBoundsType); @@ -67,7 +67,7 @@ void ReduceClipStack(const SkClipStack& stack, } else if (isectRect.intersect(stackBounds, scalarQueryBounds)) { if (NULL != tighterBounds) { isectRect.roundOut(tighterBounds); - SkRect scalarTighterBounds = SkRect::Make(*tighterBounds); + SkRect scalarTighterBounds = SkRect::MakeFromIRect(*tighterBounds); if (scalarTighterBounds == isectRect) { // the round-out didn't add any area outside the clip rect. *requiresAA = false; @@ -119,7 +119,7 @@ void ReduceClipStack(const SkClipStack& stack, } } - SkRect scalarBounds = SkRect::Make(*bounds); + SkRect scalarBounds = SkRect::MakeFromIRect(*bounds); // Now that we have determined the bounds to use and filtered out the trivial cases, call the // helper that actually walks the stack. diff --git a/src/gpu/SkGpuDevice.cpp b/src/gpu/SkGpuDevice.cpp index f47a91bf50..730dad620c 100644 --- a/src/gpu/SkGpuDevice.cpp +++ b/src/gpu/SkGpuDevice.cpp @@ -53,8 +53,8 @@ /////////////////////////////////////////////////////////////////////////////// -#define CHECK_FOR_ANNOTATION(paint) \ - do { if (paint.getAnnotation()) { return; } } while (0) +#define CHECK_FOR_NODRAW_ANNOTATION(paint) \ + do { if (paint.isNoDrawAnnotation()) { return; } } while (0) /////////////////////////////////////////////////////////////////////////////// @@ -572,7 +572,7 @@ static const GrPrimitiveType gPointMode2PrimtiveType[] = { void SkGpuDevice::drawPoints(const SkDraw& draw, SkCanvas::PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); CHECK_SHOULD_DRAW(draw, false); SkScalar width = paint.getStrokeWidth(); @@ -606,7 +606,7 @@ void SkGpuDevice::drawPoints(const SkDraw& draw, SkCanvas::PointMode mode, void SkGpuDevice::drawRect(const SkDraw& draw, const SkRect& rect, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); CHECK_SHOULD_DRAW(draw, false); bool doStroke = paint.getStyle() != SkPaint::kFill_Style; @@ -662,7 +662,7 @@ void SkGpuDevice::drawRect(const SkDraw& draw, const SkRect& rect, void SkGpuDevice::drawRRect(const SkDraw& draw, const SkRRect& rect, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); CHECK_SHOULD_DRAW(draw, false); bool usePath = !rect.isSimple(); @@ -695,7 +695,7 @@ void SkGpuDevice::drawRRect(const SkDraw& draw, const SkRRect& rect, void SkGpuDevice::drawOval(const SkDraw& draw, const SkRect& oval, const SkPaint& paint) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); CHECK_SHOULD_DRAW(draw, false); bool usePath = false; @@ -787,7 +787,7 @@ bool draw_with_mask_filter(GrContext* context, const SkPath& devPath, texture->writePixels(0, 0, desc.fWidth, desc.fHeight, desc.fConfig, dstM.fImage, dstM.fRowBytes); - SkRect maskRect = SkRect::Make(dstM.fBounds); + SkRect maskRect = SkRect::MakeFromIRect(dstM.fBounds); return draw_mask(context, maskRect, grp, texture); } @@ -860,7 +860,7 @@ SkBitmap wrap_texture(GrTexture* texture) { void SkGpuDevice::drawPath(const SkDraw& draw, const SkPath& origSrcPath, const SkPaint& paint, const SkMatrix* prePathMatrix, bool pathIsMutable) { - CHECK_FOR_ANNOTATION(paint); + CHECK_FOR_NODRAW_ANNOTATION(paint); CHECK_SHOULD_DRAW(draw, false); GrPaint grPaint; @@ -1017,7 +1017,7 @@ static void determine_clipped_src_rect(const GrContext* context, clippedSrcIRect->setEmpty(); return; } - SkRect clippedSrcRect = SkRect::Make(*clippedSrcIRect); + SkRect clippedSrcRect = SkRect::MakeFromIRect(*clippedSrcIRect); inv.mapRect(&clippedSrcRect); if (NULL != srcRectPtr) { if (!clippedSrcRect.intersect(*srcRectPtr)) { @@ -1236,7 +1236,7 @@ void SkGpuDevice::drawTiledBitmap(const SkBitmap& bitmap, const SkPaint& paint, SkCanvas::DrawBitmapRectFlags flags, int tileSize) { - SkRect clippedSrcRect = SkRect::Make(clippedSrcIRect); + SkRect clippedSrcRect = SkRect::MakeFromIRect(clippedSrcIRect); int nx = bitmap.width() / tileSize; int ny = bitmap.height() / tileSize; diff --git a/src/images/SkImageDecoder.cpp b/src/images/SkImageDecoder.cpp index 74c80bc73f..89bd0597c8 100644 --- a/src/images/SkImageDecoder.cpp +++ b/src/images/SkImageDecoder.cpp @@ -229,7 +229,7 @@ bool SkImageDecoder::cropBitmap(SkBitmap *dst, SkBitmap *src, int sampleSize, int srcX, int srcY) { int w = width / sampleSize; int h = height / sampleSize; - if (src->config() == SkBitmap::kIndex8_Config) { + if (src->getConfig() == SkBitmap::kIndex8_Config) { // kIndex8 does not allow drawing via an SkCanvas, as is done below. // Instead, use extractSubset. Note that this shares the SkPixelRef and // SkColorTable. @@ -245,7 +245,7 @@ bool SkImageDecoder::cropBitmap(SkBitmap *dst, SkBitmap *src, int sampleSize, } // if the destination has no pixels then we must allocate them. if (dst->isNull()) { - dst->setConfig(src->config(), w, h, 0, src->alphaType()); + dst->setConfig(src->getConfig(), w, h, 0, src->alphaType()); if (!this->allocPixelRef(dst, NULL)) { SkDEBUGF(("failed to allocate pixels needed to crop the bitmap")); diff --git a/src/images/SkImageDecoder_libwebp.cpp b/src/images/SkImageDecoder_libwebp.cpp index ab58aef135..68014dd392 100644 --- a/src/images/SkImageDecoder_libwebp.cpp +++ b/src/images/SkImageDecoder_libwebp.cpp @@ -521,7 +521,7 @@ private: bool SkWEBPImageEncoder::onEncode(SkWStream* stream, const SkBitmap& bm, int quality) { - const SkBitmap::Config config = bm.config(); + const SkBitmap::Config config = bm.getConfig(); const ScanlineImporter scanline_import = ChooseImporter(config); if (NULL == scanline_import) { return false; diff --git a/src/images/SkImageEncoder_argb.cpp b/src/images/SkImageEncoder_argb.cpp index 97b741b8bf..5abc23ce6f 100644 --- a/src/images/SkImageEncoder_argb.cpp +++ b/src/images/SkImageEncoder_argb.cpp @@ -87,7 +87,7 @@ static ScanlineImporter ChooseImporter(const SkBitmap::Config& config) { } bool SkARGBImageEncoder::onEncode(SkWStream* stream, const SkBitmap& bitmap, int) { - const SkBitmap::Config config = bitmap.config(); + const SkBitmap::Config config = bitmap.getConfig(); const ScanlineImporter scanline_import = ChooseImporter(config); if (NULL == scanline_import) { return false; diff --git a/src/pdf/SkPDFDevice.cpp b/src/pdf/SkPDFDevice.cpp index 77aa7a353c..d8e7b1394a 100644 --- a/src/pdf/SkPDFDevice.cpp +++ b/src/pdf/SkPDFDevice.cpp @@ -1607,13 +1607,13 @@ bool SkPDFDevice::handleRectAnnotation(const SkRect& r, const SkMatrix& matrix, SkData* urlData = annotationInfo->find(SkAnnotationKeys::URL_Key()); if (urlData) { handleLinkToURL(urlData, r, matrix); - return p.getAnnotation() != NULL; + return p.isNoDrawAnnotation(); } SkData* linkToName = annotationInfo->find( SkAnnotationKeys::Link_Named_Dest_Key()); if (linkToName) { handleLinkToNamedDest(linkToName, r, matrix); - return p.getAnnotation() != NULL; + return p.isNoDrawAnnotation(); } return false; } @@ -1631,7 +1631,7 @@ bool SkPDFDevice::handlePointAnnotation(const SkPoint* points, size_t count, for (size_t i = 0; i < count; i++) { defineNamedDestination(nameData, points[i], matrix); } - return paint.getAnnotation() != NULL; + return paint.isNoDrawAnnotation(); } return false; } diff --git a/src/pdf/SkPDFImage.cpp b/src/pdf/SkPDFImage.cpp index c3dc396fc4..ed204550e7 100644 --- a/src/pdf/SkPDFImage.cpp +++ b/src/pdf/SkPDFImage.cpp @@ -27,7 +27,7 @@ static bool skip_compression(SkPDFCatalog* catalog) { static size_t get_uncompressed_size(const SkBitmap& bitmap, const SkIRect& srcRect) { - switch (bitmap.config()) { + switch (bitmap.getConfig()) { case SkBitmap::kIndex8_Config: return srcRect.width() * srcRect.height(); case SkBitmap::kARGB_4444_Config: @@ -491,7 +491,7 @@ static SkBitmap unpremultiply_bitmap(const SkBitmap& bitmap, SkPDFImage* SkPDFImage::CreateImage(const SkBitmap& bitmap, const SkIRect& srcRect, SkPicture::EncodeBitmap encoder) { - if (bitmap.config() == SkBitmap::kNo_Config) { + if (bitmap.getConfig() == SkBitmap::kNo_Config) { return NULL; } @@ -569,7 +569,7 @@ SkPDFImage::SkPDFImage(SkStream* stream, fStreamValid = false; } - SkBitmap::Config config = fBitmap.config(); + SkBitmap::Config config = fBitmap.getConfig(); insertName("Type", "XObject"); insertName("Subtype", "Image"); diff --git a/src/pipe/SkGPipeWrite.cpp b/src/pipe/SkGPipeWrite.cpp index b61de1c912..50043c6a2d 100644 --- a/src/pipe/SkGPipeWrite.cpp +++ b/src/pipe/SkGPipeWrite.cpp @@ -294,7 +294,7 @@ private: inline void doNotify() { if (!fDone) { - size_t bytes = fWriter.bytesWritten() - fBytesNotified; + size_t bytes = fWriter.size() - fBytesNotified; if (bytes > 0) { fController->notifyWritten(bytes); fBytesNotified += bytes; @@ -467,7 +467,7 @@ bool SkGPipeCanvas::needOpBytes(size_t needed) { } needed += 4; // size of DrawOp atom - if (fWriter.bytesWritten() + needed > fBlockSize) { + if (fWriter.size() + needed > fBlockSize) { // Before we wipe out any data that has already been written, read it // out. this->doNotify(); diff --git a/src/ports/SkImageDecoder_CG.cpp b/src/ports/SkImageDecoder_CG.cpp index 9b93f81112..3f850b544d 100644 --- a/src/ports/SkImageDecoder_CG.cpp +++ b/src/ports/SkImageDecoder_CG.cpp @@ -219,7 +219,7 @@ bool SkImageEncoder_CG::onEncode(SkWStream* stream, const SkBitmap& bm, // format. // <Error>: CGImageDestinationFinalize image destination does not have enough images // So instead we copy to 8888. - if (bm.config() == SkBitmap::kARGB_4444_Config) { + if (bm.getConfig() == SkBitmap::kARGB_4444_Config) { bm.copyTo(&bitmap8888, SkBitmap::kARGB_8888_Config); bmPtr = &bitmap8888; } diff --git a/src/utils/SkCanvasStateUtils.cpp b/src/utils/SkCanvasStateUtils.cpp index 9c7d8fab3a..c62bc04e11 100644 --- a/src/utils/SkCanvasStateUtils.cpp +++ b/src/utils/SkCanvasStateUtils.cpp @@ -171,7 +171,7 @@ static void setup_MC_state(SkMCState* state, const SkMatrix& matrix, const SkReg } // allocate memory for the clip then and copy them to the struct - state->clipRects = (ClipRect*) sk_malloc_throw(clipWriter.bytesWritten()); + state->clipRects = (ClipRect*) sk_malloc_throw(clipWriter.size()); clipWriter.flatten(state->clipRects); } @@ -239,9 +239,9 @@ SkCanvasState* SkCanvasStateUtils::CaptureCanvasState(SkCanvas* canvas) { } // allocate memory for the layers and then and copy them to the struct - SkASSERT(layerWriter.bytesWritten() == layerCount * sizeof(SkCanvasLayerState)); + SkASSERT(layerWriter.size() == layerCount * sizeof(SkCanvasLayerState)); canvasState->layerCount = layerCount; - canvasState->layers = (SkCanvasLayerState*) sk_malloc_throw(layerWriter.bytesWritten()); + canvasState->layers = (SkCanvasLayerState*) sk_malloc_throw(layerWriter.size()); layerWriter.flatten(canvasState->layers); // for now, just ignore any client supplied DrawFilter. diff --git a/src/utils/debugger/SkDebugCanvas.h b/src/utils/debugger/SkDebugCanvas.h index 3df31513b4..7d1166286a 100644 --- a/src/utils/debugger/SkDebugCanvas.h +++ b/src/utils/debugger/SkDebugCanvas.h @@ -109,8 +109,8 @@ public: /** Returns the vector of draw commands + DEPRECATED: please use getDrawCommandAt and getSize instead */ - SK_ATTR_DEPRECATED("please use getDrawCommandAt and getSize instead") const SkTDArray<SkDrawCommand*>& getDrawCommands() const; /** diff --git a/src/utils/debugger/SkObjectParser.cpp b/src/utils/debugger/SkObjectParser.cpp index 107e04a258..9dd77c0d73 100644 --- a/src/utils/debugger/SkObjectParser.cpp +++ b/src/utils/debugger/SkObjectParser.cpp @@ -31,7 +31,7 @@ SkString* SkObjectParser::BitmapToString(const SkBitmap& bitmap) { SkASSERT(SkBitmap::kConfigCount == 7); mBitmap->append(" Config: "); - mBitmap->append(gConfigStrings[bitmap.config()]); + mBitmap->append(gConfigStrings[bitmap.getConfig()]); if (bitmap.isOpaque()) { mBitmap->append(" opaque"); diff --git a/src/views/SkWindow.cpp b/src/views/SkWindow.cpp index 32914502d9..d6fe32bd88 100644 --- a/src/views/SkWindow.cpp +++ b/src/views/SkWindow.cpp @@ -117,7 +117,7 @@ void SkWindow::eraseARGB(U8CPU a, U8CPU r, U8CPU g, U8CPU b) void SkWindow::eraseRGB(U8CPU r, U8CPU g, U8CPU b) { - fBitmap.eraseARGB(0xFF, r, g, b); + fBitmap.eraseRGB(r, g, b); } bool SkWindow::handleInval(const SkRect* localR) diff --git a/tests/BitmapCopyTest.cpp b/tests/BitmapCopyTest.cpp index 5cef1eb986..bf0d2f7813 100644 --- a/tests/BitmapCopyTest.cpp +++ b/tests/BitmapCopyTest.cpp @@ -96,7 +96,7 @@ static uint32_t getPixel(int x, int y, const SkBitmap& bm) { SkAutoLockPixels lock(bm); const void* rawAddr = bm.getAddr(x,y); - switch (bm.config()) { + switch (bm.getConfig()) { case SkBitmap::kARGB_8888_Config: memcpy(&val, rawAddr, sizeof(uint32_t)); break; @@ -130,7 +130,7 @@ static void setPixel(int x, int y, uint32_t val, SkBitmap& bm) { SkAutoLockPixels lock(bm); void* rawAddr = bm.getAddr(x,y); - switch (bm.config()) { + switch (bm.getConfig()) { case SkBitmap::kARGB_8888_Config: memcpy(rawAddr, &val, sizeof(uint32_t)); break; @@ -162,7 +162,7 @@ static void setPixel(int x, int y, uint32_t val, SkBitmap& bm) { // Utility to return string containing name of each format, to // simplify diagnostic output. static const char* getSkConfigName(const SkBitmap& bm) { - switch (bm.config()) { + switch (bm.getConfig()) { case SkBitmap::kNo_Config: return "SkBitmap::kNo_Config"; case SkBitmap::kA1_Config: return "SkBitmap::kA1_Config"; case SkBitmap::kA8_Config: return "SkBitmap::kA8_Config"; @@ -458,7 +458,7 @@ static void TestBitmapCopy(skiatest::Reporter* reporter) { srcReady = src.extractSubset(&subset, r); } else { - srcReady = src.copyTo(&subset, src.config()); + srcReady = src.copyTo(&subset, src.getConfig()); } // Not all configurations will generate a valid 'subset'. @@ -469,7 +469,7 @@ static void TestBitmapCopy(skiatest::Reporter* reporter) { // buf to a SkBitmap, but copies are done using the // raw buffer pointer. const size_t bufSize = subH * - SkBitmap::ComputeRowBytes(src.config(), subW) * 2; + SkBitmap::ComputeRowBytes(src.getConfig(), subW) * 2; SkAutoMalloc autoBuf (bufSize); uint8_t* buf = static_cast<uint8_t*>(autoBuf.get()); @@ -496,7 +496,8 @@ static void TestBitmapCopy(skiatest::Reporter* reporter) { memset(buf, 0xFF, bufSize); // Config with stride greater than src but that fits in buf. bufBm.setConfig(gPairs[i].fConfig, subW, subH, - SkBitmap::ComputeRowBytes(subset.config(), subW) * 2); + SkBitmap::ComputeRowBytes(subset.getConfig(), subW) + * 2); bufBm.setPixels(buf); successExpected = false; // Then attempt to copy with a stride that is too large diff --git a/tests/GpuBitmapCopyTest.cpp b/tests/GpuBitmapCopyTest.cpp index 6e1d74dd97..0d9cf2ba85 100644 --- a/tests/GpuBitmapCopyTest.cpp +++ b/tests/GpuBitmapCopyTest.cpp @@ -139,7 +139,7 @@ static void TestGpuBitmapCopy(skiatest::Reporter* reporter, GrContextFactory* fa SkCanvas drawingCanvas(device); SkPaint paint; paint.setColor(SK_ColorRED); - drawingCanvas.drawRect(SkRect::Make(subsetRect), paint); + drawingCanvas.drawRect(SkRect::MakeFromIRect(subsetRect), paint); // Extract a subset. If this succeeds we will test copying the subset. SkBitmap subset; diff --git a/tests/Matrix44Test.cpp b/tests/Matrix44Test.cpp index 51a6e7f950..772d7b2cf3 100644 --- a/tests/Matrix44Test.cpp +++ b/tests/Matrix44Test.cpp @@ -73,7 +73,8 @@ static bool nearly_equal(const SkMatrix44& a, const SkMatrix44& b) { } static bool is_identity(const SkMatrix44& m) { - SkMatrix44 identity(SkMatrix44::kIdentity_Constructor); + SkMatrix44 identity; + identity.reset(); return nearly_equal(m, identity); } @@ -84,7 +85,7 @@ static bool bits_isonly(int value, int mask) { static void test_constructor(skiatest::Reporter* reporter) { // Allocate a matrix on the heap - SkMatrix44* placeholderMatrix = new SkMatrix44(SkMatrix44::kUninitialized_Constructor); + SkMatrix44* placeholderMatrix = new SkMatrix44(); SkAutoTDelete<SkMatrix44> deleteMe(placeholderMatrix); for (int row = 0; row < 4; ++row) { @@ -114,8 +115,7 @@ static void test_constructor(skiatest::Reporter* reporter) { } static void test_translate(skiatest::Reporter* reporter) { - SkMatrix44 mat(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 inverse(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 mat, inverse; mat.setTranslate(0, 0, 0); REPORTER_ASSERT(reporter, bits_isonly(mat.getType(), SkMatrix44::kIdentity_Mask)); @@ -124,9 +124,7 @@ static void test_translate(skiatest::Reporter* reporter) { REPORTER_ASSERT(reporter, mat.invert(&inverse)); REPORTER_ASSERT(reporter, bits_isonly(inverse.getType(), SkMatrix44::kTranslate_Mask)); - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 c(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a, b, c; a.set3x3(1, 2, 3, 4, 5, 6, 7, 8, 9); b.setTranslate(10, 11, 12); @@ -142,8 +140,7 @@ static void test_translate(skiatest::Reporter* reporter) { } static void test_scale(skiatest::Reporter* reporter) { - SkMatrix44 mat(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 inverse(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 mat, inverse; mat.setScale(1, 1, 1); REPORTER_ASSERT(reporter, bits_isonly(mat.getType(), SkMatrix44::kIdentity_Mask)); @@ -152,9 +149,7 @@ static void test_scale(skiatest::Reporter* reporter) { REPORTER_ASSERT(reporter, mat.invert(&inverse)); REPORTER_ASSERT(reporter, bits_isonly(inverse.getType(), SkMatrix44::kScale_Mask)); - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 c(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a, b, c; a.set3x3(1, 2, 3, 4, 5, 6, 7, 8, 9); b.setScale(10, 11, 12); @@ -212,7 +207,7 @@ static void test_map2(skiatest::Reporter* reporter, const SkMatrix44& mat) { } static void test_map2(skiatest::Reporter* reporter) { - SkMatrix44 mat(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 mat; for (size_t i = 0; i < SK_ARRAY_COUNT(gMakeProcs); ++i) { gMakeProcs[i](&mat); @@ -221,7 +216,7 @@ static void test_map2(skiatest::Reporter* reporter) { } static void test_gettype(skiatest::Reporter* reporter) { - SkMatrix44 matrix(SkMatrix44::kIdentity_Constructor); + SkMatrix44 matrix; REPORTER_ASSERT(reporter, matrix.isIdentity()); REPORTER_ASSERT(reporter, SkMatrix44::kIdentity_Mask == matrix.getType()); @@ -256,7 +251,7 @@ static void test_gettype(skiatest::Reporter* reporter) { } static void test_common_angles(skiatest::Reporter* reporter) { - SkMatrix44 rot(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 rot; // Test precision of rotation in common cases int common_angles[] = { 0, 90, -90, 180, -180, 270, -270, 360, -360 }; for (int i = 0; i < 9; ++i) { @@ -269,10 +264,7 @@ static void test_common_angles(skiatest::Reporter* reporter) { static void test_concat(skiatest::Reporter* reporter) { int i; - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 c(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 d(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a, b, c, d; a.setTranslate(10, 10, 10); b.setScale(2, 2, 2); @@ -309,11 +301,11 @@ static void test_concat(skiatest::Reporter* reporter) { } static void test_determinant(skiatest::Reporter* reporter) { - SkMatrix44 a(SkMatrix44::kIdentity_Constructor); + SkMatrix44 a; REPORTER_ASSERT(reporter, nearly_equal_double(1, a.determinant())); a.set(1, 1, 2); REPORTER_ASSERT(reporter, nearly_equal_double(2, a.determinant())); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 b; REPORTER_ASSERT(reporter, a.invert(&b)); REPORTER_ASSERT(reporter, nearly_equal_double(0.5, b.determinant())); SkMatrix44 c = b = a; @@ -334,10 +326,11 @@ static void test_determinant(skiatest::Reporter* reporter) { } static void test_invert(skiatest::Reporter* reporter) { - SkMatrix44 inverse(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 inverse; double inverseData[16]; - SkMatrix44 identity(SkMatrix44::kIdentity_Constructor); + SkMatrix44 identity; + identity.setIdentity(); identity.invert(&inverse); inverse.asRowMajord(inverseData); assert16<double>(reporter, inverseData, @@ -346,7 +339,7 @@ static void test_invert(skiatest::Reporter* reporter) { 0, 0, 1, 0, 0, 0, 0, 1); - SkMatrix44 translation(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 translation; translation.setTranslate(2, 3, 4); translation.invert(&inverse); inverse.asRowMajord(inverseData); @@ -356,7 +349,7 @@ static void test_invert(skiatest::Reporter* reporter) { 0, 0, 1, -4, 0, 0, 0, 1); - SkMatrix44 scale(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 scale; scale.setScale(2, 4, 8); scale.invert(&inverse); inverse.asRowMajord(inverseData); @@ -366,7 +359,7 @@ static void test_invert(skiatest::Reporter* reporter) { 0, 0, 0.125, 0, 0, 0, 0, 1); - SkMatrix44 scaleTranslation(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 scaleTranslation; scaleTranslation.setScale(10, 100, 1000); scaleTranslation.preTranslate(2, 3, 4); scaleTranslation.invert(&inverse); @@ -377,10 +370,10 @@ static void test_invert(skiatest::Reporter* reporter) { 0, 0, 0.001, -4, 0, 0, 0, 1); - SkMatrix44 rotation(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 rotation; rotation.setRotateDegreesAbout(0, 0, 1, 90); rotation.invert(&inverse); - SkMatrix44 expected(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 expected; double expectedInverseRotation[16] = {0, 1, 0, 0, -1, 0, 0, 0, @@ -389,7 +382,7 @@ static void test_invert(skiatest::Reporter* reporter) { expected.setRowMajord(expectedInverseRotation); REPORTER_ASSERT(reporter, nearly_equal(expected, inverse)); - SkMatrix44 affine(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 affine; affine.setRotateDegreesAbout(0, 0, 1, 90); affine.preScale(10, 20, 100); affine.preTranslate(2, 3, 4); @@ -402,7 +395,8 @@ static void test_invert(skiatest::Reporter* reporter) { expected.setRowMajord(expectedInverseAffine); REPORTER_ASSERT(reporter, nearly_equal(expected, inverse)); - SkMatrix44 perspective(SkMatrix44::kIdentity_Constructor); + SkMatrix44 perspective; + perspective.setIdentity(); perspective.setDouble(3, 2, 1.0); perspective.invert(&inverse); double expectedInversePerspective[16] = @@ -413,7 +407,8 @@ static void test_invert(skiatest::Reporter* reporter) { expected.setRowMajord(expectedInversePerspective); REPORTER_ASSERT(reporter, nearly_equal(expected, inverse)); - SkMatrix44 affineAndPerspective(SkMatrix44::kIdentity_Constructor); + SkMatrix44 affineAndPerspective; + affineAndPerspective.setIdentity(); affineAndPerspective.setDouble(3, 2, 1.0); affineAndPerspective.preScale(10, 20, 100); affineAndPerspective.preTranslate(2, 3, 4); @@ -428,8 +423,8 @@ static void test_invert(skiatest::Reporter* reporter) { } static void test_transpose(skiatest::Reporter* reporter) { - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a; + SkMatrix44 b; int i = 0; for (int row = 0; row < 4; ++row) { @@ -444,7 +439,7 @@ static void test_transpose(skiatest::Reporter* reporter) { } static void test_get_set_double(skiatest::Reporter* reporter) { - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a; for (int row = 0; row < 4; ++row) { for (int col = 0; col < 4; ++col) { a.setDouble(row, col, 3.141592653589793); @@ -459,9 +454,7 @@ static void test_get_set_double(skiatest::Reporter* reporter) { } static void test_set_row_col_major(skiatest::Reporter* reporter) { - SkMatrix44 a(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 b(SkMatrix44::kUninitialized_Constructor); - + SkMatrix44 a, b, c, d; for (int row = 0; row < 4; ++row) { for (int col = 0; col < 4; ++col) { a.setDouble(row, col, row * 4 + col); @@ -496,7 +489,7 @@ static void test_3x3_conversion(skiatest::Reporter* reporter) { 5, 6, 0, 8, 0, 0, 1, 0, 13, 14, 0, 16 }; - SkMatrix44 a44(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 a44; a44.setRowMajor(values4x4); SkMatrix a33 = a44; @@ -505,7 +498,7 @@ static void test_3x3_conversion(skiatest::Reporter* reporter) { REPORTER_ASSERT(reporter, expected33 == a33); SkMatrix44 a44flattened = a33; - SkMatrix44 expected44flattened(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 expected44flattened; expected44flattened.setRowMajor(values4x4flattened); REPORTER_ASSERT(reporter, nearly_equal(a44flattened, expected44flattened)); @@ -529,12 +522,9 @@ static void test_3x3_conversion(skiatest::Reporter* reporter) { } static void TestMatrix44(skiatest::Reporter* reporter) { - SkMatrix44 mat(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 inverse(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 iden1(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 iden2(SkMatrix44::kUninitialized_Constructor); - SkMatrix44 rot(SkMatrix44::kUninitialized_Constructor); + SkMatrix44 mat, inverse, iden1, iden2, rot; + mat.reset(); mat.setTranslate(1, 1, 1); mat.invert(&inverse); iden1.setConcat(mat, inverse); diff --git a/tests/PathTest.cpp b/tests/PathTest.cpp index d879ea6e33..6544c0b8d7 100644 --- a/tests/PathTest.cpp +++ b/tests/PathTest.cpp @@ -1768,7 +1768,7 @@ static void write_and_read_back(skiatest::Reporter* reporter, const SkPath& p) { SkWriter32 writer(100); writer.writePath(p); - size_t size = writer.bytesWritten(); + size_t size = writer.size(); SkAutoMalloc storage(size); writer.flatten(storage.get()); SkReader32 reader(storage.get(), size); diff --git a/tests/ShaderImageFilterTest.cpp b/tests/ShaderImageFilterTest.cpp index 49fc6e040b..78aa796caa 100644 --- a/tests/ShaderImageFilterTest.cpp +++ b/tests/ShaderImageFilterTest.cpp @@ -37,8 +37,7 @@ static void test_asShaderMode(skiatest::Reporter* reporter) { SkShader* s = SkGradientShader::CreateRadial( center, radius, colors, pos, SK_ARRAY_COUNT(colors), SkShader::kClamp_TileMode); SkPaint paint; - SkImageFilter::CropRect cr(r); - paint.setImageFilter(SkRectShaderImageFilter::Create(s, &cr))->unref(); + paint.setImageFilter(SkRectShaderImageFilter::Create(s, r))->unref(); canvasFilter.drawRect(r, paint); s->unref(); } diff --git a/tests/Writer32Test.cpp b/tests/Writer32Test.cpp index 15218378ec..8b4aba5ec3 100644 --- a/tests/Writer32Test.cpp +++ b/tests/Writer32Test.cpp @@ -86,7 +86,7 @@ static void test_ptr(skiatest::Reporter* reporter) { writer.writePtr(p1); writer.write8(0x66); - size_t size = writer.bytesWritten(); + size_t size = writer.size(); REPORTER_ASSERT(reporter, 2 * sizeof(void*) + 2 * sizeof(int32_t)); char buffer[32]; @@ -103,14 +103,14 @@ static void test_ptr(skiatest::Reporter* reporter) { static void test1(skiatest::Reporter* reporter, SkWriter32* writer) { const uint32_t data[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 }; for (size_t i = 0; i < SK_ARRAY_COUNT(data); ++i) { - REPORTER_ASSERT(reporter, i*4 == writer->bytesWritten()); + REPORTER_ASSERT(reporter, i*4 == writer->size()); writer->write32(data[i]); uint32_t* addr = writer->peek32(i * 4); REPORTER_ASSERT(reporter, data[i] == *addr); } char buffer[sizeof(data)]; - REPORTER_ASSERT(reporter, sizeof(buffer) == writer->bytesWritten()); + REPORTER_ASSERT(reporter, sizeof(buffer) == writer->size()); writer->flatten(buffer); REPORTER_ASSERT(reporter, !memcmp(data, buffer, sizeof(buffer))); } @@ -124,7 +124,7 @@ static void test2(skiatest::Reporter* reporter, SkWriter32* writer) { len += SkWriter32::WriteStringSize(gStr, i); writer->writeString(gStr, i); } - REPORTER_ASSERT(reporter, writer->bytesWritten() == len); + REPORTER_ASSERT(reporter, writer->size() == len); SkAutoMalloc storage(len); writer->flatten(storage.get()); @@ -167,7 +167,7 @@ static void testWritePad(skiatest::Reporter* reporter, SkWriter32* writer) { } } - uint32_t totalBytes = writer->bytesWritten(); + uint32_t totalBytes = writer->size(); SkAutoMalloc readStorage(totalBytes); writer->flatten(readStorage.get()); diff --git a/tests/XfermodeTest.cpp b/tests/XfermodeTest.cpp index 7d23b8530a..620fbf4c93 100644 --- a/tests/XfermodeTest.cpp +++ b/tests/XfermodeTest.cpp @@ -23,7 +23,7 @@ static void test_asMode(skiatest::Reporter* reporter) { REPORTER_ASSERT(reporter, reportedMode != mode); // test IsMode - REPORTER_ASSERT(reporter, SkXfermode::AsMode(xfer, &reportedMode)); + REPORTER_ASSERT(reporter, SkXfermode::IsMode(xfer, &reportedMode)); REPORTER_ASSERT(reporter, reportedMode == mode); // repeat that test, but with asMode instead @@ -41,7 +41,7 @@ static void test_asMode(skiatest::Reporter* reporter) { SkXfermode::Mode reportedMode = ILLEGAL_MODE; REPORTER_ASSERT(reporter, !bogusXfer->asMode(&reportedMode)); REPORTER_ASSERT(reporter, reportedMode == ILLEGAL_MODE); - REPORTER_ASSERT(reporter, !SkXfermode::AsMode(bogusXfer, &reportedMode)); + REPORTER_ASSERT(reporter, !SkXfermode::IsMode(bogusXfer, &reportedMode)); REPORTER_ASSERT(reporter, reportedMode == ILLEGAL_MODE); bogusXfer->unref(); } |