869140 - Wrong colors applied to PDF elements (shading type and Smask)

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect, P3)

Description

[Loic]

Attachment: psychology-of-color.pdf

Open the PDF attached to the bug (or nowsourcing.com/blog/wp-content/uploads/2012/01/psychology-of-color.pdf).

Result: The house rooms have bad background colors and at the end of the document, the right rectangle should be pink instead of white.

Error console:
[22:00:25.740] "PDF c342eca0b644b14eb4441d1b1c21ffc7 [1.5 Adobe PDF library 9.00 / Adobe Illustrator CS4] (PDF.js: 0.8.47)"
[22:00:25.814] "Warning: TODO: graphic state operator SMask"
[22:00:26.130] "Warning: TODO: Unsupported shading type: 7"
[22:00:26.727] "Warning: TODO: graphic state operator SMask"
[22:00:26.729] "Warning: TODO: Unsupported shading type: 7"
[22:00:26.740] "Warning: TODO: graphic state operator SMask"
[22:00:26.741] "Warning: TODO: Unsupported shading type: 7"
[22:00:27.036] "Warning: TODO: graphic state operator SMask"
[22:00:27.038] "Warning: TODO: Unsupported shading type: 7"
[22:00:27.148] "Warning: TODO: graphic state operator SMask"
[22:00:27.154] "Warning: TODO: Unsupported shading type: 7"
[22:00:27.458] "Warning: TODO: graphic state operator SMask"
[22:00:27.794] "Warning: TODO: Support non-isolated groups."

Comment 1

[Yury Delendik (:yury)]

Smask was fixed by https://github.com/mozilla/pdf.js/pull/4161 and shading pattern was fixed by https://github.com/mozilla/pdf.js/pull/4192 . Changing to a performance issue.

Comment 2

[Loic]

Opening this PDF make pdf.js not responding:
"A script on this page may be busy, or it may have stopped responding. You can stop the script now, or you can continue to see if the script will complete.
Script: resource://pdf.js/build/pdf.js:5249"

Reg. range:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5e0a22097bff&tochange=d09f9a9f81ae

I guess Bug 965861 is suspected.

Should I clone this bug?

Comment 3

[Yury Delendik (:yury)]

(In reply to Loic from comment #2)
> 
> Should I clone this bug?

No. The whiteboard is changed to reflect that.

Comment 4

[Loic]

The PDF is displayed well in FF42 and the issue about not responding is fixed too. I close the bug.