1739502 - XFA PDF not opening properly

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect, P1)

Description

[Harry]

Attachment: XFA pdf.pdf

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

Steps to reproduce:

Open XFA PDF (Attached to this ticket)

Actual results:

All pages work fine except page 6

Expected results:

Page 6 would have loaded fine.

Comment 1

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

The Bugbug bot thinks this bug should belong to the 'Firefox::PDF Viewer' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Comment 2

[Calixte Denizet (:calixte)]

There are few issues in this pdf:

• on page 4, the left margin is missing;
• for any reason, the contents of page 6, 7 and 8 are overlapped.

Comment 3

[Calixte Denizet (:calixte)]

The issue with overlapping pages is because of the breakBefore element:

<breakBefore targetType="contentArea" target="flowed.flowedContent" startNew="1"/>

the startNew=1 is not fulfilled and consequently all the subforms are in the same contentArea.
The fix consist to ensure that create a new page when the target contentArea is the same as the current contentArea.

Comment 4

[GitHub Bugzilla PR Linker]

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

Comment 5

[GitHub Bugzilla PR Linker]

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

Commit merged into master by GitHub
Authored by calixteman (calixteman)

Comment 6

[Marco Castelluccio [:marco]]

There are still some problems on this PDF.

Comment 7

[Marco Castelluccio [:marco]]

Attachment: Screenshot

Comment 8

[Calixte Denizet (:calixte)]

It's because of a missing Calibri font: I'll fix.

Comment 9

[Kenan]

Fixed by:
Bug 1743838 - Update pdf.js to version 2.12.248 r=pdfjs-reviewers,calixte

Differential Revision: https://phabricator.services.mozilla.com/D132586

2021-12-17T10:49:07.708000: DEBUG : Did not find a branch, checking all integration branches
2021-12-17T10:49:07.719000: INFO : The bisection is done.
pushlog_url: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=fddb757f9137bb916c1dbe89008928d9b6ac4343&tochange=b5036885cdb973bc037e562b1e35f90649efe430

(between 2021-12-02 and 2021-12-03)

Comment 10

[Marco Castelluccio [:marco]]

Still problematic for me on Linux, probably because you have the font that I'm missing.

Comment 11

[Kenan]

So, we might have different bugs here.
The bug on pages 6,7 and 8 (overlapping text) was fixed for me. The bug you mentioned in comment 7 was not reproducible for me.

Comment 12

[Marco Castelluccio [:marco]]

(In reply to Kenan from comment #11)

So, we might have different bugs here.
The bug on pages 6,7 and 8 (overlapping text) was fixed for me. The bug you mentioned in comment 7 was not reproducible for me.

Yes, they are two separate bugs. I'll open a new one.