[Print Preview]: Added code to support pdf fit to page functionality.

chrome/renderer/print_web_view_helper.h

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 55a2e6be638d188e5fc8b1415c7fc76f5332be3a..deaa38ca7008532f705b33486f49103f67d223b5 100644
--- a/chrome/renderer/print_web_view_helper.h
+++ b/chrome/renderer/print_web_view_helper.h
@@ -160,6 +160,13 @@ class PrintWebViewHelper
   // Returns true if the current destination printer is PRINT_TO_PDF.
   bool IsPrintToPdfRequested(const base::DictionaryValue& settings);
 
+  // Updates |settings| fit to page information based on |job_settings| and

[Lei Zhang, 2012/04/26 22:52:23]

I think this comment can use some improvements to be less vague.

As an example, the following comment is useless. I cannot tell what this
function is suppose to do without reading the implementation. Even then, I don't
know if the implementation is doing what it is designed to do.

// Returns a value based on |a| and |b|.
int MagicalMathFunction(a, b);

Compare with the following. Here, I know exactly what the function does, and I
can even verify the implementation is correct.

// Returns the largest prime number between |a| and |b|.
int MagicalMathFunction(a, b);

[kmadhusu, 2012/04/27 01:28:58]

As we discussed, I have listed the details and changed the function signature to
return a bool. 

+  // |settings|.
+  void UpdateFitToPageInfo(WebKit::WebFrame* frame,
+                           const WebKit::WebNode& node,
+                           const base::DictionaryValue& job_settings,
+                           PrintMsg_PrintPages_Params* settings);
+
   // Initiate print preview.
   void OnInitiatePrintPreview();