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 80342)
+++ 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);

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

Are we in a callback from content/? If not, please followup with a CL to plumb
TCW though the printing system rather than TC, and pull this line.

[Lei Zhang, 2011/04/07 00:53:20]

For PrintPreview, we have a lot of classes in chrome/ that inherit from classes
in content/. Thus they have to deal with TCs and not TCWs. In this particular
case, PrintPreviewMessageHandler is a subclass of TabContentObserver, which
knows nothing about TCWs.

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