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

Unified Diff: chrome/browser/printing/print_system_task_proxy_unittest.cc

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
Index: chrome/browser/printing/print_system_task_proxy_unittest.cc
diff --git a/chrome/browser/printing/print_system_task_proxy_unittest.cc b/chrome/browser/printing/print_system_task_proxy_unittest.cc
index 9876583b0c2f949abec29e715d6c21747e335296..ee6321b36ac2af0736ab2e9f2b630a2755b5e524 100644
--- a/chrome/browser/printing/print_system_task_proxy_unittest.cc
+++ b/chrome/browser/printing/print_system_task_proxy_unittest.cc
@@ -250,15 +250,18 @@ TEST(PrintSystemTaskProxyTest, DetectDuplexModeCUPS) {
int printer_color_space_for_black = printing::UNKNOWN_COLOR_MODEL;
int default_duplex_setting_value = printing::UNKNOWN_DUPLEX_MODE;
- bool res = PrintSystemTaskProxy::GetPrinterCapabilitiesCUPS(
+ scoped_refptr<PrintSystemTaskProxy> proxy(
+ new PrintSystemTaskProxy(base::WeakPtr<PrintPreviewHandler>(), NULL,
+ false));
+
+ ASSERT_TRUE(proxy->ParsePrinterCapabilities(
printer_info,
"InvalidPrinter",
&set_color_as_default,
&printer_color_space_for_color,
&printer_color_space_for_black,
&set_duplex_as_default,
- &default_duplex_setting_value);
- ASSERT_TRUE(res);
+ &default_duplex_setting_value));
EXPECT_FALSE(set_duplex_as_default);
EXPECT_EQ(printing::SIMPLEX, default_duplex_setting_value);
}
@@ -277,15 +280,18 @@ TEST(PrintSystemTaskProxyTest, DetectNoDuplexModeCUPS) {
int printer_color_space_for_black = printing::UNKNOWN_COLOR_MODEL;
int default_duplex_setting_value = printing::UNKNOWN_DUPLEX_MODE;
- bool res = PrintSystemTaskProxy::GetPrinterCapabilitiesCUPS(
+ scoped_refptr<PrintSystemTaskProxy> proxy(
+ new PrintSystemTaskProxy(base::WeakPtr<PrintPreviewHandler>(), NULL,
+ false));
+
+ ASSERT_TRUE(proxy->ParsePrinterCapabilities(
printer_info,
"InvalidPrinter",
&set_color_as_default,
&printer_color_space_for_color,
&printer_color_space_for_black,
&set_duplex_as_default,
- &default_duplex_setting_value);
- ASSERT_TRUE(res);
+ &default_duplex_setting_value));
EXPECT_FALSE(set_duplex_as_default);
EXPECT_EQ(printing::UNKNOWN_DUPLEX_MODE, default_duplex_setting_value);
}
« no previous file with comments | « chrome/browser/printing/print_system_task_proxy.cc ('k') | chrome/browser/resources/print_preview/native_layer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698