Unifying NativeMetafile class interface (as much as possible) for Linux, Mac, Win

printing/printed_document.cc

Index: printing/printed_document.cc
diff --git a/printing/printed_document.cc b/printing/printed_document.cc
index 3d8f156bafcc1aa76dea54d5520bcbff927012dd..e6c21c7218c36e8532eed79cb969f77d7bcba2b5 100644
--- a/printing/printed_document.cc
+++ b/printing/printed_document.cc
@@ -258,8 +258,8 @@ void PrintedDocument::DebugDump(const PrintedPage& page) {
   filename += ASCIIToUTF16(StringPrintf("%02d", page.page_number()));
   filename += ASCIIToUTF16("_.emf");
 #if defined(OS_WIN)
-  page.native_metafile()->SaveTo(
-      g_debug_dump_info.Get().debug_dump_path.Append(filename).value());
+  page.native_metafile()->SaveTo(FilePath(
+      g_debug_dump_info.Get().debug_dump_path.Append(filename).value()));

[Lei Zhang, 2011/03/04 12:02:36]

I haven't looked closely, but I suspect
g_debug_dump_info.Get().debug_dump_path.Append(filename) is a FilePath. You
don't have to get its value() and construct another FilePath.

[dpapad, 2011/03/04 19:19:43]

Done.

 #else  // OS_WIN
   NOTIMPLEMENTED();  // TODO: convert SaveTo to accept a FilePath
 #endif  // OS_WIN