Print Preview: Set a print job's title and url correctly.

chrome/browser/printing/print_preview_message_handler.cc

Index: chrome/browser/printing/print_preview_message_handler.cc
===================================================================
--- chrome/browser/printing/print_preview_message_handler.cc	(revision 80021)
+++ chrome/browser/printing/print_preview_message_handler.cc	(working copy)
@@ -7,7 +7,9 @@
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/printing/print_job_manager.h"
 #include "chrome/browser/printing/print_preview_tab_controller.h"
+#include "chrome/browser/printing/print_view_manager.h"
 #include "chrome/browser/printing/printer_query.h"
+#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/browser/ui/webui/print_preview_handler.h"
 #include "chrome/browser/ui/webui/print_preview_ui.h"
 #include "chrome/browser/ui/webui/print_preview_ui_html_source.h"
@@ -69,6 +71,10 @@
     return;
   }
 
+  TabContentsWrapper* wrapper =
+      TabContentsWrapper::GetCurrentWrapperForContents(print_preview_tab);
+  wrapper->print_view_manager()->OverrideTitleAndUrl(tab_contents());

[kmadhusu, 2011/03/31 22:37:48]

if (wrapper)
  wrapper->print_view_manager()->OverrideTitleAndUrl(tab_contents());

[Lei Zhang, 2011/03/31 22:42:40]

avi: Would we ever run into a case where |wrapper| is NULL?

[Avi (use Gerrit), 2011/04/06 11:21:32]

No. Only in oddball testing situations.

+
   PrintPreviewUI* print_preview_ui =
       static_cast<PrintPreviewUI*>(print_preview_tab->web_ui());
   PrintPreviewUIHTMLSource* html_source = print_preview_ui->html_source();