1755507 - PDF pictures is showing black

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect, P1)

Description

[baslan]

Attachment: Chreom vs. Firefox.png

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:97.0) Gecko/20100101 Firefox/97.0

Steps to reproduce:

I tried to open PDF file here:
https://curriculum.harmonytx.org/WebShare/Math/Curriculum/Geometry/1o_u-IgNsIkr7j8P3jH-lJjnHhIMeqYPSmXgDeAg9VHyFlNJMK3MfrexOvdXWoxQ16oDkTV3RoPAXM9wzE-_rp47Yl9lejsPuEkb4cq8W8TNV9q/Homework_Helper/12-1.pdf#page=2

Actual results:

The pictures won't show properly.

Expected results:

You can compare how it show in Chrome and Firefox in the attached image.

Comment 1

[Marco Castelluccio [:marco]]

Attachment: 12-1.pdf

Comment 2

[Marco Castelluccio [:marco]]

It is working in Firefox 86, I'm trying to find a regression range.

Comment 3

[Marco Castelluccio [:marco]]

I found bug 1739229 with mozregression.

Most likely it was https://github.com/mozilla/pdf.js/pull/14175. Brendan, could you take a look?

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1739229

Comment 5

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pdf.js/pull/14569

Comment 6

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub master push: https://github.com/mozilla/pdf.js/commit/530af48b8e32ba153962604006ac680aa1a08bcd

Commit merged into master by GitHub
Authored by Jonas Jenwald (Snuffleupagus)

Comment 7

[Marco Castelluccio [:marco]]

We could uplift this to 98 if it's not too late.

Comment 8

[Pascal Chevrel:pascalc]

(In reply to Marco Castelluccio [:marco] from comment #7)

We could uplift this to 98 if it's not too late.

We have one beta left, if it's a safe patch I could take it.

Comment 9

[Marco Castelluccio [:marco]]

(In reply to Pascal Chevrel:pascalc from comment #8)

(In reply to Marco Castelluccio [:marco] from comment #7)

We could uplift this to 98 if it's not too late.

We have one beta left, if it's a safe patch I could take it.

Redirecting needinfo to Brendan.

Comment 10

[Brendan Dahl [:bdahl]]

Attachment: Bug 1755507 - Update pdf.js to version 2.13.94 r=#pdfjs-reviewers

Comment 11

[Brendan Dahl [:bdahl]]

I'm realizing now we haven't even landed this change in nightly yet, so I'm not sure we'll want to uplift.

Comment 12

[Pascal Chevrel:pascalc]

Let's not uplift, thanks.

Comment 13

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

I managed to reproduce this issue on Firefox 98.0(20220304153049) on Win10 64-bit using the attached pdf. Confirmed as fixed on Firefox 99.0b6(20220320185956) and Nightly 100.a01 (20220321214243) on Win10 64-bit, macOS 11, Linux x86_64(Ubuntu 20.04).