Notify print preview UI if getting capabilityes failed.

chrome/browser/printing/print_system_task_proxy.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PRINTING_PRINT_SYSTEM_TASK_PROXY_H_
 #define CHROME_BROWSER_PRINTING_PRINT_SYSTEM_TASK_PROXY_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
@@ skipping 46 matching lines @@
  private:
   friend struct content::BrowserThread::DeleteOnThread<
       content::BrowserThread::UI>;
   friend class base::DeleteHelper<PrintSystemTaskProxy>;
 
 #if defined(UNIT_TEST) && defined(USE_CUPS)
   FRIEND_TEST_ALL_PREFIXES(PrintSystemTaskProxyTest, DetectDuplexModeCUPS);
   FRIEND_TEST_ALL_PREFIXES(PrintSystemTaskProxyTest, DetectNoDuplexModeCUPS);
 #endif
 
-#if defined(USE_CUPS)
-  static bool GetPrinterCapabilitiesCUPS(
+  bool ParsePrinterCapabilities(

[Robert Toscano, 2012/08/22 01:05:31]

Did you mean to remove the "static" modifier?

[Vitaly Buka (NO REVIEWS), 2012/08/22 02:58:02]

Yes. CUPS can be static, WIN can't. But static does not worth splitting.

       const printing::PrinterCapsAndDefaults& printer_info,
       const std::string& printer_name,
       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);
-#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();
 
   base::WeakPtr<PrintPreviewHandler> handler_;
 
   scoped_refptr<printing::PrintBackend> print_backend_;
 
   bool has_logged_printers_count_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintSystemTaskProxy);
 };
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_SYSTEM_TASK_PROXY_H_