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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: components/printing/renderer/print_web_view_helper_linux.cc
diff --git a/components/printing/renderer/print_web_view_helper_linux.cc b/components/printing/renderer/print_web_view_helper_linux.cc
index f09e4b22c517fac502fc95c710112e474bef9f21..dac616a6bb6cabe1ca289af705c45f4789d0f31a 100644
--- a/components/printing/renderer/print_web_view_helper_linux.cc
+++ b/components/printing/renderer/print_web_view_helper_linux.cc
@@ -54,11 +54,9 @@ bool PrintWebViewHelper::PrintPagesNative(blink::WebLocalFrame* frame,
if (printed_pages.empty())
return false;
- PrintMsg_PrintPage_Params page_params;
- page_params.params = params.params;
for (int page_number : printed_pages) {
- page_params.page_number = page_number;
- PrintPageInternal(page_params, frame, &metafile, nullptr, nullptr, nullptr);
+ PrintPageInternal(params.params, page_number, frame, &metafile, nullptr,
+ nullptr, nullptr);
}
// blink::printEnd() for PDF should be called before metafile is closed.
« 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