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

Unified Diff: chrome/renderer/print_web_view_helper.h

Issue 10083059: [Print Preview] Modified PP_PrintSettings_Dev interface to support auto fit to page functionality. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: '' Created 8 years, 8 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.h
diff --git a/chrome/renderer/print_web_view_helper.h b/chrome/renderer/print_web_view_helper.h
index 99eb0a99b5349c1e9875494d507e61a291d1a48a..0994d1fab3d4552e83d18bc6ae47de90062c3ed1 100644
--- a/chrome/renderer/print_web_view_helper.h
+++ b/chrome/renderer/print_web_view_helper.h
@@ -42,7 +42,8 @@ class PrepareFrameAndViewForPrint {
// printed.
PrepareFrameAndViewForPrint(const PrintMsg_Print_Params& print_params,
WebKit::WebFrame* frame,
- const WebKit::WebNode& node);
+ const WebKit::WebNode& node,
+ bool print_for_preview);
dmichael (off chromium) 2012/04/19 20:11:46 optional suggestion: I like using enums instead of
kmadhusu 2012/04/20 22:29:57 Done. Added an enum param to specify the print sca
~PrepareFrameAndViewForPrint();
void UpdatePrintParams(const PrintMsg_Print_Params& print_params);
@@ -62,17 +63,24 @@ class PrepareFrameAndViewForPrint {
void FinishPrinting();
private:
- void StartPrinting(const gfx::Size& print_params);
+ void StartPrinting(const gfx::Size& canvas_size,
+ const gfx::Rect& printable_area,
+ const gfx::Size& paper_size,
+ bool fit_pdf_page_to_paper_size);
dmichael (off chromium) 2012/04/19 20:11:46 ditto
kmadhusu 2012/04/20 22:29:57 This bool is no longer required. So I removed from
WebKit::WebFrame* frame_;
WebKit::WebNode node_to_print_;
WebKit::WebView* web_view_;
gfx::Size print_canvas_size_;
+ gfx::Rect printable_area_;
+ gfx::Size paper_size_;
gfx::Size prev_view_size_;
gfx::Size prev_scroll_offset_;
int dpi_;
int expected_pages_count_;
bool use_browser_overlays_;
+ bool fit_pdf_page_to_paper_size_;
+ bool print_for_preview_;
bool finished_;
DISALLOW_COPY_AND_ASSIGN(PrepareFrameAndViewForPrint);

Powered by Google App Engine
This is Rietveld 408576698