Closed Bug 1720907 Opened 3 years ago Closed 3 years ago

[XFA] Drop-down options are inconsistently unavailable

Categories

(Firefox :: PDF Viewer, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
92 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox90 --- disabled
firefox91 --- verified
firefox92 --- verified

People

(Reporter: danibodea, Assigned: calixte)

References

(Regression)

Details

(Keywords: regression)

Attachments

(3 files)

Note

  • When the user loads a specific PDF file in Adobe Reader and then opens it in Firefox, scrolls it and compares the multiple drop-down elements between each other, he will notice that Adobe will show available options for each, while Firefox will only show options for some of them.

Affected versions

  • Nightly v92.0a1

Affected platforms

  • all

Steps to reproduce

  1. Launch browser.
  2. Flip "pdfjs.enableXfa" to true.
  3. Load the attached PDF file.
  4. Open multiple drop-down elements.

Expected result

  • All drop-down elements have available options to choose from.

Actual result

  • Most of the drop-down elements have no available options to choose from.

Regression range

  • Not a regression.

Additional Information

  • I cannot determine whether this issue is blocking one or the other bug. Please remove the one that isn't being blocked.

Calixte, can you check if this is dynamic?

Flags: needinfo?(cdenizet)

It's a binding issue.
For example the field "Sex" has an element bindItems and it isn't handled correctly.
It's likely a regression due to:
https://github.com/mozilla/pdf.js/commit/112645ea3dd4bcb13e08579f9971918bcd7e0ed7

Before this patch we were reaching:
https://github.com/mozilla/pdf.js/blob/683ce66a48cd54fa48e96e1925277c134859e76c/src/core/xfa/bind.js#L638

and so Binder::_bindItems was correctly called.

Flags: needinfo?(cdenizet)
Assignee: nobody → cdenizet
Status: NEW → ASSIGNED
Priority: -- → P1
No longer blocks: 1717661
Regressed by: 1715638
Has Regression Range: --- → yes
Commit merged into master by GitHub Authored by calixteman (calixteman)

Calixte, was that patch included in the recent pdf.js update we uplifted to beta last week? Thanks

Flags: needinfo?(cdenizet)
Flags: needinfo?(cdenizet)
Depends on: 1722352
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

The fix is verified in Beta v91.0b98 and Nightly v92.0a1 from 2021-07-30.

Status: RESOLVED → VERIFIED
QA Whiteboard: [pdf_xfa_generic]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: