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

Unified Diff: chrome/renderer/print_web_view_helper_win.cc

Issue 7348010: Added Header and Footer support using Skia (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Separate CL for struct change. Changes as per review comments. Created 9 years, 5 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/renderer/print_web_view_helper_win.cc
diff --git a/chrome/renderer/print_web_view_helper_win.cc b/chrome/renderer/print_web_view_helper_win.cc
index cf0df2fbd8ebf8f31b9bc5719bd286a536bed94d..b5cc022c5ba441db2b61857fe98aac3851f07f92 100644
--- a/chrome/renderer/print_web_view_helper_win.cc
+++ b/chrome/renderer/print_web_view_helper_win.cc
@@ -15,6 +15,7 @@
#include "printing/units.h"
#include "skia/ext/vector_canvas.h"
#include "skia/ext/vector_platform_device_emf_win.h"
+#include "skia/ext/vector_platform_device_skia.h"
#include "third_party/skia/include/core/SkRefCnt.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
#include "ui/gfx/gdi_util.h"
@@ -185,6 +186,16 @@ void PrintWebViewHelper::RenderPage(
}
float webkit_scale_factor = frame->printPage(page_number, canvas.get());
+
+ if (is_preview && params.display_header_footer) {
+ // The page_number count starts from 0, so we add 1.
dpapad 2011/07/25 15:40:01 Nit: |page_number| is 0-based, so 1 is added.
Aayush Kumar 2011/07/25 18:37:23 Done.
+ PrintHeaderAndFooter(static_cast<skia::VectorPlatformDeviceSkia*>(device),
+ canvas, page_number + 1,
+ print_preview_context_.total_page_count(),
+ webkit_scale_factor, page_layout_in_points,
+ header_footer_info_);
+ }
+
if (*scale_factor <= 0 || webkit_scale_factor <= 0) {
NOTREACHED() << "Printing page " << page_number << " failed.";
} else {

Powered by Google App Engine
This is Rietveld 408576698