diff options
| author | rbpotter <rbpotter@chromium.org> | 2017-10-25 11:58:41 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-10-25 23:16:37 +0000 |
| commit | 78b334a824c42138f5e98060638b1643b0279602 (patch) | |
| tree | 79fb4a460e4e0c48215b3a070ce7c9cbf313a098 | |
| parent | 06673ede2dda783187fb05f9a97d0bec2f34dd32 (diff) | |
| download | pdfium-chromium/3251.tar.xz | |
Temporarily disable mask rendering againchromium/3251
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>
| -rw-r--r-- | fpdfsdk/fpdfview.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/fpdfsdk/fpdfview.cpp b/fpdfsdk/fpdfview.cpp index 98a5b32d96..96777cca63 100644 --- a/fpdfsdk/fpdfview.cpp +++ b/fpdfsdk/fpdfview.cpp @@ -866,10 +866,12 @@ FPDF_EXPORT void FPDF_CALLCONV FPDF_RenderPage(HDC dc, // 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>(); |