commit 78b334a824c42138f5e98060638b1643b0279602
author rbpotter <rbpotter@chromium.org> Wed Oct 25 11:58:41 2017 -0700
committer Chromium commit bot <commit-bot@chromium.org> Wed Oct 25 23:16:37 2017 +0000
tree 79fb4a460e4e0c48215b3a070ce7c9cbf313a098
parent 06673ede2dda783187fb05f9a97d0bec2f34dd32

Temporarily disable mask rendering again

Bug: chromium:777837
Change-Id: I1830dc37b9c039a95509627026461134d8f2c16d
Reviewed-on: https://pdfium-review.googlesource.com/16710
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>

fpdfsdk/fpdfview.cpp

diff --git a/fpdfsdk/fpdfview.cpp b/fpdfsdk/fpdfview.cpp
index 98a5b32..96777cc 100644
--- a/fpdfsdk/fpdfview.cpp
+++ b/fpdfsdk/fpdfview.cpp
@@ -866,10 +866,12 @@
   // of masks. Full page bitmaps result in large spool sizes, so they should
   // only be used when necessary. For large numbers of masks, rendering each
   // individually is inefficient and unlikely to significantly improve spool
-  // size.
-  const bool bNewBitmap =
-      pPage->BackgroundAlphaNeeded() ||
-      (pPage->HasImageMask() && pPage->GetMaskBoundingBoxes().size() > 100);
+  // size. TODO (rbpotter): Find out why this still breaks printing for some
+  // PDFs (see crbug.com/777837).
+  const bool bEnableImageMasks = false;
+  const bool bNewBitmap = pPage->BackgroundAlphaNeeded() ||
+                          (pPage->HasImageMask() && !bEnableImageMasks) ||
+                          pPage->GetMaskBoundingBoxes().size() > 100;
   const bool bHasMask = pPage->HasImageMask() && !bNewBitmap;
   if (bNewBitmap || bHasMask) {
     pBitmap = pdfium::MakeRetain<CFX_DIBitmap>();