printing::Metafile: Simplify OS_MACOSX-specific code path

printing/pdf_metafile_cg_mac.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "printing/pdf_metafile_cg_mac.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 
@@ skipping 139 matching lines @@
       LOG(ERROR) << "Metafile context has " << extra_retain_count
                  << " extra retain(s) on Close";
     }
   }
 #endif
   CGPDFContextClose(context_.get());
   context_.reset();
   return true;
 }
 
-bool PdfMetafileCg::RenderPage(unsigned int page_number,
+/* TODO(caryclark): The set up of PluginInstance::PrintPDFOutput may result in
+   rasterized output.  Even if that flow uses PdfMetafileCg::RenderPage,
+   the drawing of the PDF into the canvas may result in a rasterized output.
+   PDFMetafileSkia::RenderPage should be not implemented as shown and instead
+   should do something like the following CL in PluginInstance::PrintPDFOutput:
+   http://codereview.chromium.org/7200040/diff/1/webkit/plugins/ppapi/ppapi_plugin_instance.cc
+*/
+bool PdfMetafileCg::RenderPage(const std::vector<char>& src_buffer,
+                               unsigned int page_number,
                                CGContextRef context,
                                const CGRect rect,
-                               const MacRenderPageParams& params) const {
-  CGPDFDocumentRef pdf_doc = GetPDFDocument();
+                               const PdfMetafileCg::RenderPageParams& params) {
+  PdfMetafileCg metafile;
+  if (!metafile.InitFromData(src_buffer.data(), src_buffer.size())) {
+    LOG(ERROR) << "Unable to initialize PDF document from data";
+    return false;
+  }
+  CGPDFDocumentRef pdf_doc = metafile.GetPDFDocument();
   if (!pdf_doc) {
     LOG(ERROR) << "Unable to create PDF document from data";
     return false;
   }
   CGPDFPageRef pdf_page = CGPDFDocumentGetPage(pdf_doc, page_number);
   CGRect source_rect = CGPDFPageGetBoxRect(pdf_page, kCGPDFCropBox);
   int pdf_src_rotation = CGPDFPageGetRotationAngle(pdf_page);
   float scaling_factor = 1.0;
   const bool source_is_landscape =
         (source_rect.size.width > source_rect.size.height);
@@ skipping 117 matching lines @@
 
   if (!pdf_doc_.get()) {
     ScopedCFTypeRef<CGDataProviderRef> pdf_data_provider(
         CGDataProviderCreateWithCFData(pdf_data_));
     pdf_doc_.reset(CGPDFDocumentCreateWithProvider(pdf_data_provider));
   }
   return pdf_doc_.get();
 }
 
 }  // namespace printing