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

Side by Side Diff: components/printing/renderer/print_web_view_helper_linux.cc

Issue 2806083002: Remove PrintMsg_PrintPage_Params. (Closed)
Patch Set: rebase Created 3 years, 8 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "components/printing/renderer/print_web_view_helper.h" 5 #include "components/printing/renderer/print_web_view_helper.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "build/build_config.h" 10 #include "build/build_config.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 bool PrintWebViewHelper::PrintPagesNative(blink::WebLocalFrame* frame, 47 bool PrintWebViewHelper::PrintPagesNative(blink::WebLocalFrame* frame,
48 int page_count) { 48 int page_count) {
49 PdfMetafileSkia metafile(PDF_SKIA_DOCUMENT_TYPE); 49 PdfMetafileSkia metafile(PDF_SKIA_DOCUMENT_TYPE);
50 CHECK(metafile.Init()); 50 CHECK(metafile.Init());
51 51
52 const PrintMsg_PrintPages_Params& params = *print_pages_params_; 52 const PrintMsg_PrintPages_Params& params = *print_pages_params_;
53 std::vector<int> printed_pages = GetPrintedPages(params, page_count); 53 std::vector<int> printed_pages = GetPrintedPages(params, page_count);
54 if (printed_pages.empty()) 54 if (printed_pages.empty())
55 return false; 55 return false;
56 56
57 PrintMsg_PrintPage_Params page_params;
58 page_params.params = params.params;
59 for (int page_number : printed_pages) { 57 for (int page_number : printed_pages) {
60 page_params.page_number = page_number; 58 PrintPageInternal(params.params, page_number, frame, &metafile, nullptr,
61 PrintPageInternal(page_params, frame, &metafile, nullptr, nullptr, nullptr); 59 nullptr, nullptr);
62 } 60 }
63 61
64 // blink::printEnd() for PDF should be called before metafile is closed. 62 // blink::printEnd() for PDF should be called before metafile is closed.
65 FinishFramePrinting(); 63 FinishFramePrinting();
66 64
67 metafile.FinishDocument(); 65 metafile.FinishDocument();
68 66
69 #if defined(OS_ANDROID) 67 #if defined(OS_ANDROID)
70 int sequence_number = -1; 68 int sequence_number = -1;
71 base::FileDescriptor fd; 69 base::FileDescriptor fd;
(...skipping 25 matching lines...) Expand all
97 Send(new PrintHostMsg_DidPrintPage(routing_id(), printed_page_params)); 95 Send(new PrintHostMsg_DidPrintPage(routing_id(), printed_page_params));
98 // Send the rest of the pages with an invalid metafile handle. 96 // Send the rest of the pages with an invalid metafile handle.
99 printed_page_params.metafile_data_handle.fd = -1; 97 printed_page_params.metafile_data_handle.fd = -1;
100 } 98 }
101 return true; 99 return true;
102 #endif // defined(OS_ANDROID) 100 #endif // defined(OS_ANDROID)
103 } 101 }
104 #endif // BUILDFLAG(ENABLE_BASIC_PRINTING) 102 #endif // BUILDFLAG(ENABLE_BASIC_PRINTING)
105 103
106 } // namespace printing 104 } // namespace printing
OLDNEW
« no previous file with comments | « components/printing/renderer/print_web_view_helper.cc ('k') | components/printing/renderer/print_web_view_helper_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698