Closed Bug 859027 Opened 11 years ago Closed 11 years ago

Garbled PDF output (bgb/gesamt.pdf)

Categories

(Firefox :: PDF Viewer, defect, P2)

22 Branch
x86_64
Linux
defect

Tracking

()

VERIFIED FIXED
Firefox 23
Tracking Status
firefox21 --- unaffected
firefox22 + verified
firefox23 + verified

People

(Reporter: whimboo, Assigned: emk)

References

Details

(Keywords: regression, Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion][pdfjs-f-fixed-upstream] https://github.com/mozilla/pdf.js/pull/2967)

Attachments

(2 files)

Attached image screenshot
Mozilla/5.0 (X11; Linux x86_64; rv:22.0) Gecko/20130406 Firefox/22.0

Opening the given PDF in Firefox starting with 22.0a2 shows garbled output across the whole document. This is a regression which was not visible with 21.0a2 before.

Open: http://www.gesetze-im-internet.de/bundesrecht/bgb/gesamt.pdf
Regression range:
m-c
good=2013-03-26
bad=2013-03-27
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=456cb08f8509&tochange=178a4a770bb1

Suspected bug:
Ryan VanderMeulen — Bug 852981 - Update pdf.js to version 0.7.390. r=bdahl
Great work Loic! Thanks a lot. Looks like a couple of fixes went into this new pdf.js release. Brendan should probably know which one caused this.
Looking into the fix still, but the regression was from https://github.com/mozilla/pdf.js/pull/2959
Priority: -- → P2
Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion]
Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion] → [pdfjs-c-rendering][pdfjs-d-font-conversion][pdfjs-f-fixed-upstream]
Tracking since this is a PDF regression.
Assignee: nobody → bdahl
I'll backport the fix when the upstream change has been merged into m-c.
No longer blocks: 852981
Depends on: 861399
Assignee: bdahl → VYV03354
Status: NEW → ASSIGNED
Attachment #738758 - Flags: review?(bdahl)
Comment on attachment 738758 [details] [diff] [review]
Uplift #2967: Refactor 'cmap' generation

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 861399
User impact if declined: Internal PDF Viewer will fail to render some PDFs which previous versions of Firefox was able to render correctly.
Testing completed (on m-c, etc.): on m-c and upstream
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none
Attachment #738758 - Flags: approval-mozilla-aurora?
Pinged bdahl about this review request.
Attachment #738758 - Flags: review?(bdahl) → review+
Attachment #738758 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/90cf16bc01d8

Fixed on trunk by bug 861399.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion][pdfjs-f-fixed-upstream] → [pdfjs-c-rendering][pdfjs-d-font-conversion][pdfjs-f-fixed-upstream] https://github.com/mozilla/pdf.js/pull/2967
Target Milestone: --- → Firefox 23
Mozilla/5.0 (X11; Linux x86_64; rv:23.0) Gecko/20130428 Firefox/23.0

Verified as fixed on Ubuntu 12.04 x86_64 with latest Nightly 23.0a1 (Build ID: 20130429030926)
Status: RESOLVED → VERIFIED
Verified as fixed on:
Mozilla/5.0 (X11; Linux x86_64; rv:22.0) Gecko/20100101 Firefox/22.0 (20130514181517)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: