Block printing from blocked popup.

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 01c6f226faedc4f60690fbfbebaebfe8fcc4db3e..5d4b1a6ac871a2c4c45514e6c5f0a57de5642bc9 100644
--- a/chrome/browser/printing/print_preview_tab_controller.cc
+++ b/chrome/browser/printing/print_preview_tab_controller.cc
@@ -215,7 +215,8 @@ void PrintPreviewTabController::PrintPreview(TabContentsWrapper* tab) {
   PrintPreviewTabController* tab_controller = GetInstance();
   if (!tab_controller)
     return;
-  tab_controller->GetOrCreatePreviewTab(tab);
+  if (tab_controller->GetOrCreatePreviewTab(tab) == NULL)

[Lei Zhang, 2012/03/19 20:19:32]

nit: It's easier to just write !foo.

[Albert Bodenhamer, 2012/03/19 23:16:17]

I have a strong preference for "== NULL" as it avoids a sloppy implicit
typecast, BUT the rest of this file is using "!foo", so I'll go with that for
consistency.
Done.

Done.

+    tab->print_view_manager()->PrintPreviewDone();
 }
 
 TabContentsWrapper* PrintPreviewTabController::GetOrCreatePreviewTab(