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

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

Issue 7621087: Print Preview: Go from event driven print preview back to print preview with sync messages. The s... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix missing lock Created 9 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/renderer/print_web_view_helper_mac.mm ('k') | printing/print_job_constants.h » ('j') | 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/logging.h" 7 #include "base/logging.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/metrics/histogram.h" 9 #include "base/metrics/histogram.h"
10 #include "base/process_util.h" 10 #include "base/process_util.h"
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 page_params.has_visible_overlays = frame->isPageBoxVisible(page_number); 114 page_params.has_visible_overlays = frame->isPageBoxVisible(page_number);
115 115
116 if (!CopyMetafileDataToSharedMem(metafile.get(), 116 if (!CopyMetafileDataToSharedMem(metafile.get(),
117 &(page_params.metafile_data_handle))) { 117 &(page_params.metafile_data_handle))) {
118 page_params.data_size = 0; 118 page_params.data_size = 0;
119 } 119 }
120 120
121 Send(new PrintHostMsg_DidPrintPage(routing_id(), page_params)); 121 Send(new PrintHostMsg_DidPrintPage(routing_id(), page_params));
122 } 122 }
123 123
124 void PrintWebViewHelper::RenderPreviewPage(int page_number) { 124 bool PrintWebViewHelper::RenderPreviewPage(int page_number) {
125 PrintMsg_Print_Params print_params = print_preview_context_.print_params(); 125 PrintMsg_Print_Params print_params = print_preview_context_.print_params();
126 // Calculate the dpi adjustment. 126 // Calculate the dpi adjustment.
127 float scale_factor = static_cast<float>(print_params.desired_dpi / 127 float scale_factor = static_cast<float>(print_params.desired_dpi /
128 print_params.dpi); 128 print_params.dpi);
129 129
130 base::TimeTicks begin_time = base::TimeTicks::Now(); 130 base::TimeTicks begin_time = base::TimeTicks::Now();
131 printing::Metafile* render_page_result = 131 printing::Metafile* render_page_result =
132 RenderPage(print_params, &scale_factor, page_number, true, 132 RenderPage(print_params, &scale_factor, page_number, true,
133 print_preview_context_.frame(), 133 print_preview_context_.frame(),
134 print_preview_context_.metafile()); 134 print_preview_context_.metafile());
135 // In the preview flow, RenderPage will never return a new metafile. 135 // In the preview flow, RenderPage will never return a new metafile.
136 DCHECK_EQ(render_page_result, print_preview_context_.metafile()); 136 DCHECK_EQ(render_page_result, print_preview_context_.metafile());
137 137
138 print_preview_context_.RenderedPreviewPage( 138 print_preview_context_.RenderedPreviewPage(
139 base::TimeTicks::Now() - begin_time); 139 base::TimeTicks::Now() - begin_time);
140 140
141 scoped_ptr<printing::Metafile> page_metafile; 141 scoped_ptr<printing::Metafile> page_metafile;
142 if (print_preview_context_.IsModifiable()) { 142 if (print_preview_context_.IsModifiable()) {
143 page_metafile.reset( 143 page_metafile.reset(
144 print_preview_context_.metafile()->GetMetafileForCurrentPage()); 144 print_preview_context_.metafile()->GetMetafileForCurrentPage());
145 } 145 }
146 PreviewPageRendered(page_number, page_metafile.get()); 146 return PreviewPageRendered(page_number, page_metafile.get());
147 } 147 }
148 148
149 Metafile* PrintWebViewHelper::RenderPage( 149 Metafile* PrintWebViewHelper::RenderPage(
150 const PrintMsg_Print_Params& params, float* scale_factor, int page_number, 150 const PrintMsg_Print_Params& params, float* scale_factor, int page_number,
151 bool is_preview, WebFrame* frame, Metafile* metafile) { 151 bool is_preview, WebFrame* frame, Metafile* metafile) {
152 PageSizeMargins page_layout_in_points; 152 PageSizeMargins page_layout_in_points;
153 GetPageSizeAndMarginsInPoints(frame, page_number, params, 153 GetPageSizeAndMarginsInPoints(frame, page_number, params,
154 &page_layout_in_points); 154 &page_layout_in_points);
155 155
156 int width; 156 int width;
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
287 shared_buf.Unmap(); 287 shared_buf.Unmap();
288 return false; 288 return false;
289 } 289 }
290 shared_buf.GiveToProcess(base::GetCurrentProcessHandle(), shared_mem_handle); 290 shared_buf.GiveToProcess(base::GetCurrentProcessHandle(), shared_mem_handle);
291 shared_buf.Unmap(); 291 shared_buf.Unmap();
292 292
293 Send(new PrintHostMsg_DuplicateSection(routing_id(), *shared_mem_handle, 293 Send(new PrintHostMsg_DuplicateSection(routing_id(), *shared_mem_handle,
294 shared_mem_handle)); 294 shared_mem_handle));
295 return true; 295 return true;
296 } 296 }
OLDNEW
« no previous file with comments | « chrome/renderer/print_web_view_helper_mac.mm ('k') | printing/print_job_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698