PrintPreview: Hide/Show the header footer option based on print frame margins.

chrome/browser/ui/webui/print_preview/print_preview_ui.cc

Index: chrome/browser/ui/webui/print_preview/print_preview_ui.cc
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_ui.cc b/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
index c1c204350329e4baca4583b181b4bf5f452eac9c..1c9a710b0e93c04222c1978a0544153ebcda7a88 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
+++ b/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
@@ -24,6 +24,7 @@
 #include "content/public/browser/web_contents.h"
 #include "printing/page_size_margins.h"
 #include "printing/print_job_constants.h"
+#include "ui/gfx/rect.h"
 
 using content::WebContents;
 using printing::PageSizeMargins;
@@ -206,10 +207,12 @@ void PrintPreviewUI::OnDidGetPreviewPageCount(
 }
 
 void PrintPreviewUI::OnDidGetDefaultPageLayout(
-    const PageSizeMargins& page_layout, bool has_custom_page_size_style) {
+    const PageSizeMargins& page_layout, const gfx::Rect& printable_area,
+    bool has_custom_page_size_style) {
   if (page_layout.margin_top < 0 || page_layout.margin_left < 0 ||
       page_layout.margin_bottom < 0 || page_layout.margin_right < 0 ||
-      page_layout.content_width < 0 || page_layout.content_height < 0) {
+      page_layout.content_width < 0 || page_layout.content_height < 0 ||
+      printable_area.width() <= 0 || printable_area.height() <= 0) {
     NOTREACHED();
     return;
   }
@@ -222,8 +225,19 @@ void PrintPreviewUI::OnDidGetDefaultPageLayout(
   layout.SetDouble(printing::kSettingContentWidth, page_layout.content_width);
   layout.SetDouble(printing::kSettingContentHeight, page_layout.content_height);
 
+  base::DictionaryValue printable_area_info;

[vandebo (ex-Chrome), 2012/03/19 17:11:30]

Seems like you could put this in the layout dictionary.  Is there any advantage
to putting it in a separate dictionary?

[kmadhusu, 2012/03/19 18:45:31]

Added printable area details in page layout dictionary.

+  printable_area_info.SetInteger(printing::kSettingPrintableAreaX,
+                                 printable_area.x());
+  printable_area_info.SetInteger(printing::kSettingPrintableAreaY,
+                                 printable_area.y());
+  printable_area_info.SetInteger(printing::kSettingPrintableAreaWidth,
+                                 printable_area.width());
+  printable_area_info.SetInteger(printing::kSettingPrintableAreaHeight,
+                                 printable_area.height());
+
   base::FundamentalValue has_page_size_style(has_custom_page_size_style);
   web_ui()->CallJavascriptFunction("onDidGetDefaultPageLayout", layout,
+                                   printable_area_info,
                                    has_page_size_style);
 }