SkLiteDL: drawAsLayer()

src/core/SkLiteDL.h

Index: src/core/SkLiteDL.h
diff --git a/src/core/SkLiteDL.h b/src/core/SkLiteDL.h
index 9dd1568ddf778076b07b0e5c5fed518473058734..4f045e9458bddb90c3d6389becf01b594eed5bee 100644
--- a/src/core/SkLiteDL.h
+++ b/src/core/SkLiteDL.h
@@ -27,6 +27,20 @@ public:
     void setDrawFilter(SkDrawFilter*);
 #endif
 
+    // Draws as if...
+    //   SkPaint p;
+    //   p.setColorFilter(cf);
+    //   p.setXfermode(xfer);
+    //   p.setAlpha(alpha);
+    //   SkRect bounds = this->getBounds();
+    //   c->saveLayer(&bounds, &p);
+    //       this->draw(c, matrix);
+    //   c->restore();
+    void drawAsLayer(SkCanvas* c, const SkMatrix* matrix

[djsollen, 2016/09/07 14:31:36]

yep, though in my current case matrix will always be a nullptr.

[reed1, 2016/09/07 15:29:55]

drawPicture has nearly this signature already:

canvas->drawPicture(picture, matrix, layerPaint)

I think this guy should take a paint* instead of separate params:
- matches drawPicture (and saveLayer for that matter)
- can be well defined (given the other layer-paint uses as to which fields are
respected)
- automatically gets other fields (not yet respected by android.. yet) like
imagefilter

[mtklein, 2016/09/08 15:20:54]

sgtm, ptal

+                                , SkColorFilter* cf
+                                , SkXfermode* xfer
+                                , U8CPU alpha);
+
     void save();
     void saveLayer(const SkRect*, const SkPaint*, const SkImageFilter*, SkCanvas::SaveLayerFlags);
     void restore();