Don't clobber initial transform with SetMatrix.

src/record/SkRecordDraw.h

Index: src/record/SkRecordDraw.h
diff --git a/src/record/SkRecordDraw.h b/src/record/SkRecordDraw.h
index 4ec6e68e928cc77207d8036c97b21ac8bb177784..359679a6d7445f3852db152c591f57f8ce27ad21 100644
--- a/src/record/SkRecordDraw.h
+++ b/src/record/SkRecordDraw.h
@@ -19,7 +19,8 @@ namespace SkRecords {
 // This is an SkRecord visitor that will draw that SkRecord to an SkCanvas.
 class Draw : SkNoncopyable {
 public:
-    explicit Draw(SkCanvas* canvas) : fCanvas(canvas), fIndex(0) {}
+    explicit Draw(SkCanvas* canvas)
+        : fInitialCTM(canvas->getTotalMatrix()), fCanvas(canvas), fIndex(0) {}
 
     unsigned index() const { return fIndex; }
     void next() { ++fIndex; }
@@ -44,6 +45,7 @@ private:
     bool skip(const PairedPushCull&);
     bool skip(const BoundedDrawPosTextH&);
 
+    const SkMatrix fInitialCTM;

[reed1, 2014/05/19 15:04:34]

10x4=40 bytes added to every record. I guess space is not an object (as we could
conceivably store an index to the finite set of different CTMs we encounter in a
given picture)?

[mtklein, 2014/05/19 15:11:23]

This is 40 bytes extra used on the stack while playing the picture back. 
Nothing extra called or stored per-op.

     SkCanvas* fCanvas;
     unsigned fIndex;
 };