Add generic text printing

chrome/utility/printing_handler.cc

Index: chrome/utility/printing_handler.cc
diff --git a/chrome/utility/printing_handler.cc b/chrome/utility/printing_handler.cc
index 146ef7a6b1b6e467acd985f86538771dba2b1c37..9e700129a010f841f27b209ee6f9c13104bcc819 100644
--- a/chrome/utility/printing_handler.cc
+++ b/chrome/utility/printing_handler.cc
@@ -91,18 +91,21 @@ void PrintingHandler::OnRenderPDFPagesToMetafile(
   pdf_rendering_settings_ = settings;
   chrome_pdf::SetPDFUseGDIPrinting(pdf_rendering_settings_.mode ==
                                    PdfRenderSettings::Mode::GDI_TEXT);
-  int postscript_level;
+  int printing_mode;
   switch (pdf_rendering_settings_.mode) {
+    case PdfRenderSettings::Mode::TEXTONLY:
+      printing_mode = 1;

[Lei Zhang, 2017/07/12 00:21:15]

If you add another enum in pdf/pdf.h and change SetPDFUsePrintMode() to use that
enum as its parameter, then you can use those enum values here instead of these
magical values. Inside pdf/pdfium, you can static_assert() those enum values
match up with FPDF_PRINTMODE_* from PDFium.

Previously, having postscript_level = 2 made sense without needing an enum.

[rbpotter, 2017/07/13 18:46:15]

Done.

+      break;
     case PdfRenderSettings::Mode::POSTSCRIPT_LEVEL2:
-      postscript_level = 2;
+      printing_mode = 2;
       break;
     case PdfRenderSettings::Mode::POSTSCRIPT_LEVEL3:
-      postscript_level = 3;
+      printing_mode = 3;
       break;
     default:
-      postscript_level = 0;  // Not using postscript.
+      printing_mode = 0;  // Not using postscript or text only.
   }
-  chrome_pdf::SetPDFPostscriptPrintingLevel(postscript_level);
+  chrome_pdf::SetPDFUsePrintMode(printing_mode);
 
   base::File pdf_file = IPC::PlatformFileForTransitToFile(pdf_transit);
   int page_count = LoadPDF(std::move(pdf_file));