Closed
Bug 859027
Opened 11 years ago
Closed 11 years ago
Garbled PDF output (bgb/gesamt.pdf)
Categories
(Firefox :: PDF Viewer, defect, P2)
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)
118.19 KB,
image/png
|
Details | |
8.35 KB,
patch
|
bdahl
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Comment 2•11 years ago
|
||
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.
Comment 3•11 years ago
|
||
Looking into the fix still, but the regression was from https://github.com/mozilla/pdf.js/pull/2959
Updated•11 years ago
|
Priority: -- → P2
Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion]
Updated•11 years ago
|
Whiteboard: [pdfjs-c-rendering][pdfjs-d-font-conversion] → [pdfjs-c-rendering][pdfjs-d-font-conversion][pdfjs-f-fixed-upstream]
Comment 5•11 years ago
|
||
Tracking since this is a PDF regression.
Assignee | ||
Comment 6•11 years ago
|
||
I'll backport the fix when the upstream change has been merged into m-c.
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 7•11 years ago
|
||
Assignee | ||
Comment 8•11 years ago
|
||
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?
Comment 9•11 years ago
|
||
Pinged bdahl about this review request.
Updated•11 years ago
|
Attachment #738758 -
Flags: review?(bdahl) → review+
Updated•11 years ago
|
Attachment #738758 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 10•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/90cf16bc01d8 Fixed on trunk by bug 861399.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-firefox23:
--- → fixed
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
Comment 11•11 years ago
|
||
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)
Updated•11 years ago
|
Updated•11 years ago
|
Status: RESOLVED → VERIFIED
Comment 12•11 years ago
|
||
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.
Description
•