Added Header and Footer support using Skia

chrome/browser/ui/webui/print_preview_handler.cc

Index: chrome/browser/ui/webui/print_preview_handler.cc
diff --git a/chrome/browser/ui/webui/print_preview_handler.cc b/chrome/browser/ui/webui/print_preview_handler.cc
index e1455bd020df077375eba0be09c5bcb1f20c1a89..220126a28812510eaa087b9d6f0e3d6877b88107 100644
--- a/chrome/browser/ui/webui/print_preview_handler.cc
+++ b/chrome/browser/ui/webui/print_preview_handler.cc
@@ -483,9 +483,33 @@ void PrintPreviewHandler::HandleGetPreview(const ListValue* args) {
     print_preview_ui->OnPrintPreviewFailed();
     return;
   }
+
+  bool displayHeaderFooter;
+  if (!settings->GetBoolean(printing::kSettingHeaderFooter,
+                            &displayHeaderFooter)) {
+    NOTREACHED();
+  }
+
+  // If headers and footers option is checked then we retrieve the page title
+  // and url and send it to the renderer process.
+  DictionaryValue header_footer_info;
+  if (displayHeaderFooter) {
+    header_footer_info.SetString(printing::kSettingHeaderFooterTitle,

[vandebo (ex-Chrome), 2011/07/19 21:31:20]

Why not just put this in settings?

[vandebo (ex-Chrome), 2011/07/22 22:58:33]

You didn't address this comment.

[Aayush Kumar, 2011/07/24 02:09:02]

Sorry about that - I somehow seemed to have missed this file.

The reason I didn't put this in settings is because the settings param contains
a lot of printer related settings that is then sent all over, and used to get
the appropriate printer settings.  The header footer strings are not printer
related and will not be used by it - thus - I created a separate parameter.
On 2011/07/22 22:58:33, vandebo wrote:

+                                 initiator_tab->GetTitle());
+
+    std::string footer_url;

[vandebo (ex-Chrome), 2011/07/19 21:31:20]

nit: url

[Aayush Kumar, 2011/07/24 02:09:02]

Done.

+    NavigationEntry* entry = initiator_tab->controller().GetActiveEntry();
+    if (entry)
+      footer_url = entry->virtual_url().spec();
+
+    header_footer_info.SetString(printing::kSettingHeaderFooterURL,
+                                 footer_url);
+  }
+
   VLOG(1) << "Print preview request start";
   RenderViewHost* rvh = initiator_tab->render_view_host();
-  rvh->Send(new PrintMsg_PrintPreview(rvh->routing_id(), *settings));
+  rvh->Send(new PrintMsg_PrintPreview(rvh->routing_id(), *settings,
+            header_footer_info));
 }
 
 void PrintPreviewHandler::HandlePrint(const ListValue* args) {