Use DPI from Print Preview on Windows, handle non square

components/printing/browser/print_manager_utils.cc

Index: components/printing/browser/print_manager_utils.cc
diff --git a/components/printing/browser/print_manager_utils.cc b/components/printing/browser/print_manager_utils.cc
index ec383a232d2d464764ce37b364e9e17c13264f4e..e948b8bf786d8458f8563d508b176f6c4ab17606 100644
--- a/components/printing/browser/print_manager_utils.cc
+++ b/components/printing/browser/print_manager_utils.cc
@@ -23,8 +23,9 @@ void RenderParamsFromPrintSettings(const PrintSettings& settings,
   params->margin_left = settings.page_setup_device_units().content_area().x();
   params->dpi = settings.dpi();
   params->scale_factor = settings.scale_factor();
-  // Currently hardcoded at 72dpi. See PrintSettings' constructor.
-  params->desired_dpi = settings.desired_dpi();
+  params->desired_dpi = settings.desired_dpi_horizontal() > 0
+                            ? settings.desired_dpi_horizontal()
+                            : 72;

[Lei Zhang, 2017/03/31 22:21:13]

Can we leave this file with only getter calls, and move the 72 DPI value into
printing/ ?

[rbpotter, 2017/04/01 00:08:01]

Removed this parameter entirely and used kPointsPerInch instead,
which is what this was actually used for.

   params->rasterize_pdf = settings.rasterize_pdf();
   // Always use an invalid cookie.
   params->document_cookie = 0;