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

chrome/browser/ui/tab_contents/tab_contents_wrapper.h

Index: chrome/browser/ui/tab_contents/tab_contents_wrapper.h
===================================================================
--- chrome/browser/ui/tab_contents/tab_contents_wrapper.h	(revision 80342)
+++ chrome/browser/ui/tab_contents/tab_contents_wrapper.h	(working copy)
@@ -11,6 +11,7 @@
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
+#include "chrome/browser/printing/print_view_manager.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/tab_contents/tab_contents_observer.h"
 #include "content/common/notification_registrar.h"
@@ -21,7 +22,6 @@
 
 namespace printing {
 class PrintPreviewMessageHandler;
-class PrintViewManager;
 }
 
 class AutocompleteHistoryManager;
@@ -110,6 +110,10 @@
     return search_engine_tab_helper_.get();
   }
 
+  printing::PrintViewManager* print_view_manager() {
+    return print_view_manager_.get();
+  }
+
   // Overrides -----------------------------------------------------------------
 
   // TabContentsObserver overrides:
@@ -153,7 +157,7 @@
   bool is_starred_;
 
   // Handles print job for this contents.
-  scoped_ptr<printing::PrintViewManager> printing_;
+  scoped_ptr<printing::PrintViewManager> print_view_manager_;

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

If this is a Tab Helper, put it in the Tab Helper section below.

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

Done.

 
   // Handles Print Preview messages.
   scoped_ptr<printing::PrintPreviewMessageHandler> print_preview_;

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

No. This is a class that just sits there per-tab and watches. This goes in the
"per-tab observers" section below. I missed when you put it here. Please move
it.

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

Done.