Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6034)

Unified Diff: chrome/browser/ui/webui/print_preview/print_preview_handler.cc

Issue 568633002: Extracted MetafilePlayer interface from printing::MetafilePlayer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@no_expose
Patch Set: Thu Sep 11 16:10:13 PDT 2014 Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/print_preview/print_preview_handler.cc
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_handler.cc b/chrome/browser/ui/webui/print_preview/print_preview_handler.cc
index 06894a481c3d33cff4af311905f1c117a9fa7a35..2b3ac6b1989b385a55ed30d08521a1d03447f22e 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_handler.cc
+++ b/chrome/browser/ui/webui/print_preview/print_preview_handler.cc
@@ -246,7 +246,9 @@ void PrintToPdfCallback(const scoped_refptr<base::RefCountedBytes>& data,
DCHECK_CURRENTLY_ON(BrowserThread::FILE);
printing::PdfMetafileSkia metafile;
metafile.InitFromData(static_cast<const void*>(data->front()), data->size());
- metafile.SaveTo(path);
+ base::File file(path,
+ base::File::FLAG_CREATE_ALWAYS || base::File::FLAG_WRITE);
+ metafile.SaveTo(&file);
if (!pdf_file_saved_closure.is_null())
pdf_file_saved_closure.Run();
}

Powered by Google App Engine
This is Rietveld 408576698