Print Preview: Display a throbber when the user requests the system print

chrome/browser/printing/print_preview_tab_controller.cc

Index: chrome/browser/printing/print_preview_tab_controller.cc
diff --git a/chrome/browser/printing/print_preview_tab_controller.cc b/chrome/browser/printing/print_preview_tab_controller.cc
index 0d94be172f689e541f0bf6fe1d5f13e7c51cd1d1..5a65d8ead92dfc1fb30c6929c72d4d6d0c3944a1 100644
--- a/chrome/browser/printing/print_preview_tab_controller.cc
+++ b/chrome/browser/printing/print_preview_tab_controller.cc
@@ -138,6 +138,11 @@ void PrintPreviewTabController::Observe(NotificationType type,
     if (initiator_tab)
       RemoveObservers(initiator_tab);
 
+    PrintPreviewUI* print_preview_ui =
+        static_cast<PrintPreviewUI*>(source_tab->web_ui());

[Lei Zhang, 2011/06/28 20:55:35]

For the case of: open print preview; close initiator tab; click "reopen the
page", normally you would get NULL here and skip the OnNavigation() call.
However, if the initiator tab is a WebUI page, e.g. the NTP, then you would get
a WebUI back and incorrectly cast it to a PrintPreviewUI.

[Lei Zhang, 2011/06/28 20:58:04]

Probably just want to check with IsPrintPreviewTab(source_tab) here.

+    if (print_preview_ui)
+      print_preview_ui->OnNavigation();
+
     // Erase the map entry.
     preview_tab_map_.erase(source_tab);
   } else {