Chromium Code Reviews| Index: printing/printing_context.h |
| diff --git a/printing/printing_context.h b/printing/printing_context.h |
| index dd7e058b373444b16ad55f749b7e1885bb5e5b3a..c7184e7fb845ba16075ebfff8ad24f3a766080e1 100644 |
| --- a/printing/printing_context.h |
| +++ b/printing/printing_context.h |
| @@ -51,10 +51,16 @@ class PrintingContext { |
| // default device settings. |
| virtual Result UseDefaultSettings() = 0; |
| - // Updates print settings. |job_settings| contains all print job settings |
| - // information. |ranges| has the new page range settings. |
| - virtual Result UpdatePrintSettings(const base::DictionaryValue& job_settings, |
| - const PageRanges& ranges) = 0; |
| + // Updates printer related settings. |job_settings| contains all print job |
| + // settings information. |ranges| has the new page range settings. |
| + virtual Result UpdatePrinterSettings( |
|
kmadhusu
2011/08/15 16:46:54
How about "UpdatePrintJobSettings"?
vandebo (ex-Chrome)
2011/08/15 19:35:49
I think the reason for changing the name is to dis
|
| + const base::DictionaryValue& job_settings, |
| + const PageRanges& ranges) = 0; |
| + |
| + // Updates Print Settings. |job_settings| contains all print job |
| + // settings information. |ranges| has the new page range settings. |
| + Result UpdatePrintSettings(const base::DictionaryValue& job_settings, |
| + const PageRanges& ranges); |
| // Initializes with predefined settings. |
| virtual Result InitWithSettings(const PrintSettings& settings) = 0; |