SkPDF s/SkAutoTDelete/std::unique_ptr/

src/pdf/SkPDFDevice.h

Index: src/pdf/SkPDFDevice.h
diff --git a/src/pdf/SkPDFDevice.h b/src/pdf/SkPDFDevice.h
index 095bbe96379a1d21adf9b34859c85795174dcdb4..c3543df6cdcc6cd6ce8cf55cfc5265ee5f36d85b 100644
--- a/src/pdf/SkPDFDevice.h
+++ b/src/pdf/SkPDFDevice.h
@@ -248,19 +248,19 @@ private:
     SkTDArray<SkPDFFont*> fFontResources;
     SkTDArray<SkPDFObject*> fShaderResources;
 
-    SkAutoTDelete<ContentEntry> fContentEntries;
+    std::unique_ptr<ContentEntry> fContentEntries;
     ContentEntry* fLastContentEntry;
-    SkAutoTDelete<ContentEntry> fMarginContentEntries;
+    std::unique_ptr<ContentEntry> fMarginContentEntries;
     ContentEntry* fLastMarginContentEntry;
     DrawingArea fDrawingArea;
 
     const SkClipStack* fClipStack;
 
     // Accessor and setter functions based on the current DrawingArea.
-    SkAutoTDelete<ContentEntry>* getContentEntries();
+    std::unique_ptr<ContentEntry>* getContentEntries();

[bungeman-skia, 2016/03/25 19:54:22]

Not really for this CL, but shouldn't this return either a 'const
std::unique_ptr<ContentEntry>&' or just a 'const ContentEntry&' (or non-const,
as the case may be).

[hal.canary, 2016/03/28 14:23:32]

I removed this function altogether in crrev.com/1837453003

 
     // Glyph ids used for each font on this device.
-    SkAutoTDelete<SkPDFGlyphSetMap> fFontGlyphUsage;
+    std::unique_ptr<SkPDFGlyphSetMap> fFontGlyphUsage;
 
     SkScalar fRasterDpi;