799695 - Background too dark

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect)

Description

[Mark Côté [:mcote]]

The pdf at http://www.5saisonssushi.com/inset.pdf isn't rendering correctly--as compared to my desktop pdf viewer, the background is much darker, to the point of making part of the foreground text unreadable.

Comment 1

[Mihaela Velimiroviciu (:mihaelav)]

This causes documents with some backgrounds to be unreadable.
Other documents with this issue:
http://www.koryu.or.jp/nihongo/ez3_graphics.nsf/DBBDC3186266DE0049257791002967FD/$File/1-24.pdf?OpenElement
http://www.numark.jp/products/ns6/data/ns6_product_overview_JP_web.pdf

Reproducible on other platform, as well (tested wit Win Vista x86).

Comment 2

[u279076]

What I'm seeing in the latest Nightly under Windows 8:

> http://www.5saisonssushi.com/inset.pdf
Black "5 SAISON" covers some of the text.
No info-bar appears but I can manually download the PDF.
Adobe Reader does not render the black "5 SAISON" text.

> http://www.koryu.or.jp/nihongo/ez3_graphics.nsf/DBBDC3186266DE0049257791002967FD/$File/1-24.pdf?OpenElement
Black and white flower image renders making the text unreadable.
No info-bar appears but I can manually download the PDF.
Adobe Reader renders the flower image as light grey, making the text readable.

> http://www.numark.jp/products/ns6/data/ns6_product_overview_JP_web.pdf
Dark grey "NS6" renders on top of some of the text.
Info-bar appears allowing me to load the PDF in Adobe Reader.
Adobe Reader does not render the "NS6" text.

In my opinion, this is definitely a bug we should fix (if nothing else than to achieve parity with Reader) but I don't think fixing this should block shipping PDF.js in Firefox 19. One can work around this fairly easily by using another PDF viewer. However, I do think we should be offering the info-bar far more aggressively at launch.

Comment 3

[Alex Keybl [:akeybl]]

Not a feature blocker, but we'll track for upcoming releases.

Comment 4

[Douglas Bagnall]

Another one: http://fann.sf.net/fann_en.pdf

Comment 5

[Masatoshi Kimura [:emk]]

Console logs the message "Warning: TODO: Support non-isolated groups.", but the display was improved.
Sanitizer rejects fonts from some PDFs, but it is a separate problem.

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Too late for speculative fixes on FF20 so this will remain unfixed for that release.

Comment 8

[Masatoshi Kimura [:emk]]

Fixed by bug 852981.

Comment 9

[Brendan Dahl [:bdahl]]

This patch was rather large and had follow up fixes that we'd also need to merge.  I would rather not uplift for FF21 and just let it ride trains and be fixed in FF22.

Comment 10

[bhavana bajaj [:bajaj]]

Marking this wontfix for Fx 21 , per comment 9.