Chromium Code Reviews| 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); |