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

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

Issue 740983002: Implement window.print() on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: WIP - removing enums and new methods Created 6 years 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/print_job_worker.h
diff --git a/chrome/browser/printing/print_job_worker.h b/chrome/browser/printing/print_job_worker.h
index a8378bb2353de2cd98746c087bfb5e02a595bba6..cadd2c62f030c5f8dd77d5afacbf32c4b376215c 100644
--- a/chrome/browser/printing/print_job_worker.h
+++ b/chrome/browser/printing/print_job_worker.h
@@ -9,6 +9,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
#include "base/threading/thread.h"
+#include "chrome/browser/printing/printer_query.h"
#include "content/public/browser/browser_thread.h"
#include "printing/page_number.h"
#include "printing/print_job_constants.h"
@@ -39,13 +40,18 @@ class PrintJobWorker {
void SetNewOwner(PrintJobWorkerOwner* new_owner);
- // Initializes the print settings. If |ask_user_for_settings| is true, a
- // Print... dialog box will be shown to ask the user his preference.
+ // Initializes the print settings. If |ask_settings_type| is ASK_USER, a
+ // Print... dialog box will be shown by the browser to ask the user his
+ // preference. If it is SYSTEM_SPECIFIC, showing that dialog box will be
+ // delegated to the system. It's used for script printing on android
+ // (see https://codereview.chromium.org/740983002/)
+ // (TODO: doc)
void GetSettings(
- bool ask_user_for_settings,
+ PrinterQuery::GetSettingsAskParam ask_settings_type,
int document_page_count,
bool has_selection,
- MarginType margin_type);
+ MarginType margin_type,
+ bool is_scripted);
// Set the new print settings.
void SetSettings(scoped_ptr<base::DictionaryValue> new_settings);
@@ -108,7 +114,8 @@ class PrintJobWorker {
// but sticks with this for consistency.
void GetSettingsWithUI(
int document_page_count,
- bool has_selection);
+ bool has_selection,
+ bool is_scripted);
// The callback used by PrintingContext::GetSettingsWithUI() to notify this
// object that the print settings are set. This is needed in order to bounce

Powered by Google App Engine
This is Rietveld 408576698