Print preview not showing if default print is invalid.

chrome/renderer/print_web_view_helper_browsertest.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/print_messages.h"
 #include "chrome/renderer/print_web_view_helper.h"
 #include "chrome/test/base/render_view_test.h"
 #include "printing/print_job_constants.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 327 matching lines @@
       EXPECT_NE(0U, preview_param.a.data_size);
     }
   }
 
   void VerifyPrintFailed(bool did_fail) {
     bool print_failed = (render_thread_.sink().GetUniqueMessageMatching(
         PrintHostMsg_PrintingFailed::ID) != NULL);
     EXPECT_EQ(did_fail, print_failed);
   }
 
+  void VerifyPrintPreviewInvalidPrinterSettings(bool settings_invalid) {
+    bool print_preview_invalid_printer_settings =
+        (render_thread_.sink().GetUniqueMessageMatching(
+            PrintHostMsg_PrintPreviewInvalidPrinterSettings::ID) != NULL);
+    EXPECT_EQ(settings_invalid, print_preview_invalid_printer_settings);
+  }
+
   DISALLOW_COPY_AND_ASSIGN(PrintWebViewHelperPreviewTest);
 };
 
 // Tests that print preview work and sending and receiving messages through
 // that channel all works.
 TEST_F(PrintWebViewHelperPreviewTest, OnPrintPreview) {
   LoadHTML(kHelloWorldHTML);
 
   PrintWebViewHelper::Get(view_)->OnInitiatePrintPreview();
   // Fill in some dummy values.
@@ skipping 63 matching lines @@
 TEST_F(PrintWebViewHelperPreviewTest, OnPrintForPrintPreviewFail) {
   LoadHTML(kPrintPreviewHTML);
 
   // An empty dictionary should fail.
   DictionaryValue empty_dict;
   PrintWebViewHelper::Get(view_)->OnPrintForPrintPreview(empty_dict);
 
   VerifyPrintFailed(true);
   VerifyPagesPrinted(false);
 }
+
+// Tests that when default printer has invalid printer settings, print preview
+// receives error message.
+TEST_F(PrintWebViewHelperPreviewTest, OnPrintPreviewInvalidDefaultPrinter) {

[kmadhusu, 2011/08/30 02:11:27]

How about? 

OnPrintPreviewInvalidDefaultPrinter => OnPrintPreviewUsingInvalidPrinterSettings

[arthurhsu, 2011/08/30 20:43:03]

Done.

+  LoadHTML(kPrintPreviewHTML);
+
+  PrintWebViewHelper::Get(view_)->OnInitiatePrintPreview();
+
+  // Set mock printer to provide invalid settings.
+  render_thread_.printer()->UseInvalidSettings();
+  render_thread_.printer()->ResetPrinter();
+
+  // Fill in some dummy values.
+  DictionaryValue dict;
+  CreatePrintSettingsDictionary(&dict);
+  PrintWebViewHelper::Get(view_)->OnPrintPreview(dict);
+
+  // We should have received invalid printer settings from |printer_|.
+  VerifyPrintPreviewInvalidPrinterSettings(true);
+  EXPECT_EQ(0, render_thread_.print_preview_pages_remaining());

[kmadhusu, 2011/08/30 02:11:27]

print_preview_pages_remaining() return 1 because we haven't rendered any pages
for preview. 

[arthurhsu, 2011/08/30 20:43:03]

It should be zero since it can't tell how many pages are there (there's no
printer settings for it to determine).

+
+  // It should receive the invalid printer settings message only.
+  VerifyPrintPreviewFailed(false);
+
+  VerifyPrintPreviewGenerated(false);

[kmadhusu, 2011/08/30 02:11:27]

nit: Remove the blank line above this statement.

[arthurhsu, 2011/08/30 20:43:03]

Done.

+}
+
+TEST_F(PrintWebViewHelperPreviewTest, OnPrintForPrintPreviewInvalidPrinter) {

[kmadhusu, 2011/08/30 02:11:27]

How about?

OnPrintForPrintPreviewInvalidPrinter =>
OnPrintForPrintPreviewUsingInvalidPrinterSetting

[arthurhsu, 2011/08/30 20:43:03]

Done.

+  LoadHTML(kPrintPreviewHTML);
+
+  // Set mock printer to provide invalid settings.
+  render_thread_.printer()->UseInvalidSettings();
+  render_thread_.printer()->ResetPrinter();

[kmadhusu, 2011/08/30 02:11:27]

Is ResetPrinter() required? I think mockprinter is not shared among tests.

[arthurhsu, 2011/08/30 20:43:03]

Done.

+
+  // Fill in some dummy values.
+  DictionaryValue dict;
+  CreatePrintSettingsDictionary(&dict);
+  PrintWebViewHelper::Get(view_)->OnPrintForPrintPreview(dict);
+
+  VerifyPagesPrinted(false);
+}
+
 #endif  // !defined(OS_CHROMEOS)