PrintPreview: While printing the preview data, set the initiator tab title as print job name.

chrome/browser/printing/print_view_manager.h

Index: chrome/browser/printing/print_view_manager.h
diff --git a/chrome/browser/printing/print_view_manager.h b/chrome/browser/printing/print_view_manager.h
index afd551985794ecb8bf2efd9dadaa3ecaa77ed21e..df428fd9635ba7c6da693a0b40cdc35c4b080a4d 100644
--- a/chrome/browser/printing/print_view_manager.h
+++ b/chrome/browser/printing/print_view_manager.h
@@ -51,7 +51,9 @@ class PrintViewManager : public NotificationObserver,
   virtual void StopNavigation();
 
  private:
-  void OnDidGetPrintedPagesCount(int cookie, int number_pages);
+  void OnDidGetBasicPrintJobInfo(int cookie,
+                                 int number_pages,
+                                 const string16& job_title);
   void OnDidPrintPage(const PrintHostMsg_DidPrintPage_Params& params);
 
   // Processes a NOTIFY_PRINT_JOB_EVENT notification.
@@ -128,6 +130,14 @@ class PrintViewManager : public NotificationObserver,
   bool expecting_first_page_;
 #endif
 
+  // Current print job title.
+  // For preview printing, |job_title_| is set to initiator tab title in
+  // PrintHostMsg_DidGetBasicPrintJobInfo message handler.

[stuartmorgan, 2011/03/30 22:59:17]

Having this much detail about the implementation in the header is almost always
a bad idea; the chances of it getting out of sync over time are extremely high.

+  // For normal printing, |job_title_| is empty. Print view manager will use
+  // the current tab title as print job title. If the current tab title is
+  // empty, use the default job title.

[stuartmorgan, 2011/03/30 22:59:17]

I wouldn't even mention print preview vs "normal" (keep in mind that at some
point print preview will be the normal case, so that will become confusing).
Really, discussion of what happens when this is empty belongs in the code
reading it, since this is not public.

+  string16 job_title_;
+
   DISALLOW_COPY_AND_ASSIGN(PrintViewManager);
 };