Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(342)

Side by Side Diff: src/pdf/SkPDFDevice.h

Issue 1837553002: SkPDF s/SkAutoTDelete/std::unique_ptr/ (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2011 Google Inc. 2 * Copyright 2011 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #ifndef SkPDFDevice_DEFINED 8 #ifndef SkPDFDevice_DEFINED
9 #define SkPDFDevice_DEFINED 9 #define SkPDFDevice_DEFINED
10 10
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 241
242 SkTArray<RectWithData> fLinkToURLs; 242 SkTArray<RectWithData> fLinkToURLs;
243 SkTArray<RectWithData> fLinkToDestinations; 243 SkTArray<RectWithData> fLinkToDestinations;
244 SkTArray<NamedDestination> fNamedDestinations; 244 SkTArray<NamedDestination> fNamedDestinations;
245 245
246 SkTDArray<SkPDFObject*> fGraphicStateResources; 246 SkTDArray<SkPDFObject*> fGraphicStateResources;
247 SkTDArray<SkPDFObject*> fXObjectResources; 247 SkTDArray<SkPDFObject*> fXObjectResources;
248 SkTDArray<SkPDFFont*> fFontResources; 248 SkTDArray<SkPDFFont*> fFontResources;
249 SkTDArray<SkPDFObject*> fShaderResources; 249 SkTDArray<SkPDFObject*> fShaderResources;
250 250
251 SkAutoTDelete<ContentEntry> fContentEntries; 251 std::unique_ptr<ContentEntry> fContentEntries;
252 ContentEntry* fLastContentEntry; 252 ContentEntry* fLastContentEntry;
253 SkAutoTDelete<ContentEntry> fMarginContentEntries; 253 std::unique_ptr<ContentEntry> fMarginContentEntries;
254 ContentEntry* fLastMarginContentEntry; 254 ContentEntry* fLastMarginContentEntry;
255 DrawingArea fDrawingArea; 255 DrawingArea fDrawingArea;
256 256
257 const SkClipStack* fClipStack; 257 const SkClipStack* fClipStack;
258 258
259 // Accessor and setter functions based on the current DrawingArea. 259 // Accessor and setter functions based on the current DrawingArea.
260 SkAutoTDelete<ContentEntry>* getContentEntries(); 260 std::unique_ptr<ContentEntry>* getContentEntries();
bungeman-skia 2016/03/25 19:54:22 Not really for this CL, but shouldn't this return
hal.canary 2016/03/28 14:23:32 I removed this function altogether in crrev.com/18
261 261
262 // Glyph ids used for each font on this device. 262 // Glyph ids used for each font on this device.
263 SkAutoTDelete<SkPDFGlyphSetMap> fFontGlyphUsage; 263 std::unique_ptr<SkPDFGlyphSetMap> fFontGlyphUsage;
264 264
265 SkScalar fRasterDpi; 265 SkScalar fRasterDpi;
266 266
267 SkBitmap fLegacyBitmap; 267 SkBitmap fLegacyBitmap;
268 268
269 SkPDFDocument* fDocument; 269 SkPDFDocument* fDocument;
270 //////////////////////////////////////////////////////////////////////////// 270 ////////////////////////////////////////////////////////////////////////////
271 271
272 SkPDFDevice(SkISize pageSize, 272 SkPDFDevice(SkISize pageSize,
273 SkScalar rasterDpi, 273 SkScalar rasterDpi,
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 338
339 typedef SkBaseDevice INHERITED; 339 typedef SkBaseDevice INHERITED;
340 340
341 // TODO(edisonn): Only SkDocument_PDF and SkPDFImageShader should be able to create 341 // TODO(edisonn): Only SkDocument_PDF and SkPDFImageShader should be able to create
342 // an SkPDFDevice 342 // an SkPDFDevice
343 //friend class SkDocument_PDF; 343 //friend class SkDocument_PDF;
344 //friend class SkPDFImageShader; 344 //friend class SkPDFImageShader;
345 }; 345 };
346 346
347 #endif 347 #endif
OLDNEW
« no previous file with comments | « src/pdf/SkPDFBitmap.cpp ('k') | src/pdf/SkPDFDevice.cpp » ('j') | src/pdf/SkPDFDevice.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698