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

Unified Diff: android_webview/browser/aw_print_manager.cc

Issue 2770713003: Enable page selection for WebView printing (Closed)
Patch Set: enable IsAskPrintSettingsEnabled Created 3 years, 9 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: android_webview/browser/aw_print_manager.cc
diff --git a/android_webview/browser/aw_print_manager.cc b/android_webview/browser/aw_print_manager.cc
index 6aa43f476a3e90271c618d2beeccfcb42cd2d979..31defb6d7980e537aae6f795a2af4414a5072cd3 100644
--- a/android_webview/browser/aw_print_manager.cc
+++ b/android_webview/browser/aw_print_manager.cc
@@ -5,7 +5,6 @@
#include "android_webview/browser/aw_print_manager.h"
#include "components/printing/browser/print_manager_utils.h"
-#include "components/printing/common/print_messages.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/render_frame_host.h"
@@ -53,6 +52,8 @@ bool AwPrintManager::OnMessageReceived(
IPC_BEGIN_MESSAGE_MAP_WITH_PARAM(AwPrintManager, message, render_frame_host)
IPC_MESSAGE_HANDLER_WITH_PARAM_DELAY_REPLY(
PrintHostMsg_GetDefaultPrintSettings, OnGetDefaultPrintSettings)
+ IPC_MESSAGE_HANDLER_WITH_PARAM_DELAY_REPLY(PrintHostMsg_ScriptedPrint,
+ OnScriptedPrint)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
return handled ? true
@@ -71,4 +72,17 @@ void AwPrintManager::OnGetDefaultPrintSettings(
render_frame_host->Send(reply_msg);
}
+void AwPrintManager::OnScriptedPrint(
+ content::RenderFrameHost* render_frame_host,
+ const PrintHostMsg_ScriptedPrint_Params& scripted_params,
+ IPC::Message* reply_msg) {
+ DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+ PrintMsg_PrintPages_Params params;
+ printing::RenderParamsFromPrintSettings(settings_, &params.params);
+ params.params.document_cookie = scripted_params.cookie;
+ params.pages = printing::PageRange::GetPages(settings_.ranges());
+ PrintHostMsg_UpdatePrintSettings::WriteReplyParams(reply_msg, params, false);
Lei Zhang 2017/03/22 22:26:43 Shouldn't this be PrintHostMsg_ScriptedPrint::Writ
Shimi Zhang 2017/03/23 23:21:56 Done.
+ render_frame_host->Send(reply_msg);
+}
+
} // namespace android_webview

Powered by Google App Engine
This is Rietveld 408576698