Adding CreateFromData to NativeMetafileFactory

printing/native_metafile_factory.cc

 // Copyright (c) 2011 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/native_metafile_factory.h"
 
 #if defined(OS_WIN)
 #include "printing/emf_win.h"
 #elif defined(OS_MACOSX)
 #include "printing/pdf_metafile_mac.h"
 #elif defined(OS_POSIX)
 #include "printing/pdf_ps_metafile_cairo.h"
 #endif
 
 namespace printing {
 
 // static
 printing::NativeMetafile* NativeMetafileFactory::Create() {
+  NativeMetafile* metafile;
 #if defined(OS_WIN)
-  return new printing::Emf;
+  metafile = new printing::Emf;

[vandebo (ex-Chrome), 2011/03/24 20:11:55]

This code is duplicated.  You should pull it out into a local function.

[dpapad, 2011/03/24 21:08:14]

Done.

 #elif defined(OS_MACOSX)
-  return new printing::PdfMetafile;
+  metafile = new printing::PdfMetafile;
 #elif defined(OS_POSIX)
-  return new printing::PdfPsMetafile;
+  metafile = new printing::PdfPsMetafile;
 #endif
+  if (!metafile->Init()) {
+    delete metafile;
+    return NULL;
+  }
+  return metafile;
+}
+
+// static
+printing::NativeMetafile* NativeMetafileFactory::CreateFromData(
+    const void* src_buffer, uint32 src_buffer_size) {
+  NativeMetafile* metafile;
+#if defined(OS_WIN)
+  metafile = new printing::Emf;
+#elif defined(OS_MACOSX)
+  metafile = new printing::PdfMetafile;
+#elif defined(OS_POSIX)
+  metafile = new printing::PdfPsMetafile;
+#endif
+  if (!metafile->InitFromData(src_buffer, src_buffer_size)) {
+    delete metafile;
+    return NULL;
+  }
+  return metafile;
 }
 
 }  // namespace printing