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

chrome/browser/ui/tab_contents/tab_contents_wrapper.cc

Index: chrome/browser/ui/tab_contents/tab_contents_wrapper.cc
===================================================================
--- chrome/browser/ui/tab_contents/tab_contents_wrapper.cc	(revision 80342)
+++ chrome/browser/ui/tab_contents/tab_contents_wrapper.cc	(working copy)
@@ -19,7 +19,6 @@
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prerender/prerender_observer.h"
 #include "chrome/browser/printing/print_preview_message_handler.h"
-#include "chrome/browser/printing/print_view_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/tab_contents/simple_alert_infobar_delegate.h"
 #include "chrome/browser/ui/find_bar/find_tab_helper.h"
@@ -71,7 +70,7 @@
   dev_tools_observer_.reset(new DevToolsObserver(contents));
   file_select_observer_.reset(new FileSelectObserver(contents));
   prerender_observer_.reset(new prerender::PrerenderObserver(contents));
-  printing_.reset(new printing::PrintViewManager(contents));
+  print_view_manager_.reset(new printing::PrintViewManager(contents));

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

Is this a per-tab observer or a Tab Helper? If it provides public functions,
it's a Tab Helper; move it up 15 lines to be with its friends.

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

It's a tab helper then.

   print_preview_.reset(new printing::PrintPreviewMessageHandler(contents));
 }