Cleanup NativeMetafile (win) interface and EMF class.

chrome/service/service_utility_process_host.h

-// Copyright (c) 2010 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.
 
 #ifndef CHROME_SERVICE_SERVICE_UTILITY_PROCESS_HOST_H_
 #define CHROME_SERVICE_SERVICE_UTILITY_PROCESS_HOST_H_
 #pragma once
 
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif  // defined(OS_WIN)
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/ref_counted.h"
 #include "base/task.h"
 #include "ipc/ipc_channel.h"
 #include "chrome/service/service_child_process_host.h"
-#include "printing/native_metafile.h"
 
 class CommandLine;
 class ScopedTempDir;
 
 namespace base {
 class MessageLoopProxy;
 }  // namespace base
 
 namespace gfx {
 class Rect;
 }  // namespace gfx
 
 namespace printing {
+class Emf;
 struct PageRange;
 struct PrinterCapsAndDefaults;
 }  // namespace printing
 
 // Acts as the service-side host to a utility child process. A
 // utility process is a short-lived sandboxed process that is created to run
 // a specific task.
 class ServiceUtilityProcessHost : public ServiceChildProcessHost {
  public:
   // Consumers of ServiceUtilityProcessHost must implement this interface to
   // get results back.  All functions are called on the thread passed along
   // to ServiceUtilityProcessHost.
   class Client : public base::RefCountedThreadSafe<Client> {
    public:
     Client() {}
 
     // Called when the child process died before a reply was receieved.
     virtual void OnChildDied() {}
 
     // Called when at least one page in the specified PDF has been rendered
     // successfully into |metafile|.
     virtual void OnRenderPDFPagesToMetafileSucceeded(
-        const printing::NativeMetafile& metafile,
+        const printing::Emf& metafile,

[M-A Ruel, 2011/03/17 13:19:21]

I am kind of surprised that you switched back from NativeMetafile to Emf, what's
the exact purpose? 

[vandebo (ex-Chrome), 2011/03/17 23:38:06]

This method doesn't really want to render to a metafile, it wants to render to
an EMF (we already have PDF data and we want to be able to print it).  This work
is in preparation for an additional metafile implementation that internally uses
PDFs. Furthermore, the methods of the metafile that this method wants to use are
becoming EMF specific (they don't apply to a PDF implementation of metafile).

         int highest_rendered_page_number) {}
     // Called when no page in the passed in PDF could be rendered.
     virtual void OnRenderPDFPagesToMetafileFailed() {}
 
     // Called when the printer capabilities and defaults have been
     // retrieved successfully.
     virtual void OnGetPrinterCapsAndDefaultsSucceeded(
         const std::string& printer_name,
         const printing::PrinterCapsAndDefaults& caps_and_defaults) {}
 
@@ skipping 68 matching lines @@
   bool waiting_for_reply_;
   // The path to the temp file where the metafile will be written to.
   FilePath metafile_path_;
   // The temporary folder created for the metafile.
   scoped_ptr<ScopedTempDir> scratch_metafile_dir_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceUtilityProcessHost);
 };
 
 #endif  // CHROME_SERVICE_SERVICE_UTILITY_PROCESS_HOST_H_