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/native_metafile_factory.h"

[James Hawkins, 2011/02/28 22:26:45]

nit: The rule for alphabetizing includes is to remove all punctuation
characters, then sort. That would put native_metafile_factory.h before
native_metafile.h.

This applies to a few other files in this CL.

[dpapad, 2011/02/28 23:50:59]

Done.

 #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 91 matching lines @@
 
   base::SharedMemory shared_buf(params.metafile_data_handle, true);
   if (metafile_must_be_valid) {
     if (!shared_buf.Map(params.data_size)) {
       NOTREACHED() << "couldn't map";
       tab_contents()->Stop();
       return;
     }
   }
 
-  scoped_ptr<NativeMetafile> metafile(new NativeMetafile());
+  scoped_ptr<NativeMetafile> metafile(NativeMetafileFactory::CreateMetafile());
   if (metafile_must_be_valid) {
     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,
@@ skipping 279 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