Chromium Code Reviews| 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..446fe5be4d34868ffb3c6e17216c1f0ae3a26f32 100644 |
| --- a/chrome/browser/printing/print_job_worker.h |
| +++ b/chrome/browser/printing/print_job_worker.h |
| @@ -14,6 +14,8 @@ |
| #include "printing/print_job_constants.h" |
| #include "printing/printing_context.h" |
| +#include "chrome/browser/printing/printer_query.h" |
|
Vitaly Buka (NO REVIEWS)
2014/12/09 23:46:48
wrong include location
|
| + |
| namespace base { |
| class DictionaryValue; |
| } |
| @@ -39,10 +41,13 @@ 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/) |
| void GetSettings( |
| - bool ask_user_for_settings, |
| + PrinterQuery::GetSettingsAskParam ask_settings_type, |
| int document_page_count, |
| bool has_selection, |
| MarginType margin_type); |
| @@ -115,6 +120,9 @@ class PrintJobWorker { |
| // back into the IO thread for GetSettingsDone(). |
| void GetSettingsWithUIDone(PrintingContext::Result result); |
| + // Called on the UI thread to ask the system to show a print dialog. |
| + void ShowSystemDialog(); |
| + |
| // Called on the UI thread to update the print settings. |
| void UpdatePrintSettings(scoped_ptr<base::DictionaryValue> new_settings); |