diff options
author | robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-17 17:34:20 +0000 |
---|---|---|
committer | robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2013-10-17 17:34:20 +0000 |
commit | c2cc1dbe818c8a5a699fbe18c4fc79b9d93daa94 (patch) | |
tree | d9d5b8aa79d17c0a5683a70fd57f92e55de27ca2 | |
parent | 4256d2494e80cb6c9a2db4157adc9b9daff028a6 (diff) |
Revert r11842 (call drawRect to try GrAARectRenderer if the path is a rect - https://codereview.chromium.org/23484007) due to changes to the following GM images:
inverse_paths
pathopsinverse
git-svn-id: http://skia.googlecode.com/svn/trunk@11845 2bbb7eff-a529-9590-31e7-b0007b416f81
-rw-r--r-- | experimental/PdfViewer/SkNulCanvas.h | 4 | ||||
-rw-r--r-- | experimental/SimpleCocoaApp/SimpleApp.mm | 2 | ||||
-rw-r--r-- | include/core/SkCanvas.h | 26 | ||||
-rw-r--r-- | include/utils/SkDeferredCanvas.h | 5 | ||||
-rw-r--r-- | include/utils/SkDumpCanvas.h | 6 | ||||
-rw-r--r-- | include/utils/SkLuaCanvas.h | 6 | ||||
-rw-r--r-- | include/utils/SkNWayCanvas.h | 4 | ||||
-rw-r--r-- | include/utils/SkProxyCanvas.h | 6 | ||||
-rw-r--r-- | src/core/SkBBoxRecord.cpp | 10 | ||||
-rw-r--r-- | src/core/SkBBoxRecord.h | 6 | ||||
-rw-r--r-- | src/core/SkCanvas.cpp | 4 | ||||
-rw-r--r-- | src/core/SkPictureRecord.cpp | 4 | ||||
-rw-r--r-- | src/core/SkPictureRecord.h | 6 | ||||
-rw-r--r-- | src/pipe/SkGPipeWrite.cpp | 11 | ||||
-rw-r--r-- | src/utils/SkDeferredCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/SkDumpCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/SkLuaCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/SkNWayCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/SkProxyCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/debugger/SkDebugCanvas.cpp | 4 | ||||
-rw-r--r-- | src/utils/debugger/SkDebugCanvas.h | 9 | ||||
-rw-r--r-- | tests/TileGridTest.cpp | 7 |
22 files changed, 54 insertions, 86 deletions
diff --git a/experimental/PdfViewer/SkNulCanvas.h b/experimental/PdfViewer/SkNulCanvas.h index c43ea82918..8ec2d32d0f 100644 --- a/experimental/PdfViewer/SkNulCanvas.h +++ b/experimental/PdfViewer/SkNulCanvas.h @@ -55,8 +55,10 @@ public: virtual void drawPaint(const SkPaint& paint) {} virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) {} + virtual void drawRect(const SkRect& rect, const SkPaint& paint) {} virtual void drawOval(const SkRect& oval, const SkPaint&) {} virtual void drawRRect(const SkRRect& rrect, const SkPaint& paint) {} + virtual void drawPath(const SkPath& path, const SkPaint& paint) {} virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint* paint = NULL) {} virtual void drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* src, @@ -93,8 +95,6 @@ public: virtual SkDrawFilter* setDrawFilter(SkDrawFilter* filter) {return NULL;} protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint& paint) {} - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) {} virtual SkCanvas* canvasForDrawIter() {return NULL;} virtual SkBaseDevice* setDevice(SkBaseDevice* device) {return NULL;} diff --git a/experimental/SimpleCocoaApp/SimpleApp.mm b/experimental/SimpleCocoaApp/SimpleApp.mm index 2156209798..7dac2a4fff 100644 --- a/experimental/SimpleCocoaApp/SimpleApp.mm +++ b/experimental/SimpleCocoaApp/SimpleApp.mm @@ -135,7 +135,7 @@ static int showPathContour(SkPath::Iter& iter) { } class PathCanvas : public SkCanvas { - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) { + virtual void drawPath(const SkPath& path, const SkPaint& paint) { if (nameonly) { SkDebugf(" %s%d,\n", filename.c_str(), ++count); return; diff --git a/include/core/SkCanvas.h b/include/core/SkCanvas.h index 8c09a9c91e..dc3953ed4b 100644 --- a/include/core/SkCanvas.h +++ b/include/core/SkCanvas.h @@ -575,20 +575,16 @@ public: will be filled or stroked based on the Style in the paint. @param rect The rect to be drawn @param paint The paint used to draw the rect - - Overriding this function is deprecated. It will be made non-virtual - soon. Instead override onDrawRect. */ - virtual void drawRect(const SkRect& rect, const SkPaint& paint) { - this->onDrawRect(rect, paint); - } + virtual void drawRect(const SkRect& rect, const SkPaint& paint); /** Draw the specified rectangle using the specified paint. The rectangle will be filled or framed based on the Style in the paint. @param rect The rect to be drawn @param paint The paint used to draw the rect */ - void drawIRect(const SkIRect& rect, const SkPaint& paint) { + void drawIRect(const SkIRect& rect, const SkPaint& paint) + { SkRect r; r.set(rect); // promotes the ints to scalars this->drawRect(r, paint); @@ -660,18 +656,8 @@ public: filled or framed based on the Style in the paint. @param path The path to be drawn @param paint The paint used to draw the path - - Overriding this function is deprecated. It will be made non-virtual - soon. Instead override onDrawRect. */ - virtual void drawPath(const SkPath& path, const SkPaint& paint) { - SkRect rect; - if (path.isRect(&rect)) { - this->onDrawRect(rect, paint); - } else { - this->onDrawPath(path, paint); - } - } + virtual void drawPath(const SkPath& path, const SkPaint& paint); /** Draw the specified bitmap, with its top/left corner at (x,y), using the specified paint, transformed by the current matrix. Note: if the paint @@ -1048,10 +1034,6 @@ protected: // can perform copy-on-write or invalidate any cached images void predrawNotify(); - virtual void onDrawRect(const SkRect& rect, const SkPaint& paint); - - virtual void onDrawPath(const SkPath& path, const SkPaint& paint); - /** DEPRECATED -- use constructor(device) Marked as 'protected' to avoid new clients using this before we can diff --git a/include/utils/SkDeferredCanvas.h b/include/utils/SkDeferredCanvas.h index 052c667123..c0613ed544 100644 --- a/include/utils/SkDeferredCanvas.h +++ b/include/utils/SkDeferredCanvas.h @@ -162,7 +162,10 @@ public: virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawRect(const SkRect& rect, const SkPaint& paint) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint& paint) + SK_OVERRIDE; virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint* paint) SK_OVERRIDE; @@ -231,8 +234,6 @@ public: }; protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint& paint) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; virtual SkCanvas* canvasForDrawIter(); DeferredDevice* getDeferredDevice() const; diff --git a/include/utils/SkDumpCanvas.h b/include/utils/SkDumpCanvas.h index d3ff9a1c18..96b45e7ba2 100644 --- a/include/utils/SkDumpCanvas.h +++ b/include/utils/SkDumpCanvas.h @@ -93,7 +93,9 @@ public: virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint* paint) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* src, @@ -124,10 +126,6 @@ public: virtual void addComment(const char* kywd, const char* value) SK_OVERRIDE; virtual void endCommentGroup() SK_OVERRIDE; -protected: - virtual void onDrawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; - private: Dumper* fDumper; int fNestLevel; // for nesting recursive elements like pictures diff --git a/include/utils/SkLuaCanvas.h b/include/utils/SkLuaCanvas.h index b604d3c92c..c34d134423 100644 --- a/include/utils/SkLuaCanvas.h +++ b/include/utils/SkLuaCanvas.h @@ -42,7 +42,9 @@ public: virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint* paint) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* src, @@ -70,10 +72,6 @@ public: const SkPaint& paint) SK_OVERRIDE; virtual void drawData(const void* data, size_t length) SK_OVERRIDE; -protected: - virtual void onDrawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; - private: lua_State* fL; SkString fFunc; diff --git a/include/utils/SkNWayCanvas.h b/include/utils/SkNWayCanvas.h index 6791c51d58..c48bcab3c4 100644 --- a/include/utils/SkNWayCanvas.h +++ b/include/utils/SkNWayCanvas.h @@ -43,7 +43,9 @@ public: virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint&) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawRect(const SkRect&, const SkPaint&) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint*) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* src, @@ -78,8 +80,6 @@ public: virtual void endCommentGroup() SK_OVERRIDE; protected: - virtual void onDrawRect(const SkRect&, const SkPaint&) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; SkTDArray<SkCanvas*> fList; class Iter; diff --git a/include/utils/SkProxyCanvas.h b/include/utils/SkProxyCanvas.h index 54cdc3b2b0..383e532fbf 100644 --- a/include/utils/SkProxyCanvas.h +++ b/include/utils/SkProxyCanvas.h @@ -48,7 +48,9 @@ public: virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap& bitmap, SkScalar left, SkScalar top, const SkPaint* paint = NULL) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap& bitmap, const SkRect* src, @@ -83,10 +85,6 @@ public: virtual SkBounder* setBounder(SkBounder* bounder) SK_OVERRIDE; virtual SkDrawFilter* setDrawFilter(SkDrawFilter* filter) SK_OVERRIDE; -protected: - virtual void onDrawRect(const SkRect&, const SkPaint& paint) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; - private: SkCanvas* fProxy; diff --git a/src/core/SkBBoxRecord.cpp b/src/core/SkBBoxRecord.cpp index 08ae990135..1e6c69ba09 100644 --- a/src/core/SkBBoxRecord.cpp +++ b/src/core/SkBBoxRecord.cpp @@ -20,23 +20,23 @@ void SkBBoxRecord::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } } -void SkBBoxRecord::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkBBoxRecord::drawRect(const SkRect& rect, const SkPaint& paint) { if (this->transformBounds(rect, &paint)) { - INHERITED::onDrawRect(rect, paint); + INHERITED::drawRect(rect, paint); } } -void SkBBoxRecord::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkBBoxRecord::drawPath(const SkPath& path, const SkPaint& paint) { if (path.isInverseFillType()) { // If path is inverse filled, use the current clip bounds as the // path's device-space bounding box. SkIRect clipBounds; if (this->getClipDeviceBounds(&clipBounds)) { this->handleBBox(SkRect::MakeFromIRect(clipBounds)); - INHERITED::onDrawPath(path, paint); + INHERITED::drawPath(path, paint); } } else if (this->transformBounds(path.getBounds(), &paint)) { - INHERITED::onDrawPath(path, paint); + INHERITED::drawPath(path, paint); } } diff --git a/src/core/SkBBoxRecord.h b/src/core/SkBBoxRecord.h index e95ece4954..fa8b282247 100644 --- a/src/core/SkBBoxRecord.h +++ b/src/core/SkBBoxRecord.h @@ -31,6 +31,8 @@ public: virtual void drawOval(const SkRect& rect, const SkPaint& paint) SK_OVERRIDE; virtual void drawRRect(const SkRRect& rrect, const SkPaint& paint) SK_OVERRIDE; + virtual void drawRect(const SkRect& rect, const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; virtual void drawPoints(PointMode mode, size_t count, const SkPoint pts[], const SkPaint& paint) SK_OVERRIDE; virtual void drawPaint(const SkPaint& paint) SK_OVERRIDE; @@ -63,10 +65,6 @@ public: const SkPaint& paint) SK_OVERRIDE; virtual void drawPicture(SkPicture& picture) SK_OVERRIDE; -protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint& paint) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint& paint) SK_OVERRIDE; - private: /** * Takes a bounding box in current canvas view space, accounts for stroking and effects, and diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp index 8fbdda62c1..7add524260 100644 --- a/src/core/SkCanvas.cpp +++ b/src/core/SkCanvas.cpp @@ -1631,7 +1631,7 @@ void SkCanvas::drawPoints(PointMode mode, size_t count, const SkPoint pts[], LOOPER_END } -void SkCanvas::onDrawRect(const SkRect& r, const SkPaint& paint) { +void SkCanvas::drawRect(const SkRect& r, const SkPaint& paint) { CHECK_SHADER_NOSETCONTEXT(paint); if (paint.canComputeFastBounds()) { @@ -1699,7 +1699,7 @@ void SkCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } -void SkCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkCanvas::drawPath(const SkPath& path, const SkPaint& paint) { CHECK_SHADER_NOSETCONTEXT(paint); if (!path.isFinite()) { diff --git a/src/core/SkPictureRecord.cpp b/src/core/SkPictureRecord.cpp index bf7a34eef0..f7a7633faa 100644 --- a/src/core/SkPictureRecord.cpp +++ b/src/core/SkPictureRecord.cpp @@ -868,7 +868,7 @@ void SkPictureRecord::drawOval(const SkRect& oval, const SkPaint& paint) { this->validate(initialOffset, size); } -void SkPictureRecord::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkPictureRecord::drawRect(const SkRect& rect, const SkPaint& paint) { // op + paint index + rect uint32_t size = 2 * kUInt32Size + sizeof(rect); size_t initialOffset = this->addDraw(DRAW_RECT, &size); @@ -895,7 +895,7 @@ void SkPictureRecord::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } } -void SkPictureRecord::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkPictureRecord::drawPath(const SkPath& path, const SkPaint& paint) { // op + paint index + path index uint32_t size = 3 * kUInt32Size; size_t initialOffset = this->addDraw(DRAW_PATH, &size); diff --git a/src/core/SkPictureRecord.h b/src/core/SkPictureRecord.h index 33229cf45d..d80182cf8c 100644 --- a/src/core/SkPictureRecord.h +++ b/src/core/SkPictureRecord.h @@ -53,7 +53,9 @@ public: virtual void drawPoints(PointMode, size_t count, const SkPoint pts[], const SkPaint&) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawRect(const SkRect&, const SkPaint&) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap&, SkScalar left, SkScalar top, const SkPaint*) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap&, const SkRect* src, @@ -104,10 +106,6 @@ public: void beginRecording(); void endRecording(); -protected: - virtual void onDrawRect(const SkRect&, const SkPaint&) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; - private: void handleOptimization(int opt); void recordRestoreOffsetPlaceholder(SkRegion::Op); diff --git a/src/pipe/SkGPipeWrite.cpp b/src/pipe/SkGPipeWrite.cpp index f14cdacc9b..57d4a0fdc3 100644 --- a/src/pipe/SkGPipeWrite.cpp +++ b/src/pipe/SkGPipeWrite.cpp @@ -224,7 +224,9 @@ public: virtual void drawPoints(PointMode, size_t count, const SkPoint pts[], const SkPaint&) SK_OVERRIDE; virtual void drawOval(const SkRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawRect(const SkRect& rect, const SkPaint&) SK_OVERRIDE; virtual void drawRRect(const SkRRect&, const SkPaint&) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; virtual void drawBitmap(const SkBitmap&, SkScalar left, SkScalar top, const SkPaint*) SK_OVERRIDE; virtual void drawBitmapRectToRect(const SkBitmap&, const SkRect* src, @@ -262,11 +264,6 @@ public: * according to slot. */ bool shuttleBitmap(const SkBitmap&, int32_t slot); - -protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint&) SK_OVERRIDE; - virtual void onDrawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; - private: enum { kNoSaveLayer = -1, @@ -718,7 +715,7 @@ void SkGPipeCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { } } -void SkGPipeCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkGPipeCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { NOTIFY_SETUP(this); this->writePaint(paint); if (this->needOpBytes(sizeof(SkRect))) { @@ -736,7 +733,7 @@ void SkGPipeCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } } -void SkGPipeCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkGPipeCanvas::drawPath(const SkPath& path, const SkPaint& paint) { NOTIFY_SETUP(this); this->writePaint(paint); if (this->needOpBytes(path.writeToMemory(NULL))) { diff --git a/src/utils/SkDeferredCanvas.cpp b/src/utils/SkDeferredCanvas.cpp index bc00ab00d3..ce5eb5e6ab 100644 --- a/src/utils/SkDeferredCanvas.cpp +++ b/src/utils/SkDeferredCanvas.cpp @@ -846,7 +846,7 @@ void SkDeferredCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { this->recordedDrawCommand(); } -void SkDeferredCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkDeferredCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { if (fDeferredDrawing && this->isFullFrame(&rect, &paint) && isPaintOpaque(&paint)) { this->getDeferredDevice()->skipPendingCommands(); @@ -869,7 +869,7 @@ void SkDeferredCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } } -void SkDeferredCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkDeferredCanvas::drawPath(const SkPath& path, const SkPaint& paint) { AutoImmediateDrawIfNeeded autoDraw(*this, &paint); this->drawingCanvas()->drawPath(path, paint); this->recordedDrawCommand(); diff --git a/src/utils/SkDumpCanvas.cpp b/src/utils/SkDumpCanvas.cpp index 6658ee2a93..0e1a2321e1 100644 --- a/src/utils/SkDumpCanvas.cpp +++ b/src/utils/SkDumpCanvas.cpp @@ -323,7 +323,7 @@ void SkDumpCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { this->dump(kDrawOval_Verb, &paint, "drawOval(%s)", str.c_str()); } -void SkDumpCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkDumpCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { SkString str; toString(rect, &str); this->dump(kDrawRect_Verb, &paint, "drawRect(%s)", str.c_str()); @@ -335,7 +335,7 @@ void SkDumpCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { this->dump(kDrawRRect_Verb, &paint, "drawRRect(%s)", str.c_str()); } -void SkDumpCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkDumpCanvas::drawPath(const SkPath& path, const SkPaint& paint) { SkString str; toString(path, &str); this->dump(kDrawPath_Verb, &paint, "drawPath(%s)", str.c_str()); diff --git a/src/utils/SkLuaCanvas.cpp b/src/utils/SkLuaCanvas.cpp index 3b74596a8e..c5ec006913 100644 --- a/src/utils/SkLuaCanvas.cpp +++ b/src/utils/SkLuaCanvas.cpp @@ -187,7 +187,7 @@ void SkLuaCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { lua.pushPaint(paint, "paint"); } -void SkLuaCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkLuaCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { AUTO_LUA("drawRect"); lua.pushRect(rect, "rect"); lua.pushPaint(paint, "paint"); @@ -199,7 +199,7 @@ void SkLuaCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { lua.pushPaint(paint, "paint"); } -void SkLuaCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkLuaCanvas::drawPath(const SkPath& path, const SkPaint& paint) { AUTO_LUA("drawPath"); lua.pushPath(path, "path"); lua.pushPaint(paint, "paint"); diff --git a/src/utils/SkNWayCanvas.cpp b/src/utils/SkNWayCanvas.cpp index 8075bc4f35..eeb1bc2a09 100644 --- a/src/utils/SkNWayCanvas.cpp +++ b/src/utils/SkNWayCanvas.cpp @@ -190,7 +190,7 @@ void SkNWayCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { } } -void SkNWayCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkNWayCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { Iter iter(fList); while (iter.next()) { iter->drawRect(rect, paint); @@ -204,7 +204,7 @@ void SkNWayCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { } } -void SkNWayCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkNWayCanvas::drawPath(const SkPath& path, const SkPaint& paint) { Iter iter(fList); while (iter.next()) { iter->drawPath(path, paint); diff --git a/src/utils/SkProxyCanvas.cpp b/src/utils/SkProxyCanvas.cpp index 3bc9652d29..f530313b16 100644 --- a/src/utils/SkProxyCanvas.cpp +++ b/src/utils/SkProxyCanvas.cpp @@ -87,7 +87,7 @@ void SkProxyCanvas::drawOval(const SkRect& rect, const SkPaint& paint) { fProxy->drawOval(rect, paint); } -void SkProxyCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkProxyCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { fProxy->drawRect(rect, paint); } @@ -95,7 +95,7 @@ void SkProxyCanvas::drawRRect(const SkRRect& rrect, const SkPaint& paint) { fProxy->drawRRect(rrect, paint); } -void SkProxyCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkProxyCanvas::drawPath(const SkPath& path, const SkPaint& paint) { fProxy->drawPath(path, paint); } diff --git a/src/utils/debugger/SkDebugCanvas.cpp b/src/utils/debugger/SkDebugCanvas.cpp index 2add91465b..bf81983548 100644 --- a/src/utils/debugger/SkDebugCanvas.cpp +++ b/src/utils/debugger/SkDebugCanvas.cpp @@ -321,7 +321,7 @@ void SkDebugCanvas::drawPaint(const SkPaint& paint) { addDrawCommand(new SkDrawPaintCommand(paint)); } -void SkDebugCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) { +void SkDebugCanvas::drawPath(const SkPath& path, const SkPaint& paint) { addDrawCommand(new SkDrawPathCommand(path, paint)); } @@ -345,7 +345,7 @@ void SkDebugCanvas::drawPosTextH(const void* text, size_t byteLength, new SkDrawPosTextHCommand(text, byteLength, xpos, constY, paint)); } -void SkDebugCanvas::onDrawRect(const SkRect& rect, const SkPaint& paint) { +void SkDebugCanvas::drawRect(const SkRect& rect, const SkPaint& paint) { // NOTE(chudy): Messing up when renamed to DrawRect... Why? addDrawCommand(new SkDrawRectCommand(rect, paint)); } diff --git a/src/utils/debugger/SkDebugCanvas.h b/src/utils/debugger/SkDebugCanvas.h index 2b17b6ff93..705f49af83 100644 --- a/src/utils/debugger/SkDebugCanvas.h +++ b/src/utils/debugger/SkDebugCanvas.h @@ -182,6 +182,8 @@ public: virtual void drawPaint(const SkPaint& paint) SK_OVERRIDE; + virtual void drawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; + virtual void drawPicture(SkPicture& picture) SK_OVERRIDE; virtual void drawPoints(PointMode, size_t count, const SkPoint pts[], @@ -194,6 +196,8 @@ public: const SkScalar xpos[], SkScalar constY, const SkPaint&) SK_OVERRIDE; + virtual void drawRect(const SkRect& rect, const SkPaint&) SK_OVERRIDE; + virtual void drawRRect(const SkRRect& rrect, const SkPaint& paint) SK_OVERRIDE; virtual void drawSprite(const SkBitmap&, int left, int top, @@ -231,11 +235,6 @@ public: static const int kVizImageHeight = 256; static const int kVizImageWidth = 256; -protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint&) SK_OVERRIDE; - - virtual void onDrawPath(const SkPath& path, const SkPaint&) SK_OVERRIDE; - private: SkTDArray<SkDrawCommand*> fCommandVector; int fHeight; diff --git a/tests/TileGridTest.cpp b/tests/TileGridTest.cpp index 3466871220..8f5b18a6e9 100644 --- a/tests/TileGridTest.cpp +++ b/tests/TileGridTest.cpp @@ -25,14 +25,13 @@ public: MockCanvas(SkBaseDevice* device) : SkCanvas(device) {} - SkTDArray<SkRect> fRects; - -protected: - virtual void onDrawRect(const SkRect& rect, const SkPaint&) + virtual void drawRect(const SkRect& rect, const SkPaint&) { // This capture occurs before quick reject. fRects.push(rect); } + + SkTDArray<SkRect> fRects; }; class TileGridTest { |