Create a Factory for NativeMetafile.

chrome/renderer/webplugin_delegate_pepper.cc

Index: chrome/renderer/webplugin_delegate_pepper.cc
diff --git a/chrome/renderer/webplugin_delegate_pepper.cc b/chrome/renderer/webplugin_delegate_pepper.cc
index 921138f5f633f625fc9ca0fcfa83bb8b30f90185..292874bd4e735e2d7eb61c4f12647b855b3bba30 100644
--- a/chrome/renderer/webplugin_delegate_pepper.cc
+++ b/chrome/renderer/webplugin_delegate_pepper.cc
@@ -50,7 +50,7 @@
 #include "ui/gfx/codec/jpeg_codec.h"
 #include "ui/gfx/gdi_util.h"
 #endif
-#include "printing/native_metafile.h"

[vandebo (ex-Chrome), 2011/02/22 22:18:16]

NativeMetafile is still used.

[dpapad, 2011/02/23 00:44:39]

Done.

+#include "printing/metafile_factory.h"
 #include "third_party/npapi/bindings/npapi_extensions.h"
 #include "third_party/npapi/bindings/npapi_extensions_private.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCursorInfo.h"
@@ -1222,7 +1222,7 @@ bool WebPluginDelegatePepper::VectorPrintPage(int page_number,
   // directly.
   cairo_t* context = canvas->beginPlatformPaint();
   printing::NativeMetafile* metafile =
-      printing::NativeMetafile::FromCairoContext(context);
+      printing::MetafileFactory::GetMetafileFrom(context);

[vandebo (ex-Chrome), 2011/02/22 22:18:16]

This is cairo specific specific, so probably should go into the factory.  Can
you look into the details of what's going on here and then figure out what needs
to be done when we have two different ways to produce PDFs?

   DCHECK(metafile);
   if (metafile) {
     ret = metafile->SetRawData(pdf_output, output_size);
@@ -1231,14 +1231,15 @@ bool WebPluginDelegatePepper::VectorPrintPage(int page_number,
   }
   canvas->endPlatformPaint();
 #elif defined(OS_MACOSX)
-  printing::NativeMetafile metafile;
+  scoped_ptr<printing::NativeMetafile> metafile(
+      printing::MetafileFactory::GetMetafile());
   // Create a PDF metafile and render from there into the passed in context.
-  if (metafile.Init(pdf_output, output_size)) {
+  if (metafile->Init(pdf_output, output_size)) {
     // Flip the transform.
     CGContextSaveGState(canvas);
     CGContextTranslateCTM(canvas, 0, current_printable_area_.height());
     CGContextScaleCTM(canvas, 1.0, -1.0);
-    ret = metafile.RenderPage(1, canvas, current_printable_area_.ToCGRect(),
+    ret = metafile->RenderPage(1, canvas, current_printable_area_.ToCGRect(),
                               true, false, true, true);
     CGContextRestoreGState(canvas);
   }