Bug 5540

Summary: Add CairoOutputDev::drawSoftMaskedImage
Product: poppler Reporter: Pablo Rodríguez <freedesktop>
Component: generalAssignee: Jeff Muizelaar <jmuizelaar>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high CC: amayer, ed, jmuizelaar, josuealcalde, wrobell
Version: unspecified   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments: document with wrong displayed image on first page
an ugly patch to add smask support to the cairo backend
smask support for the cairo backend

Description Pablo Rodríguez 2006-01-07 23:20:27 UTC
poppler 0.4.3 is not able to display image on first page of the attached document.

xpdf displays it fine and gpdf doesn't.
Comment 1 Pablo Rodríguez 2006-01-07 23:21:43 UTC
Created attachment 4279 [details]
document with wrong displayed image on first page
Comment 2 Jeff Muizelaar 2006-01-09 12:24:22 UTC
looks like this is caused by missing smask support
Comment 3 Albert Astals Cid 2006-01-09 23:22:21 UTC
Works in development branch 
Comment 4 Jeff Muizelaar 2006-01-11 10:17:19 UTC
Doesn't work with cairo backend yet though.
Comment 5 Jeff Muizelaar 2006-01-11 16:01:54 UTC
Created attachment 4326 [details] [review]
an ugly patch to add smask support to the cairo backend
Comment 6 Jeff Muizelaar 2006-01-13 13:44:29 UTC
*** Bug 5304 has been marked as a duplicate of this bug. ***
Comment 7 August Mayer 2006-01-13 21:00:41 UTC
As my bug was duped, I humbly add the URL with PDFs that I was complaining about:

http://www.escapistmagazine.com/

All (?) of the Escapist PDFs don't render correctly with poppler, so any PDF
from there should be a test-case.
Comment 8 Jeff Muizelaar 2006-02-10 09:51:07 UTC
*** Bug 3098 has been marked as a duplicate of this bug. ***
Comment 9 Jeff Muizelaar 2006-02-10 09:53:34 UTC
change title
Comment 10 Jeff Muizelaar 2006-02-10 09:54:21 UTC
The patch I posted here is broken for some pdfs. I haven't had time to track
down the problem yet.
Comment 11 Ed Catmur 2006-02-10 10:19:38 UTC
Wouldn't we also need CairoOutputDev::drawMaskedImage to attain parity with
splash backend; or is that another bug?
Comment 12 Jeff Muizelaar 2006-02-10 10:43:01 UTC
Yes drawMaskedImage should be added as well, however, it is used less often so
not as high priority. (I don't recall seeing many documents that used that feature.)
Comment 13 Jeff Muizelaar 2006-02-11 05:30:04 UTC
*** Bug 3020 has been marked as a duplicate of this bug. ***
Comment 14 Jeff Muizelaar 2006-02-19 06:05:46 UTC
Created attachment 4667 [details] [review]
smask support for the cairo backend

still ugly but should work this time.
Comment 15 Jeff Muizelaar 2006-02-19 07:22:17 UTC
should be fixed in cvs

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.