Close web contents modal dialogs on content load start

chrome/browser/printing/print_view_manager.cc

Index: chrome/browser/printing/print_view_manager.cc
diff --git a/chrome/browser/printing/print_view_manager.cc b/chrome/browser/printing/print_view_manager.cc
index f5efeb714f15704be6e2b0936599260865609571..2aca362e70232b969857d8eccc5f74e18c238cf5 100644
--- a/chrome/browser/printing/print_view_manager.cc
+++ b/chrome/browser/printing/print_view_manager.cc
@@ -153,7 +153,6 @@ void PrintViewManager::PrintPreviewForWebNode() {
 
 void PrintViewManager::PrintPreviewDone() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  DCHECK_NE(NOT_PREVIEWING, print_preview_state_);

[Lei Zhang, 2013/06/27 02:28:46]

If I put this DCHECK back, I can't seem to trigger it, at least on Linux views.
That is, PrintPreviewDone() always gets called for me before RenderViewGone().
Is there a specific set of steps that reliably triggers this for you? I can try
those steps, or try on Windows.

[Mike Wittman, 2013/06/27 17:34:04]

I didn't experience this DCHECK firing while running Chrome, but it does so
reliably with the
PrintPreviewDialogControllerBrowserTest.NavigateFromInitiatorTab browser test.

[Lei Zhang, 2013/06/27 21:36:18]

No luck for me on Linux-views. I'll fire up the Windoze box and get back to
you...

 
   if (print_preview_state_ == SCRIPTED_PREVIEW) {
     ScriptedPrintPreviewClosureMap& map =