Cleanup NativeMetafile (win) interface and EMF class.

printing/emf_win.h

 // 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 PRINTING_EMF_WIN_H_
 #define PRINTING_EMF_WIN_H_
 
 #include <windows.h>
 #include <vector>
 
@@ skipping 11 matching lines @@
 
 namespace printing {
 
 // Simple wrapper class that manage an EMF data stream and its virtual HDC.
 class Emf : public NativeMetafile {
  public:
   class Record;
   class Enumerator;
   struct EnumerationContext;
 
+  // Generates a virtual HDC that will record every GDI commands and compile
+  // it in a EMF data stream.
+  Emf();
   virtual ~Emf();
 
+  // Generates a new metafile that will record every GDI command, and will
+  // be saved to |metafile_path|.
+  virtual bool InitToFile(const FilePath& metafile_path);
+
+  // Initializes the Emf with the data in |metafile_path|.
+  virtual bool InitFromFile(const FilePath& metafile_path);
+
   // NativeMetafile methods.
-  virtual bool Init() { return true; }
+  virtual bool Init();
   virtual bool InitFromData(const void* src_buffer, uint32 src_buffer_size);
 
   virtual skia::PlatformDevice* StartPageForVectorCanvas(
       const gfx::Size& page_size, const gfx::Point& content_origin,
       const float& scale_factor);
   // Inserts a custom GDICOMMENT records indicating StartPage/EndPage calls
   // (since StartPage and EndPage do not work in a metafile DC). Only valid
   // when hdc_ is non-NULL. |page_size| and |content_origin| are ignored.
   // |scale_factor|  must be 1.0.
   virtual bool StartPage(const gfx::Size& page_size,
@@ skipping 15 matching lines @@
 
   virtual unsigned int GetPageCount() const {
     // TODO(dpapad): count the number of times StartPage() is called
     return 1;
   }
 
   virtual HDC context() const {
     return hdc_;
   }
 
-  virtual bool CreateDc(HDC sibling, const RECT* rect);
-  virtual bool CreateFileBackedDc(HDC sibling,
-                                  const RECT* rect,
-                                  const FilePath& path);
-  virtual bool CreateFromFile(const FilePath& file_path);
-
-  virtual void CloseEmf();
-
   virtual bool Playback(HDC hdc, const RECT* rect) const;
   virtual bool SafePlayback(HDC hdc) const;
 
   virtual bool GetData(std::vector<uint8>* buffer) const;
 
   virtual HENHMETAFILE emf() const {
     return emf_;
   }
 
- protected:
-  Emf();
-
  private:
-  friend class NativeMetafileFactory;
-  FRIEND_TEST_ALL_PREFIXES(EmfTest, DC);
-  FRIEND_TEST_ALL_PREFIXES(EmfTest, FileBackedDC);
-  FRIEND_TEST_ALL_PREFIXES(EmfPrintingTest, Enumerate);
-  FRIEND_TEST_ALL_PREFIXES(EmfPrintingTest, PageBreak);
-
   // Playbacks safely one EMF record.
   static int CALLBACK SafePlaybackProc(HDC hdc,
                                        HANDLETABLE* handle_table,
                                        const ENHMETARECORD* record,
                                        int objects_count,
                                        LPARAM param);
 
   // Compiled EMF data handle.
   HENHMETAFILE emf_;
 
@@ skipping 67 matching lines @@
   std::vector<Record> items_;
 
   EnumerationContext context_;
 
   DISALLOW_COPY_AND_ASSIGN(Enumerator);
 };
 
 }  // namespace printing
 
 #endif  // PRINTING_EMF_WIN_H_