| Index: chrome/renderer/print_web_view_helper.cc
|
| diff --git a/chrome/renderer/print_web_view_helper.cc b/chrome/renderer/print_web_view_helper.cc
|
| index 72c7f10bf7d114458c0e74790085931b5a38f6ff..bb97e80afc48e502a2699d6cb1d35b32ada9520a 100644
|
| --- a/chrome/renderer/print_web_view_helper.cc
|
| +++ b/chrome/renderer/print_web_view_helper.cc
|
| @@ -517,14 +517,8 @@ void PrintWebViewHelper::OnPrintForPrintPreview(
|
| return;
|
| }
|
|
|
| - scoped_ptr<PrepareFrameAndViewForPrint> prepare;
|
| - prepare.reset(new PrepareFrameAndViewForPrint(print_pages_params_->params,
|
| - pdf_frame, pdf_element));
|
| - UpdatePrintableSizeInPrintParameters(pdf_frame, pdf_element, prepare.get(),
|
| - &print_pages_params_->params);
|
| -
|
| // Render Pages for printing.
|
| - if (!RenderPagesForPrint(pdf_frame, pdf_element, prepare.get())) {
|
| + if (!RenderPagesForPrint(pdf_frame, pdf_element)) {
|
| LOG(ERROR) << "RenderPagesForPrint failed";
|
| DidFinishPrinting(FAIL_PRINT);
|
| }
|
| @@ -766,7 +760,7 @@ void PrintWebViewHelper::Print(WebKit::WebFrame* frame,
|
| }
|
|
|
| // Render Pages for printing.
|
| - if (!RenderPagesForPrint(frame, node, NULL)) {
|
| + if (!RenderPagesForPrint(frame, node)) {
|
| LOG(ERROR) << "RenderPagesForPrint failed";
|
| DidFinishPrinting(FAIL_PRINT);
|
| }
|
| @@ -838,25 +832,20 @@ bool PrintWebViewHelper::CopyAndPrint(WebKit::WebFrame* web_frame) {
|
| #if defined(OS_MACOSX) || defined(OS_WIN)
|
| bool PrintWebViewHelper::PrintPages(const PrintMsg_PrintPages_Params& params,
|
| WebFrame* frame,
|
| - const WebNode& node,
|
| - PrepareFrameAndViewForPrint* prepare) {
|
| + const WebNode& node) {
|
| PrintMsg_Print_Params print_params = params.params;
|
| - scoped_ptr<PrepareFrameAndViewForPrint> prep_frame_view;
|
| - if (!prepare) {
|
| - prep_frame_view.reset(new PrepareFrameAndViewForPrint(print_params, frame,
|
| - node));
|
| - prepare = prep_frame_view.get();
|
| - }
|
| - UpdatePrintableSizeInPrintParameters(frame, node, prepare, &print_params);
|
| + PrepareFrameAndViewForPrint prep_frame_view(print_params, frame, node);
|
| + UpdatePrintableSizeInPrintParameters(frame, node, &prep_frame_view,
|
| + &print_params);
|
|
|
| - int page_count = prepare->GetExpectedPageCount();
|
| + int page_count = prep_frame_view.GetExpectedPageCount();
|
| if (!page_count)
|
| return false;
|
| Send(new PrintHostMsg_DidGetPrintedPagesCount(routing_id(),
|
| print_params.document_cookie,
|
| page_count));
|
|
|
| - const gfx::Size& canvas_size = prepare->GetPrintCanvasSize();
|
| + const gfx::Size& canvas_size = prep_frame_view.GetPrintCanvasSize();
|
| PrintMsg_PrintPage_Params page_params;
|
| page_params.params = print_params;
|
| if (params.pages.empty()) {
|
| @@ -879,7 +868,7 @@ bool PrintWebViewHelper::PrintPages(const PrintMsg_PrintPages_Params& params,
|
| void PrintWebViewHelper::didStopLoading() {
|
| PrintMsg_PrintPages_Params* params = print_pages_params_.get();
|
| DCHECK(params != NULL);
|
| - PrintPages(*params, print_web_view_->mainFrame(), WebNode(), NULL);
|
| + PrintPages(*params, print_web_view_->mainFrame(), WebNode());
|
| }
|
|
|
| // static - Not anonymous so that platform implementations can use it.
|
| @@ -1192,14 +1181,13 @@ bool PrintWebViewHelper::GetPrintSettingsFromUser(WebKit::WebFrame* frame,
|
|
|
| bool PrintWebViewHelper::RenderPagesForPrint(
|
| WebKit::WebFrame* frame,
|
| - const WebKit::WebNode& node,
|
| - PrepareFrameAndViewForPrint* prepare) {
|
| + const WebKit::WebNode& node) {
|
| PrintMsg_PrintPages_Params print_settings = *print_pages_params_;
|
| if (print_settings.params.selection_only) {
|
| return CopyAndPrint(frame);
|
| } else {
|
| // TODO: Always copy before printing.
|
| - return PrintPages(print_settings, frame, node, prepare);
|
| + return PrintPages(print_settings, frame, node);
|
| }
|
| }
|
|
|
|
|