SkMultiPictureDraw API

src/core/SkMultiPictureDraw.cpp

Index: src/core/SkMultiPictureDraw.cpp
diff --git a/src/core/SkMultiPictureDraw.cpp b/src/core/SkMultiPictureDraw.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..b5783a5f0dc0e8a2cf1df6a79679d8094328828f
--- /dev/null
+++ b/src/core/SkMultiPictureDraw.cpp
@@ -0,0 +1,62 @@
+/*
+ * Copyright 2014 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include "SkCanvas.h"
+#include "SkMultiPictureDraw.h"
+#include "SkPicture.h"
+
+SkMultiPictureDraw::SkMultiPictureDraw(int sizeHint) {
+    if (sizeHint > 0) {
+        fDrawData.setReserve(sizeHint);
+    }
+}
+
+void SkMultiPictureDraw::unrefAll() {

[bsalomon, 2014/08/21 17:58:40]

reset()? clear()? 

Maybe this should be public? It seems potentially useful and safe to expose.

[robertphillips, 2014/08/21 18:21:35]

Done - as reset.

+    for (int i = 0; i < fDrawData.count(); ++i) {
+        fDrawData[i].picture->unref();
+        fDrawData[i].canvas->unref();
+        SkDELETE(fDrawData[i].paint);
+    }
+
+    fDrawData.rewind();
+}
+
+void SkMultiPictureDraw::add(SkCanvas* canvas, 
+                             const SkPicture* picture,
+                             const SkMatrix* matrix, 
+                             const SkPaint* paint) {
+    if (NULL == canvas || NULL == picture) {
+        SkDEBUGFAIL("parameters to SkMultiPictureDraw::add should be non-NULL");
+        return;
+    }
+
+    DrawData* data = fDrawData.append();
+
+    data->picture = SkRef(picture);
+    data->canvas = SkRef(canvas);
+    if (NULL != matrix) {
+        data->matrix = *matrix;
+    } else {
+        data->matrix.setIdentity();
+    }
+    if (NULL != paint) {
+        data->paint = SkNEW_ARGS(SkPaint, (*paint));
+    } else {
+        data->paint = NULL;
+    }
+}
+
+void SkMultiPictureDraw::draw() {
+    for (int i = 0; i < fDrawData.count(); ++i) {
+        fDrawData[i].canvas->drawPicture(fDrawData[i].picture, 
+                                         &fDrawData[i].matrix, 
+                                         fDrawData[i].paint);
+    }
+
+    this->unrefAll();
+}
+