DM: Add --skps.

dm/DMRecordTask.h

 #ifndef DMRecordTask_DEFINED
 #define DMRecordTask_DEFINED
 
 #include "DMTask.h"
 #include "SkBitmap.h"
+#include "SkPicture.h"
 #include "SkString.h"
 #include "SkTemplates.h"
 #include "gm.h"
 
-// Records a GM through an SkRecord, draws it, and compares against the reference bitmap.
+// Records a GM or SKP through an SkRecord, draws it, and compares against the reference bitmap.
 
 namespace DM {
 
 class RecordTask : public CpuTask {
 
 public:
     RecordTask(const Task& parent, skiagm::GM*, SkBitmap reference, bool optimize);

[reed1, 2014/05/14 12:40:11]

nit: why isn't this parameter const SkBitmap& ?

[mtklein, 2014/05/14 17:47:18]

It could be, but we've got to make the copy either way because we could be
crossing threads.  That is, it can't safely be const SkBitmap& fReference below.

It seemed clearer to remind the caller that the SkBitmap would be copied in the
method signature.  Because the constructor initializes fReference in the
intializer list, the copy only happens once, so it's the same cost as passing by
reference.

+    RecordTask(const Task& parent, SkPicture*,  SkBitmap reference, bool optimize);
 
     virtual void draw() SK_OVERRIDE;
     virtual bool shouldSkip() const SK_OVERRIDE;
     virtual SkString name() const SK_OVERRIDE { return fName; }
 
 private:
     const SkString fName;
+    SkAutoTUnref<SkPicture> fPicture;
     SkAutoTDelete<skiagm::GM> fGM;
     const SkBitmap fReference;
     bool fOptimize;
 };
 
 }  // namespace DM
 
 #endif  // DMRecordTask_DEFINED