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
diff --git a/bench/BitmapRectBench.cpp b/bench/BitmapRectBench.cpp
index 6029aeb..104cb24 100644
--- a/bench/BitmapRectBench.cpp
+++ b/bench/BitmapRectBench.cpp
@@ -39,20 +39,18 @@
*/
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 @@
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 @@
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 @@
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 3d371be..7812285 100644
--- a/bench/GameBench.cpp
+++ b/bench/GameBench.cpp
@@ -134,11 +134,11 @@
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 1316997..a4b36d8 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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 432d5be..e11ebcf 100644
--- a/bench/SkBenchmark.cpp
+++ b/bench/SkBenchmark.cpp
@@ -19,7 +19,6 @@
SkBenchmark::SkBenchmark() {
fForceAlpha = 0xFF;
fForceAA = true;
- fForceFilter = false;
fDither = SkTriState::kDefault;
fIsRendering = true;
fOrMask = fClearMask = 0;
@@ -49,8 +48,7 @@
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 4619d8b..03a758c 100644
--- a/bench/TileBench.cpp
+++ b/bench/TileBench.cpp
@@ -85,8 +85,7 @@
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 7eb0aee..c3f80df 100644
--- a/gm/bicubicfilter.cpp
+++ b/gm/bicubicfilter.cpp
@@ -57,7 +57,7 @@
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 9b49a64..d414486 100644
--- a/gm/bigmatrix.cpp
+++ b/gm/bigmatrix.cpp
@@ -80,7 +80,7 @@
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 f0153e4..d74dbf5 100644
--- a/gm/bitmapfilters.cpp
+++ b/gm/bitmapfilters.cpp
@@ -39,7 +39,7 @@
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 4609891..f89cc97 100644
--- a/gm/bitmapmatrix.cpp
+++ b/gm/bitmapmatrix.cpp
@@ -81,7 +81,7 @@
// 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 aaca62b..0d09394 100644
--- a/gm/bleed.cpp
+++ b/gm/bleed.cpp
@@ -91,7 +91,7 @@
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 @@
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 @@
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 @@
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 431f022..070a59c 100644
--- a/gm/blurquickreject.cpp
+++ b/gm/blurquickreject.cpp
@@ -55,7 +55,7 @@
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 c880063..106f6d3 100644
--- a/gm/drawbitmaprect.cpp
+++ b/gm/drawbitmaprect.cpp
@@ -163,7 +163,7 @@
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 ab75dc7..0d342b8 100644
--- a/gm/giantbitmap.cpp
+++ b/gm/giantbitmap.cpp
@@ -110,7 +110,7 @@
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 bc6f13d..551f020 100644
--- a/gm/ninepatchstretch.cpp
+++ b/gm/ninepatchstretch.cpp
@@ -93,7 +93,7 @@
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 a7da436..e760e54 100644
--- a/gm/shadertext2.cpp
+++ b/gm/shadertext2.cpp
@@ -102,7 +102,7 @@
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 36abf9f..fa327c8 100644
--- a/gm/shadertext3.cpp
+++ b/gm/shadertext3.cpp
@@ -74,7 +74,7 @@
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 @@
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 dcfc133..5a5fbbe 100644
--- a/gm/tilemodes.cpp
+++ b/gm/tilemodes.cpp
@@ -47,7 +47,7 @@
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 59b6482..3e22069 100644
--- a/include/core/SkBitmap.h
+++ b/include/core/SkBitmap.h
@@ -96,19 +96,20 @@
*/
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 @@
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 @@
*/
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 @@
*/
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 530b85c..a288f8a 100644
--- a/include/core/SkBitmapDevice.h
+++ b/include/core/SkBitmapDevice.h
@@ -87,7 +87,7 @@
/** 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 d0b8fc9..dc3953e 100644
--- a/include/core/SkCanvas.h
+++ b/include/core/SkCanvas.h
@@ -57,7 +57,7 @@
*/
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 @@
*/
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 @@
// 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 e38e2aa..ed1c8fd 100644
--- a/include/core/SkDevice.h
+++ b/include/core/SkDevice.h
@@ -90,9 +90,8 @@
*/
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 @@
* 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 1244795..6c3a10f 100644
--- a/include/core/SkPaint.h
+++ b/include/core/SkPaint.h
@@ -304,18 +304,18 @@
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 @@
/**
* 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 785eb77..02e3423 100644
--- a/include/core/SkPath.h
+++ b/include/core/SkPath.h
@@ -123,6 +123,7 @@
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 @@
}
/**
+ * 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 b94f730..5eca72f 100644
--- a/include/core/SkPoint.h
+++ b/include/core/SkPoint.h
@@ -353,7 +353,7 @@
/** 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 @@
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 61998f6..0c2358a 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 ab995d7..db40f69 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 98acb1c..d9ac3a6 100644
--- a/include/core/SkRect.h
+++ b/include/core/SkRect.h
@@ -401,7 +401,7 @@
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 9fb1f7b..ba8893e 100644
--- a/include/core/SkWriter32.h
+++ b/include/core/SkWriter32.h
@@ -46,8 +46,7 @@
// 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 3ef2241..5bf8a22 100644
--- a/include/core/SkXfermode.h
+++ b/include/core/SkXfermode.h
@@ -186,7 +186,7 @@
*/
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 013e807..8e0c2dd 100644
--- a/include/effects/SkBlurDrawLooper.h
+++ b/include/effects/SkBlurDrawLooper.h
@@ -38,7 +38,7 @@
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 9b352dc..d590b27 100644
--- a/include/effects/SkBlurMaskFilter.h
+++ b/include/effects/SkBlurMaskFilter.h
@@ -33,7 +33,9 @@
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 @@
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 7210b78..4b5bd6f 100644
--- a/include/effects/SkEmbossMaskFilter.h
+++ b/include/effects/SkEmbossMaskFilter.h
@@ -25,7 +25,7 @@
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 e984e8e..c5f5492 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 @@
/** 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 ea97478..650a583 100644
--- a/include/effects/SkRectShaderImageFilter.h
+++ b/include/effects/SkRectShaderImageFilter.h
@@ -25,9 +25,7 @@
* 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 29f59f2..0862514 100644
--- a/include/gpu/SkGpuDevice.h
+++ b/include/gpu/SkGpuDevice.h
@@ -43,16 +43,16 @@
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 26247a0..9282770 100644
--- a/include/utils/SkMatrix44.h
+++ b/include/utils/SkMatrix44.h
@@ -112,7 +112,7 @@
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 @@
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 @@
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 04d2cdd..f044e70 100644
--- a/samplecode/SampleApp.cpp
+++ b/samplecode/SampleApp.cpp
@@ -505,8 +505,7 @@
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 422ed58..04297e0 100644
--- a/samplecode/SampleBitmapRect.cpp
+++ b/samplecode/SampleBitmapRect.cpp
@@ -133,7 +133,7 @@
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 @@
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 71ab62f..3eb9cdc 100644
--- a/samplecode/SampleFilter.cpp
+++ b/samplecode/SampleFilter.cpp
@@ -59,7 +59,7 @@
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 e982778..f3a8b46 100644
--- a/samplecode/SampleFilter2.cpp
+++ b/samplecode/SampleFilter2.cpp
@@ -76,7 +76,7 @@
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 @@
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 b10fb31..c04292d 100644
--- a/samplecode/SampleFilterFuzz.cpp
+++ b/samplecode/SampleFilterFuzz.cpp
@@ -231,9 +231,9 @@
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 c630e89..3da4b0c 100644
--- a/samplecode/SamplePatch.cpp
+++ b/samplecode/SamplePatch.cpp
@@ -269,7 +269,7 @@
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 3dba782..98e4e72 100644
--- a/samplecode/SampleSlides.cpp
+++ b/samplecode/SampleSlides.cpp
@@ -444,7 +444,7 @@
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 16e7a16..154eaa9 100644
--- a/samplecode/SampleTextureDomain.cpp
+++ b/samplecode/SampleTextureDomain.cpp
@@ -48,7 +48,7 @@
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 c35cf88..eb791c5 100644
--- a/samplecode/SampleTiling.cpp
+++ b/samplecode/SampleTiling.cpp
@@ -52,7 +52,7 @@
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 323c4b5..53db2ea 100644
--- a/samplecode/SampleVertices.cpp
+++ b/samplecode/SampleVertices.cpp
@@ -91,7 +91,7 @@
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 825d048..22d99e4 100644
--- a/src/animator/SkDrawPaint.cpp
+++ b/src/animator/SkDrawPaint.cpp
@@ -220,7 +220,7 @@
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 8e5861b..61a82ce 100644
--- a/src/core/SkBBoxHierarchyRecord.cpp
+++ b/src/core/SkBBoxHierarchyRecord.cpp
@@ -22,7 +22,7 @@
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::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 @@
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 @@
// 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 2349b1c..1e6c69b 100644
--- a/src/core/SkBBoxRecord.cpp
+++ b/src/core/SkBBoxRecord.cpp
@@ -32,7 +32,7 @@
// 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 bff0880..2f1c9a4 100644
--- a/src/core/SkBitmap.cpp
+++ b/src/core/SkBitmap.cpp
@@ -502,11 +502,11 @@
// 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 @@
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 @@
}
} 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 @@
* 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 @@
#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 @@
}
// 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 (*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 @@
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 eac21e2..ff68481 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::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::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::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 a171138..f3428db 100644
--- a/src/core/SkBitmapHeap.cpp
+++ b/src/core/SkBitmapHeap.cpp
@@ -259,7 +259,7 @@
// 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 f4e1a37..6ba82bf 100644
--- a/src/core/SkBlitter.cpp
+++ b/src/core/SkBlitter.cpp
@@ -857,7 +857,7 @@
// 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 @@
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 9322e20..db7cc69 100644
--- a/src/core/SkBlitter_Sprite.cpp
+++ b/src/core/SkBlitter_Sprite.cpp
@@ -66,7 +66,7 @@
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 32a6e09..6036186 100644
--- a/src/core/SkCanvas.cpp
+++ b/src/core/SkCanvas.cpp
@@ -1307,7 +1307,7 @@
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 df73f7a..d2d6180 100644
--- a/src/core/SkDraw.cpp
+++ b/src/core/SkDraw.cpp
@@ -1156,7 +1156,7 @@
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 @@
// 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 @@
}
}
- 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 @@
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 @@
// 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 4888b57..0673c7e 100644
--- a/src/core/SkMipMap.cpp
+++ b/src/core/SkMipMap.cpp
@@ -129,7 +129,7 @@
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 25ca769..1c15e43 100644
--- a/src/core/SkOrderedWriteBuffer.cpp
+++ b/src/core/SkOrderedWriteBuffer.cpp
@@ -310,10 +310,10 @@
// 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 94dd651..83f481a 100644
--- a/src/core/SkPath.cpp
+++ b/src/core/SkPath.cpp
@@ -97,8 +97,7 @@
}
~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 f2d959d..4ce29c8 100644
--- a/src/core/SkPicturePlayback.cpp
+++ b/src/core/SkPicturePlayback.cpp
@@ -66,10 +66,10 @@
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 @@
}
{
- 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 c59dfe3..f7a7633 100644
--- a/src/core/SkPictureRecord.cpp
+++ b/src/core/SkPictureRecord.cpp
@@ -146,7 +146,7 @@
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 @@
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 @@
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 @@
*/
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 @@
}
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 @@
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 @@
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 @@
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::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 @@
}
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 @@
prevOffset = 0;
}
- size_t offset = fWriter.bytesWritten();
+ size_t offset = fWriter.size();
addInt(prevOffset);
fRestoreOffsetStack.top() = offset;
}
@@ -840,7 +840,7 @@
// 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 @@
// 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 @@
// 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 @@
// 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 @@
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 @@
// 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 @@
// 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 @@
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 @@
// 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 @@
// 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 @@
// 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 @@
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 @@
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 @@
}
}
#ifdef SK_DEBUG_SIZE
- fPointBytes += fWriter.bytesWritten() - start;
+ fPointBytes += fWriter.size() - start;
fPointWrites += points;
#endif
this->validate(initialOffset, size);
@@ -1136,7 +1136,7 @@
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 @@
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 @@
// 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 @@
}
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::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::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::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 da79fc1..d80182c 100644
--- a/src/core/SkPictureRecord.h
+++ b/src/core/SkPictureRecord.h
@@ -131,7 +131,7 @@
* 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 @@
#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 8f6bfb5..32f3df4 100644
--- a/src/core/SkShader.cpp
+++ b/src/core/SkShader.cpp
@@ -51,7 +51,7 @@
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 8255336..255ef26 100644
--- a/src/core/SkSpriteBlitter_ARGB32.cpp
+++ b/src/core/SkSpriteBlitter_ARGB32.cpp
@@ -277,7 +277,7 @@
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 2bce41e..8cef767 100644
--- a/src/core/SkSpriteBlitter_RGB16.cpp
+++ b/src/core/SkSpriteBlitter_RGB16.cpp
@@ -324,7 +324,7 @@
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 226b317..1b54425 100644
--- a/src/effects/SkBitmapAlphaThresholdShader.cpp
+++ b/src/effects/SkBitmapAlphaThresholdShader.cpp
@@ -249,7 +249,7 @@
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 e0b8d54..061cfde 100644
--- a/src/effects/SkBlurMask.h
+++ b/src/effects/SkBlurMask.h
@@ -44,19 +44,17 @@
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 5b72a6c..6a5a397 100644
--- a/src/effects/SkBlurMaskFilter.cpp
+++ b/src/effects/SkBlurMaskFilter.cpp
@@ -386,7 +386,7 @@
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 152ec56..52c01fe 100644
--- a/src/effects/SkMorphologyImageFilter.cpp
+++ b/src/effects/SkMorphologyImageFilter.cpp
@@ -501,7 +501,7 @@
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 28911313..d0e21a3 100755
--- a/src/effects/SkTestImageFilters.cpp
+++ b/src/effects/SkTestImageFilters.cpp
@@ -48,7 +48,7 @@
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 970e74f..0274481 100644
--- a/src/effects/SkTransparentShader.cpp
+++ b/src/effects/SkTransparentShader.cpp
@@ -23,7 +23,7 @@
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 @@
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::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 e2597df..c8713b6 100644
--- a/src/gpu/GrClipMaskManager.cpp
+++ b/src/gpu/GrClipMaskManager.cpp
@@ -366,7 +366,7 @@
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 @@
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 @@
// 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 6a1cc02..c75bba2 100644
--- a/src/gpu/GrDrawTarget.h
+++ b/src/gpu/GrDrawTarget.h
@@ -362,7 +362,7 @@
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 a299512..e426de3 100644
--- a/src/gpu/GrInOrderDrawBuffer.cpp
+++ b/src/gpu/GrInOrderDrawBuffer.cpp
@@ -213,7 +213,7 @@
// 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 a5f4519..2b6583e 100644
--- a/src/gpu/GrReducedClip.cpp
+++ b/src/gpu/GrReducedClip.cpp
@@ -51,7 +51,7 @@
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 @@
} 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 @@
}
}
- 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 f47a91b..730dad6 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 @@
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::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::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::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 @@
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 @@
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 @@
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 @@
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 74c80bc..89bd059 100644
--- a/src/images/SkImageDecoder.cpp
+++ b/src/images/SkImageDecoder.cpp
@@ -229,7 +229,7 @@
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 @@
}
// 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 ab58aef..68014dd 100644
--- a/src/images/SkImageDecoder_libwebp.cpp
+++ b/src/images/SkImageDecoder_libwebp.cpp
@@ -521,7 +521,7 @@
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 97b741b..5abc23c 100644
--- a/src/images/SkImageEncoder_argb.cpp
+++ b/src/images/SkImageEncoder_argb.cpp
@@ -87,7 +87,7 @@
}
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 77aa7a3..d8e7b13 100644
--- a/src/pdf/SkPDFDevice.cpp
+++ b/src/pdf/SkPDFDevice.cpp
@@ -1607,13 +1607,13 @@
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 @@
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 c3dc396..ed20455 100644
--- a/src/pdf/SkPDFImage.cpp
+++ b/src/pdf/SkPDFImage.cpp
@@ -27,7 +27,7 @@
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 @@
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 @@
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 b61de1c..50043c6 100644
--- a/src/pipe/SkGPipeWrite.cpp
+++ b/src/pipe/SkGPipeWrite.cpp
@@ -294,7 +294,7 @@
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 @@
}
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 9b93f81..3f850b5 100644
--- a/src/ports/SkImageDecoder_CG.cpp
+++ b/src/ports/SkImageDecoder_CG.cpp
@@ -219,7 +219,7 @@
// 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 9c7d8fa..c62bc04 100644
--- a/src/utils/SkCanvasStateUtils.cpp
+++ b/src/utils/SkCanvasStateUtils.cpp
@@ -171,7 +171,7 @@
}
// 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 @@
}
// 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 3df3151..7d11662 100644
--- a/src/utils/debugger/SkDebugCanvas.h
+++ b/src/utils/debugger/SkDebugCanvas.h
@@ -109,8 +109,8 @@
/**
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 107e04a..9dd77c0 100644
--- a/src/utils/debugger/SkObjectParser.cpp
+++ b/src/utils/debugger/SkObjectParser.cpp
@@ -31,7 +31,7 @@
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 3291450..d6fe32b 100644
--- a/src/views/SkWindow.cpp
+++ b/src/views/SkWindow.cpp
@@ -117,7 +117,7 @@
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 5cef1eb..bf0d2f7 100644
--- a/tests/BitmapCopyTest.cpp
+++ b/tests/BitmapCopyTest.cpp
@@ -96,7 +96,7 @@
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 @@
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 @@
// 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 @@
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 @@
// 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 @@
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 6e1d74d..0d9cf2b 100644
--- a/tests/GpuBitmapCopyTest.cpp
+++ b/tests/GpuBitmapCopyTest.cpp
@@ -139,7 +139,7 @@
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 51a6e7f..772d7b2 100644
--- a/tests/Matrix44Test.cpp
+++ b/tests/Matrix44Test.cpp
@@ -73,7 +73,8 @@
}
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 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_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 @@
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_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 @@
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) {
- 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_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_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_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_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_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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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_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_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_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 @@
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 @@
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 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 d879ea6..6544c0b 100644
--- a/tests/PathTest.cpp
+++ b/tests/PathTest.cpp
@@ -1768,7 +1768,7 @@
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 49fc6e0..78aa796 100644
--- a/tests/ShaderImageFilterTest.cpp
+++ b/tests/ShaderImageFilterTest.cpp
@@ -37,8 +37,7 @@
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 1521837..8b4aba5 100644
--- a/tests/Writer32Test.cpp
+++ b/tests/Writer32Test.cpp
@@ -86,7 +86,7 @@
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 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 @@
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 @@
}
}
- 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 7d23b85..620fbf4 100644
--- a/tests/XfermodeTest.cpp
+++ b/tests/XfermodeTest.cpp
@@ -23,7 +23,7 @@
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 @@
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();
}