Bug 14619 - Poppler-cairo not displaying pdf correctly
Summary: Poppler-cairo not displaying pdf correctly
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: cairo backend (show other bugs)
Version: unspecified
Hardware: x86 (IA32) All
: medium normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 27456
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-22 03:49 UTC by Pedro Villavicencio
Modified: 2012-01-20 06:37 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Fix-test-for-rotation.patch (1.55 KB, patch)
2012-01-19 06:32 UTC, Patrick
Details | Splinter Review

Description Pedro Villavicencio 2008-02-22 03:49:44 UTC
This report has been filed here:

https://bugs.launchpad.net/ubuntu/+source/poppler/+bug/193927

"Evince does not display a pdf correctly. It seems like a layer or something is missing.
I have no idea of how the pdf was made but I have attached the example.pdf to this bug report. I will also attach screenshots from evince and also xpdf (which does display it correctly)."

http://launchpadlibrarian.net/12128520/example.pdf
http://launchpadlibrarian.net/12128526/evince.png
http://launchpadlibrarian.net/12128527/xpdf.png

Thanks,
Comment 1 Albert Astals Cid 2008-02-22 14:31:10 UTC
Can reproduce it on cairo backend, not with splash
Comment 2 Carlos Garcia Campos 2009-11-18 09:06:52 UTC
This is another bug in drawImageMaskPrescaled :-(
Comment 3 Patrick 2012-01-19 06:32:54 UTC
Created attachment 55785 [details] [review]
Fix-test-for-rotation.patch

This patch fixes the bug, which stems from the wrong assumption
that rotation can be detected by testing (matrix.xy == 0 && matrix.yx == 0) only.

Defined LOG_CAIRO in CairoOutputDev.cc and checked the output to
make sure the test works, with the old attached pdf: Edited it in vim and changed /Rotate 180 to 0, 90 and 270 ("attempting to reconstruct xref table" succeeded...). Prints "drawImageMaskPrescaled" if the mask is not rotated, and "drawImageMask" if it is, as intended.
Comment 4 Carlos Garcia Campos 2012-01-20 06:37:19 UTC
Comment on attachment 55785 [details] [review]
Fix-test-for-rotation.patch

Review of attachment 55785 [details] [review]:
-----------------------------------------------------------------

Pushed to git master, thanks!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.