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

Unified Diff: printing/metafile_skia_wrapper.h

Issue 2512783002: store MetafileSkiaWrapper in plugin instead of canvas (Closed)
Patch Set: doh -- add include to impl file, not header Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: printing/metafile_skia_wrapper.h
diff --git a/printing/metafile_skia_wrapper.h b/printing/metafile_skia_wrapper.h
index 6236e85a4c4c5e372009b7c237b94c01648212eb..1ad72223d4a5e8718098f196b6350f2a247075b3 100644
--- a/printing/metafile_skia_wrapper.h
+++ b/printing/metafile_skia_wrapper.h
@@ -23,7 +23,9 @@ class PRINTING_EXPORT MetafileSkiaWrapper : public SkRefCnt {
static void SetMetafileOnCanvas(const SkCanvas& canvas,
PdfMetafileSkia* metafile);
- static PdfMetafileSkia* GetMetafileFromCanvas(const SkCanvas& canvas);
+ static sk_sp<MetafileSkiaWrapper> RefFromCanvas(const SkCanvas& canvas);
+
+ PdfMetafileSkia* getMetafile() const { return metafile_; }
Lei Zhang 2016/11/18 01:53:37 nit: GetMetafile(), or just metafile() since it's
private:
explicit MetafileSkiaWrapper(PdfMetafileSkia* metafile);

Powered by Google App Engine
This is Rietveld 408576698