Block printing from blocked popup.

chrome/renderer/print_web_view_helper_browsertest.cc

 // 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.
 
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/print_messages.h"
 #include "chrome/renderer/print_web_view_helper.h"
 #include "chrome/test/base/chrome_render_view_test.h"
 #include "content/public/renderer/render_view.h"
 #include "printing/print_job_constants.h"
@@ skipping 170 matching lines @@
   LoadHTML(kPrintWithJSHTML);
   LoadHTML(kPrintWithJSHTML);
   LoadHTML(kPrintWithJSHTML);
   VerifyPagesPrinted(false);
 
   // Pretend user will print. (but printing is blocked.)
   chrome_render_thread_->set_print_dialog_user_response(true);
   LoadHTML(kPrintWithJSHTML);
   VerifyPagesPrinted(false);
 
-  // Unblock script initiated printing and verify printing works.
+  // Unblock based on count.
   PrintWebViewHelper::Get(view_)->ResetScriptedPrintCount();

[Lei Zhang, 2012/03/19 20:19:32]

You are missing a test here to make sure resetting the count works.

[Albert Bodenhamer, 2012/03/19 23:16:17]

Good catch.  I've split this into 2 different tests.

+
+  // Simulate a blocked popup.
+  chrome_render_thread_->printer()->ResetPrinter();
+  chrome_render_thread_->printer()->SetScriptedPrintAllowed(false);
+  LoadHTML(kPrintWithJSHTML);
+  VerifyPagesPrinted(false);
+
+  // Unblock and verify printing works.
+  chrome_render_thread_->printer()->SetScriptedPrintAllowed(true);
   chrome_render_thread_->printer()->ResetPrinter();
   LoadHTML(kPrintWithJSHTML);
   VerifyPageCount(1);
   VerifyPagesPrinted(true);
 }
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
 // TODO(estade): I don't think this test is worth porting to Linux. We will have
 // to rip out and replace most of the IPC code if we ever plan to improve
 // printing, and the comment below by sverrir suggests that it doesn't do much
@@ skipping 606 matching lines @@
   // Fill in some dummy values.
   DictionaryValue dict;
   CreatePrintSettingsDictionary(&dict);
   OnPrintForPrintPreview(dict);
 
   VerifyPrintFailed(true);
   VerifyPagesPrinted(false);
 }
 
 #endif  // !defined(OS_CHROMEOS)