Windowed plugins like Flash would cause the hung plugin dialog to show up whe...

chrome/renderer/print_web_view_helper.cc

 // Copyright (c) 2009 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/renderer/print_web_view_helper.h"
 
 #include "app/l10n_util.h"
 #include "base/logging.h"
 #include "base/gfx/size.h"
 #include "chrome/common/render_messages.h"
@@ skipping 139 matching lines @@
       {
         PrepareFrameAndViewForPrint prep_frame_view(default_settings,
                                                     frame,
                                                     frame->GetView());
         expected_pages_count = prep_frame_view.GetExpectedPageCount();
         DCHECK(expected_pages_count);
       }
 
       // Ask the browser to show UI to retrieve the final print settings.
       ViewMsg_PrintPages_Params print_settings;
+
+      ViewHostMsg_ScriptedPrint_Params params;
+
+      // The routing id is sent across as it is needed to look up the
+      // corresponding RenderViewHost instance to signal and reset the
+      // pump messages event.
+      params.routing_id = routing_id();
       // host_window_ may be NULL at this point if the current window is a popup
       // and the print() command has been issued from the parent. The receiver
       // of this message has to deal with this.
-      msg = new ViewHostMsg_ScriptedPrint(routing_id(),
-                                          render_view_->host_window(),
-                                          default_settings.document_cookie,
-                                          expected_pages_count,
-                                          frame->HasSelection(),
+      params.host_window_id = render_view_->host_window();
+      params.cookie = default_settings.document_cookie;
+      params.has_selection = frame->HasSelection();
+      params.expected_pages_count = expected_pages_count;
+
+      msg = new ViewHostMsg_ScriptedPrint(params,
                                           &print_settings);
+      msg->set_pump_messages_event(render_view_->modal_dialog_event());
+
       if (Send(msg)) {
         msg = NULL;
 
         // If the settings are invalid, early quit.
         if (print_settings.params.dpi &&
             print_settings.params.document_cookie) {
           if (print_settings.params.selection_only) {
             CopyAndPrint(print_settings, frame);
           } else {
             // TODO: Always copy before printing.
@@ skipping 236 matching lines @@
 WebKit::WebScreenInfo PrintWebViewHelper::GetScreenInfo(WebWidget* webwidget) {
   WebKit::WebScreenInfo info;
   NOTREACHED();
   return info;
 }
 
 bool PrintWebViewHelper::IsHidden(WebWidget* webwidget) {
   NOTREACHED();
   return true;
 }