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 80021)
+++ 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"

[stuartmorgan, 2011/04/01 22:34:14]

Why do we need this?

[Lei Zhang, 2011/04/04 22:21:03]

Yes, we do for print_view_manager().

 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/browser/tab_contents/tab_contents_observer.h"
 #include "content/common/notification_registrar.h"
@@ -115,6 +116,10 @@
     return search_engine_tab_helper_.get();
   }
 
+  printing::PrintViewManager* print_view_manager() {
+    return print_view_manager_.get();
+  }
+
   // Overrides -----------------------------------------------------------------
 
   // TabContentsObserver overrides:
@@ -158,7 +163,7 @@
   bool is_starred_;
 
   // Handles print job for this contents.
-  scoped_ptr<printing::PrintViewManager> printing_;
+  scoped_ptr<printing::PrintViewManager> print_view_manager_;
 
   // Handles Print Preview messages.
   scoped_ptr<printing::PrintPreviewMessageHandler> print_preview_;