1778692 - Contents like barcode or image placeholder are not rendered in pdf viewer

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect)

Description

[Catalin Sasca, Desktop Test Engineering [:csasca]]

Attachment: barcode pdf missing.gif

Found in

• Firefox 103.0b6

Affected versions

• Firefox 103.0b6
• Firefox 104.0a1

Affected platforms

• Windows 7
• macOS 11.6.7
• Ubuntu 22.04

Steps to reproduce

1. Launch Firefox
2. Access this pdf and scroll to the second page

Expected result

• The barcode lines and the "add image here" placeholder are visible

Actual result

• The barcode lines and the "add image here" placeholder are missing

Regression range

• Will see for a regression

Additional notes

• The issue can be seen in the attachment
• The barcode and image placeholder are rendered in print preview as expected

Comment 1

[Calixte Denizet (:calixte)]

There are several issues with this pdf:

• there are several annotations on page 2 which are not rendered;
• the barcode isn't rendered correctly because we don't use the font from the pdf to render text content in the textfield.

For now I'll fix the first issue, about the second it's a known bug:
https://github.com/mozilla/pdf.js/issues/13329

Comment 2

[Calixte Denizet (:calixte)]

Attachment: interactiveform_enabled.pdf

Comment 3

[GitHub Bugzilla PR Linker]

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

Comment 4

[Alin Ilea, Desktop Test Engineering]

Attachment: 2022-07-08_15h47_30.mp4

Also related to this pdf, the "Reason for Leaving" checkboxes don't work. Just let me know if this is a known issue or I should file a new bug for it.
Thanks.

Comment 5

[Calixte Denizet (:calixte)]

:Alin, yep it's what I meant by "there are several annotations on page 2 which are not rendered".

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 1778692 - [Annotation] A push button can have no action.

Backport of upstream commit:
https://github.com/mozilla/pdf.js/pull/15153

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9285633 [details]
Bug 1778692 - [Annotation] A push button can have no action.

Beta/Release Uplift Approval Request

• User impact if declined: Missing text on some PDFs (see duplicate bug).
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Very minor change, has good upstream test coverage, already verified on Nightly.
• String changes made/needed:
• Is Android affected?: No

Comment 9

[Donal Meehan [:dmeehan]]

Comment on attachment 9285633 [details]
Bug 1778692 - [Annotation] A push button can have no action.

Approved beta uplift request, available on the beta channel with 103.0 RC1.

Comment 10

[Donal Meehan [:dmeehan]]

https://hg.mozilla.org/releases/mozilla-beta/rev/fdf7d6feec42

Comment 11

[Virgil Sangerean]

It seems that now both on latest Nightly And 103.0 Firefox the issue is not fixed, but the behavior is slightly modified.
A string of "0" is now displayed.

Please see attached GIF.

Comment 12

[Virgil Sangerean]

Attachment: Stringof0s.gif

Comment 13

[Marco Castelluccio [:marco]]

This was the case with older versions too, could you file a separate bug? We will use this bug only to track the regression of some elements not being rendered at all.

Comment 15

[Virgil Sangerean]

Considered as "Verified - Fixed" since another issue was opened.

Comment 16

[Brindusa Tot, Desktop QA]

Based on previous comments, the remaining issue will be verified in bug 1780548. Updating the flags accordingly.