Windowed plugins like Flash would cause the hung plugin dialog to show up when the print dialog was displayed.

The proposed fix is to signal the modal dialog event which would ensure that the renderer and plugins both pump
messages when the print dialog is displayed.

This fixes http://code.google.com/p/chromium/issues/detail?id=13804

Bug=13804

Review URL: http://codereview.chromium.org/155133

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20243 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 8 deletions

diff --git a/chrome/renderer/mock_render_thread.cc b/chrome/renderer/mock_render_thread.cc
index a9df6fd..3494b12 100644
--- a/chrome/renderer/mock_render_thread.cc
+++ b/chrome/renderer/mock_render_thread.cc
@@ -114,15 +114,13 @@ void MockRenderThread::OnGetDefaultPrintSettings(ViewMsg_Print_Params* params) {
     printer_->GetDefaultPrintSettings(params);
 }
 
-void MockRenderThread::OnScriptedPrint(gfx::NativeViewId host_window,
-                                       int cookie,
-                                       int expected_pages_count,
-                                       bool has_selection,
-                                       ViewMsg_PrintPages_Params* settings) {
+void MockRenderThread::OnScriptedPrint(
+    const ViewHostMsg_ScriptedPrint_Params& params,
+    ViewMsg_PrintPages_Params* settings) {
   if (printer_.get()) {
-    printer_->ScriptedPrint(cookie,
-                            expected_pages_count,
-                            has_selection,
+    printer_->ScriptedPrint(params.cookie,
+                            params.expected_pages_count,
+                            params.has_selection,
                             settings);
   }
 }