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

Side by Side Diff: chrome/renderer/print_web_view_helper_linux.cc

Issue 8566055: PrintPreview: [Cleanup] Remove unwanted function calls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: '' Created 9 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/renderer/print_web_view_helper.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/renderer/print_web_view_helper.h" 5 #include "chrome/renderer/print_web_view_helper.h"
6 6
7 #include "base/file_descriptor_posix.h" 7 #include "base/file_descriptor_posix.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 print_preview_context_.generate_draft_pages()){ 50 print_preview_context_.generate_draft_pages()){
51 DCHECK(!draft_metafile.get()); 51 DCHECK(!draft_metafile.get());
52 draft_metafile.reset( 52 draft_metafile.reset(
53 print_preview_context_.metafile()->GetMetafileForCurrentPage()); 53 print_preview_context_.metafile()->GetMetafileForCurrentPage());
54 } 54 }
55 return PreviewPageRendered(page_number, draft_metafile.get()); 55 return PreviewPageRendered(page_number, draft_metafile.get());
56 } 56 }
57 57
58 bool PrintWebViewHelper::PrintPages(const PrintMsg_PrintPages_Params& params, 58 bool PrintWebViewHelper::PrintPages(const PrintMsg_PrintPages_Params& params,
59 WebFrame* frame, 59 WebFrame* frame,
60 const WebNode& node, 60 const WebNode& node) {
61 PrepareFrameAndViewForPrint* prepare) {
62 printing::NativeMetafile metafile; 61 printing::NativeMetafile metafile;
63 if (!metafile.Init()) 62 if (!metafile.Init())
64 return false; 63 return false;
65 64
66 scoped_ptr<PrepareFrameAndViewForPrint> prep_frame_view; 65 PrepareFrameAndViewForPrint prep_frame_view(params.params, frame, node);
67 if (!prepare) { 66 int page_count = 0;
68 prep_frame_view.reset(new PrepareFrameAndViewForPrint(params.params, frame, 67 if (!RenderPages(params, frame, node, &page_count, &prep_frame_view,
69 node)); 68 &metafile)) {
70 prepare = prep_frame_view.get(); 69 return false;
71 } 70 }
72 int page_count = 0;
73 if (!RenderPages(params, frame, node, &page_count, prepare, &metafile))
74 return false;
75 71
76 // Get the size of the resulting metafile. 72 // Get the size of the resulting metafile.
77 uint32 buf_size = metafile.GetDataSize(); 73 uint32 buf_size = metafile.GetDataSize();
78 DCHECK_GT(buf_size, 0u); 74 DCHECK_GT(buf_size, 0u);
79 75
80 #if defined(OS_CHROMEOS) 76 #if defined(OS_CHROMEOS)
81 int sequence_number = -1; 77 int sequence_number = -1;
82 base::FileDescriptor fd; 78 base::FileDescriptor fd;
83 79
84 // Ask the browser to open a file for us. 80 // Ask the browser to open a file for us.
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 // The scale factor on Linux is 1. 214 // The scale factor on Linux is 1.
219 PrintHeaderAndFooter(canvas.get(), params.page_number + 1, 215 PrintHeaderAndFooter(canvas.get(), params.page_number + 1,
220 print_preview_context_.total_page_count(), 1, 216 print_preview_context_.total_page_count(), 1,
221 page_layout_in_points, *header_footer_info_); 217 page_layout_in_points, *header_footer_info_);
222 } 218 }
223 219
224 // Done printing. Close the device context to retrieve the compiled metafile. 220 // Done printing. Close the device context to retrieve the compiled metafile.
225 if (!metafile->FinishPage()) 221 if (!metafile->FinishPage())
226 NOTREACHED() << "metafile failed"; 222 NOTREACHED() << "metafile failed";
227 } 223 }
OLDNEW
« no previous file with comments | « chrome/renderer/print_web_view_helper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698