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

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

Issue 10867004: Notify print preview UI if getting capabilityes failed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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
« no previous file with comments | « no previous file | chrome/browser/printing/print_system_task_proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/print_system_task_proxy.h
diff --git a/chrome/browser/printing/print_system_task_proxy.h b/chrome/browser/printing/print_system_task_proxy.h
index cfad8ec05d65ba2ee19d2a5c6d24ca25bcdbc94b..1eab9cba6cdbac91ecad0d12fff3b13fe565680f 100644
--- a/chrome/browser/printing/print_system_task_proxy.h
+++ b/chrome/browser/printing/print_system_task_proxy.h
@@ -64,8 +64,7 @@ class PrintSystemTaskProxy
FRIEND_TEST_ALL_PREFIXES(PrintSystemTaskProxyTest, DetectNoDuplexModeCUPS);
#endif
-#if defined(USE_CUPS)
- static bool GetPrinterCapabilitiesCUPS(
+ bool ParsePrinterCapabilities(
const printing::PrinterCapsAndDefaults& printer_info,
const std::string& printer_name,
bool* set_color_as_default,
@@ -73,20 +72,12 @@ class PrintSystemTaskProxy
int* printer_color_space_for_black,
bool* set_duplex_as_default,
int* default_duplex_setting_value);
-#elif defined(OS_WIN)
- void GetPrinterCapabilitiesWin(
- const printing::PrinterCapsAndDefaults& printer_info,
- bool* set_color_as_default,
- int* printer_color_space_for_color,
- int* printer_color_space_for_black,
- bool* set_duplex_as_default,
- int* default_duplex_setting_value);
-#endif
- void SendDefaultPrinter(const std::string* default_printer,
- const std::string* cloud_print_data);
+ void SendDefaultPrinter(const std::string& default_printer,
+ const std::string& cloud_print_data);
void SetupPrinterList(base::ListValue* printers);
void SendPrinterCapabilities(base::DictionaryValue* settings_info);
+ void SendFailedToGetPrinterCapabilities(const std::string& printer_name);
~PrintSystemTaskProxy();
« no previous file with comments | « no previous file | chrome/browser/printing/print_system_task_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698