Make sp variants for SkData

src/core/SkPictureRecord.h

Index: src/core/SkPictureRecord.h
diff --git a/src/core/SkPictureRecord.h b/src/core/SkPictureRecord.h
index 607f6f10f7a99da1615fbcf47da84d0af51f54b1..089bc81ee56bb3cff93a11acb02e7796fe825c91 100644
--- a/src/core/SkPictureRecord.h
+++ b/src/core/SkPictureRecord.h
@@ -43,15 +43,15 @@ public:
         return fImageRefs;
     }
 
-    SkData* opData(bool deepCopy) const {
+    sk_sp<SkData> opData(bool deepCopy) const {
         this->validate(fWriter.bytesWritten(), 0);
 
         if (fWriter.bytesWritten() == 0) {
-            return SkData::NewEmpty();
+            return SkData::MakeEmpty();
         }
 
         if (deepCopy) {
-            return SkData::NewWithCopy(fWriter.contiguousArray(), fWriter.bytesWritten());
+            return SkData::MakeWithCopy(fWriter.contiguousArray(), fWriter.bytesWritten());

[scroggo, 2016/03/11 15:28:35]

Not related to your change, but it appears that this function does the same
thing whether deepCopy is true or not.

[reed1, 2016/03/11 19:23:34]

Acknowledged.

         }
 
         return fWriter.snapshotAsData();