SkCanvas interface for drawing a patch.

src/core/SkBitmapDevice.cpp

Index: src/core/SkBitmapDevice.cpp
diff --git a/src/core/SkBitmapDevice.cpp b/src/core/SkBitmapDevice.cpp
index 09b3b605f85a3e4b36a28e77d88a75135abfae3b..047a3d6fafdf4eca539637fbc44cbfa5995bc919 100644
--- a/src/core/SkBitmapDevice.cpp
+++ b/src/core/SkBitmapDevice.cpp
@@ -343,6 +343,13 @@ void SkBitmapDevice::drawVertices(const SkDraw& draw, SkCanvas::VertexMode vmode
                       indices, indexCount, paint);
 }
 
+void SkBitmapDevice::drawPatch(const SkDraw& draw, const SkPatch& patch, const SkPaint& paint) {
+    SkPatch::VertexData data;
+    patch.getVertexData(&data, draw.fMatrix->getScaleX() * 10, draw.fMatrix->getScaleY() * 10);

[bsalomon, 2014/07/28 20:56:26]

I don't think getScaleX and getScaleY are what you want. I think that will give
you 0 for this matrix:

0   100 0
100   0 0
0     0 1

You might want SkMatrix::getMaxScale().

The 10x scale factor should have some kind of comment. Does the tess. level not
depend on the size of the patch?

This implementation could be in SkDevice and then all the other subclasses can
share it (until they have their own specializations where it makes sense to do
so).

[dandov, 2014/07/29 18:45:46]

I moved this to SkBaseDevice and created a default implementation, the level of
detail now takes into account the size of the patch using its corner points
already mapped to the transform matrix in SkDraw. 

+    draw.drawVertices(SkCanvas::kTriangles_VertexMode, data.fVertexCount, data.fPoints,
+                      data.fTexCoords, data.fColors, NULL, data.fIndices, data.fIndexCount, paint);
+}
+
 void SkBitmapDevice::drawDevice(const SkDraw& draw, SkBaseDevice* device,
                                 int x, int y, const SkPaint& paint) {
     const SkBitmap& src = device->accessBitmap(false);