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

Unified Diff: chrome/browser/printing/printing_message_filter.h

Issue 6533006: Print Preview: Hook up the print button to initiate printing without displaying a print dialog. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: '' Created 9 years, 10 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: chrome/browser/printing/printing_message_filter.h
diff --git a/chrome/browser/printing/printing_message_filter.h b/chrome/browser/printing/printing_message_filter.h
index 6a41742a37bbbeb1f0f5dded77f5557d166e8027..809ba830ac027d117d57e6126f2d2720d3fc0219 100644
--- a/chrome/browser/printing/printing_message_filter.h
+++ b/chrome/browser/printing/printing_message_filter.h
@@ -12,7 +12,9 @@
#include "base/shared_memory.h"
#endif
+class DictionaryValue;
struct ViewHostMsg_ScriptedPrint_Params;
+struct ViewMsg_Print_Params;
namespace printing {
class PrinterQuery;
@@ -66,6 +68,18 @@ class PrintingMessageFilter : public BrowserMessageFilter {
int routing_id,
IPC::Message* reply_msg);
+ // Get the current print settings right here synchronously.
jam 2011/03/02 18:25:53 nit: here and below, no need to comment IPC dispat
kmadhusu 2011/03/04 19:39:02 Done.
+ void OnGetCurrentPrintSettings(int document_cookie,
Lei Zhang 2011/03/03 00:22:44 You don't need |document_cookie| here or below sin
kmadhusu 2011/03/04 19:39:02 Removed this message handler.
+ ViewMsg_Print_Params* params);
+
+ // Update current print settings with new |job_settings|.
+ void OnUpdatePrintSettings(int document_cookie,
kmadhusu 2011/03/04 19:39:02 I need document_cookie here.
+ const DictionaryValue& job_settings,
+ IPC::Message* reply_msg);
+ void OnUpdatePrintSettingsReply(
+ scoped_refptr<printing::PrinterQuery> printer_query,
+ IPC::Message* reply_msg);
+
printing::PrintJobManager* print_job_manager_;
bool cloud_print_enabled_;

Powered by Google App Engine
This is Rietveld 408576698