Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7257)

Unified Diff: chrome/browser/ui/webui/print_preview/print_preview_ui.cc

Issue 9699040: PrintPreview: Hide/Show the header footer option based on print frame margins. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..bbdd183a3cc2058b498ed4f767c09c189c7737ba 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
+++ b/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
@@ -206,7 +206,8 @@ void PrintPreviewUI::OnDidGetPreviewPageCount(
}
void PrintPreviewUI::OnDidGetDefaultPageLayout(
- const PageSizeMargins& page_layout, bool has_custom_page_size_style) {
+ const PageSizeMargins& page_layout, bool has_custom_page_size_style,
+ bool header_footer_applies) {
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) {
@@ -223,8 +224,10 @@ void PrintPreviewUI::OnDidGetDefaultPageLayout(
layout.SetDouble(printing::kSettingContentHeight, page_layout.content_height);
base::FundamentalValue has_page_size_style(has_custom_page_size_style);
+ base::FundamentalValue has_header_footer_option(header_footer_applies);
web_ui()->CallJavascriptFunction("onDidGetDefaultPageLayout", layout,
- has_page_size_style);
+ has_page_size_style,
+ has_header_footer_option);
}
void PrintPreviewUI::OnDidPreviewPage(int page_number,

Powered by Google App Engine
This is Rietveld 408576698