Created attachment 72749 [details] PDF test for color mismatch As it was reported in GNOME Bugzilla: -------------------------------------------------------------- On one specific document that I have, some colors are not displayed correcly. With Adode Reader, I have different colors. With Evince, everything is gray. Here is the link to the document: http://www.info.ucl.ac.be/~pecheur/talks/VVof2GRLV-FAABS02.pdf Slide 13 has this issue: yellow, red and green are all gray. I've attached a a picture showing how this slide is displayed by both pdf viewers. -------------------------------------------------------------- I took out only the page 13 that shows the issue. You will find it attached. I can reproduce the problem with pdftocairo -png.
Created attachment 72750 [details] Screenshot of Acroread and Evince output
I forgot to add: I tested it against cairo 1.12 and poppler master (bef2c42f381c74fd, the commit after 0.22.0).
FWIW it works fine in splash
Created attachment 73562 [details] [review] support uncolored tiling patterns The cairo backend does not support uncolored tiling patterns. This caused uncolored patterns to be drawn in gray. The attached patch fixes this.
(In reply to comment #4) > Created attachment 73562 [details] [review] [review] > support uncolored tiling patterns > > The cairo backend does not support uncolored tiling patterns. This caused > uncolored patterns to be drawn in gray. > > The attached patch fixes this. This looks sane to me, but I'm getting a small regression in one of my files, I'll send the pdf file to you by email. Thanks
Created attachment 73644 [details] [review] support uncolored tiling patterns It turns out that we can't use cairo_mask for uncolored patterns since cairo_mask uses the alpha and not the luminosity to mask the source. This patch uses the same solution as uncolored patterns in PSOutputDev. It adds a flag to indicate we are rendering an uncolored pattern and disables setting the color when the flag is true.
(In reply to comment #6) > Created attachment 73644 [details] [review] [review] > support uncolored tiling patterns > > It turns out that we can't use cairo_mask for uncolored patterns since > cairo_mask uses the alpha and not the luminosity to mask the source. > > This patch uses the same solution as uncolored patterns in PSOutputDev. It > adds a flag to indicate we are rendering an uncolored pattern and disables > setting the color when the flag is true. Great, no regressions this time. Thanks! Please, don't use braces for single statement ifs. Feel free to push.
I've removed the surplus braces and pushed.
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.