Create a Factory for NativeMetafile.

chrome/browser/printing/print_view_manager.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 "chrome/browser/printing/print_view_manager.h"
 
 #include "base/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/browser/printing/print_job_manager.h"
 #include "chrome/browser/printing/printer_query.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/notification_details.h"
 #include "chrome/common/notification_source.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/render_messages_params.h"
 #include "grit/generated_resources.h"
-#include "printing/native_metafile.h"
+#include "printing/metafile_factory.h"
 #include "printing/printed_document.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using base::TimeDelta;
 
 namespace printing {
 
 PrintViewManager::PrintViewManager(TabContents* tab_contents)
     : TabContentsObserver(tab_contents),
       number_pages_(0),
@@ skipping 85 matching lines @@
   base::SharedMemory shared_buf(params.metafile_data_handle, true);
   if (!shared_buf.Map(params.data_size)) {
     NOTREACHED() << "couldn't map";
     tab_contents()->Stop();
     return;
   }
 
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   NOTIMPLEMENTED() << " this printing code doesn't quite work yet.";
 #else
-  scoped_ptr<NativeMetafile> metafile(new NativeMetafile());
+  scoped_ptr<NativeMetafile> metafile(printing::MetafileFactory::GetMetafile());

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

We're already in the printing namespace, you probably don't need printing::

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

Done.

   if (!metafile->Init(shared_buf.memory(), params.data_size)) {
     NOTREACHED() << "Invalid metafile header";
     tab_contents()->Stop();
     return;
   }
 
   // Update the rendered document. It will send notifications to the listener.
   document->SetPage(params.page_number,
                     metafile.release(),
                     params.actual_shrink,
@@ skipping 277 matching lines @@
     return false;
   }
 
   // Settings are already loaded. Go ahead. This will set
   // print_job_->is_job_pending() to true.
   print_job_->StartPrinting();
   return true;
 }
 
 }  // namespace printing