Create a Factory for NativeMetafile.

chrome/plugin/webplugin_delegate_stub.cc

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// 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/plugin/webplugin_delegate_stub.h"
 
 #include "build/build_config.h"
 
 #include "base/command_line.h"
+#include "base/scoped_ptr.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/plugin_messages.h"
 #include "chrome/plugin/npobject_stub.h"
 #include "chrome/plugin/plugin_channel.h"
 #include "chrome/plugin/plugin_thread.h"
 #include "chrome/plugin/webplugin_proxy.h"
+#include "printing/native_metafile_factory.h"

[Lei Zhang, 2011/03/02 00:05:15]

nit: new includes are Windows only. Same goes for:
chrome/renderer/webplugin_delegate_proxy.cc
chrome/utility/utility_thread.cc
chrome/service/service_utility_process_host.cc

[dpapad, 2011/03/02 01:17:18]

Done. Also I think that the same holds for the already existing include
"printing/native_metafile.h" (for all these files). So i moved this as well
inside the conditional includes.

 #include "printing/native_metafile.h"
 #include "third_party/npapi/bindings/npapi.h"
 #include "third_party/npapi/bindings/npruntime.h"
 #include "skia/ext/platform_device.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebBindings.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCursorInfo.h"
 #include "webkit/plugins/npapi/webplugin_delegate_impl.h"
 #include "webkit/glue/webcursor.h"
 
 #if defined(ENABLE_GPU)
@@ skipping 247 matching lines @@
   webplugin_->Paint(damaged_rect);
 }
 
 void WebPluginDelegateStub::OnDidPaint() {
   webplugin_->DidPaint();
 }
 
 void WebPluginDelegateStub::OnPrint(base::SharedMemoryHandle* shared_memory,
                                     uint32* size) {
 #if defined(OS_WIN)
-  printing::NativeMetafile metafile;
-  if (!metafile.CreateDc(NULL, NULL)) {
+  scoped_ptr<printing::NativeMetafile> metafile(
+      printing::NativeMetafileFactory::CreateMetafile());
+  if (!metafile->CreateDc(NULL, NULL)) {
     NOTREACHED();
     return;
   }
-  HDC hdc = metafile.hdc();
+  HDC hdc = metafile->hdc();
   skia::PlatformDevice::InitializeDC(hdc);
   delegate_->Print(hdc);
-  if (!metafile.CloseDc()) {
+  if (!metafile->CloseDc()) {
     NOTREACHED();
     return;
   }
 
-  *size = metafile.GetDataSize();
+  *size = metafile->GetDataSize();
   DCHECK(*size);
   base::SharedMemory shared_buf;
   CreateSharedBuffer(*size, &shared_buf, shared_memory);
 
   // Retrieve a copy of the data.
-  bool success = metafile.GetData(shared_buf.memory(), *size);
+  bool success = metafile->GetData(shared_buf.memory(), *size);
   DCHECK(success);
 #else
   // TODO(port): plugin printing.
   NOTIMPLEMENTED();
 #endif
 }
 
 void WebPluginDelegateStub::OnUpdateGeometry(
     const PluginMsg_UpdateGeometry_Param& param) {
   webplugin_->UpdateGeometry(
@@ skipping 160 matching lines @@
   webplugin_->OnResourceCreated(resource_id, resource_client);
 }
 
 #if defined(OS_MACOSX)
 void WebPluginDelegateStub::OnSetFakeAcceleratedSurfaceWindowHandle(
     gfx::PluginWindowHandle window) {
   delegate_->set_windowed_handle(window);
 }
 #endif