Remove a "temporary" hack from when ChromeOS generated PDFs with Cairo.

pdf/pdfium/pdfium_engine.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "pdf/pdfium/pdfium_engine.h"
 
 #include <math.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 3867 matching lines @@
   int rotate = CalculatePosition(page, new_settings, &dest);
 
   int save_state = SaveDC(dc);
   // The caller wanted all drawing to happen within the bounds specified.
   // Based on scale calculations, our destination rect might be larger
   // than the bounds. Set the clip rect to the bounds.
   IntersectClipRect(dc, settings.bounds.x(), settings.bounds.y(),
                     settings.bounds.x() + settings.bounds.width(),
                     settings.bounds.y() + settings.bounds.height());
 
-  // A temporary hack. PDFs generated by Cairo (used by Chrome OS to generate
-  // a PDF output from a webpage) result in very large metafiles and the
-  // rendering using FPDF_RenderPage is incorrect. In this case, render as a
-  // bitmap. Note that this code does not kick in for PDFs printed from Chrome
-  // because in that case we create a temp PDF first before printing and this
-  // temp PDF does not have a creator string that starts with "cairo".
-  bool use_bitmap = false;
-  if (base::StartsWith(GetDocumentMetadata(doc, "Creator"), "cairo",
-                       base::CompareCase::INSENSITIVE_ASCII)) {
-    use_bitmap = true;
-  }
-
-  // Another temporary hack. Some PDFs seems to render very slowly if
-  // FPDF_RenderPage is directly used on a printer DC. I suspect it is
+  // A "temporary" hack. Some PDFs seems to render very slowly if
+  // FPDF_RenderPage() is directly used on a printer DC. I suspect it is
   // because of the code to talk Postscript directly to the printer if
   // the printer supports this. Need to discuss this with PDFium. For now,
   // render to a bitmap and then blit the bitmap to the DC if we have been
   // supplied a printer DC.
   int device_type = GetDeviceCaps(dc, TECHNOLOGY);
-  if (use_bitmap ||
-      (device_type == DT_RASPRINTER) || (device_type == DT_PLOTTER)) {
+  if (device_type == DT_RASPRINTER || device_type == DT_PLOTTER) {
     FPDF_BITMAP bitmap = FPDFBitmap_Create(dest.width(), dest.height(),
                                            FPDFBitmap_BGRx);
       // Clear the bitmap
     FPDFBitmap_FillRect(bitmap, 0, 0, dest.width(), dest.height(), 0xFFFFFFFF);
     FPDF_RenderPageBitmap(
         bitmap, page, 0, 0, dest.width(), dest.height(), rotate,
         FPDF_ANNOT | FPDF_PRINTING | FPDF_NO_CATCH);
     int stride = FPDFBitmap_GetStride(bitmap);
     BITMAPINFO bmi;
     memset(&bmi, 0, sizeof(bmi));
@@ skipping 89 matching lines @@
     double* height) {
   FPDF_DOCUMENT doc = FPDF_LoadMemDocument(pdf_buffer, pdf_buffer_size, NULL);
   if (!doc)
     return false;
   bool success = FPDF_GetPageSizeByIndex(doc, page_number, width, height) != 0;
   FPDF_CloseDocument(doc);
   return success;
 }
 
 }  // namespace chrome_pdf