Close web contents modal dialogs on content load start

chrome/browser/printing/print_preview_dialog_controller.cc

Index: chrome/browser/printing/print_preview_dialog_controller.cc
diff --git a/chrome/browser/printing/print_preview_dialog_controller.cc b/chrome/browser/printing/print_preview_dialog_controller.cc
index b9413a0f7cee7f60dc6784884c23ffe8f572b8e8..8f509b85c6d3eb84d67b461245da84ada2478ba0 100644
--- a/chrome/browser/printing/print_preview_dialog_controller.cc
+++ b/chrome/browser/printing/print_preview_dialog_controller.cc
@@ -388,8 +388,6 @@ void PrintPreviewDialogController::OnNavEntryCommitted(
     NOTREACHED();
     return;
   }
-
-  RemoveInitiatorTab(contents);

[Lei Zhang, 2013/06/27 00:31:18]

Once we do this, then maybe we don't even need watch for
NOTIFICATION_NAV_ENTRY_COMMITTED for initiator tabs.

[Mike Wittman, 2013/06/27 00:58:11]

Yes, that's true. It would require a bit more complexity around
AddObservers/RemoveObservers which currently do not distinguish between tab
types for WebContents.

[Lei Zhang, 2013/06/27 02:29:37]

Yes it would. If you don't want to, then just put a TODO(thestig) and I can take
care of it in the future.

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

Not a problem, added the additional logic to not observe
NOTIFICATION_NAV_ENTRY_COMMITTED for the initiator tab.

 }
 
 WebContents* PrintPreviewDialogController::CreatePrintPreviewDialog(