New issue
Advanced search Search tips

Issue 625848 link

Starred by 7 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug-Regression



Sign in to add a comment

PDF text scrambled when viewed with Chrome

Reported by j...@schiresandwhite.com, Jul 5 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0

Steps to reproduce the problem:
1. Open file to view pdf (link to pdf is private)
2. 
3. 

What is the expected behavior?
Text should be readable.
Text is fine using FireFox, Edge, and Acrobat.

What went wrong?
Text is all over the place.

Did this work before? Yes version 49

Chrome version: 51.0.2704.106 m  Channel: n/a
OS Version: 10.0
Flash Version: Shockwave Flash 22.0 r0
 
Bridle_Pass_Tree_Installation_Information.pdf
15.2 KB Download
Components: Internals>Plugins>PDF
Labels: -OS-Windows OS-All
- The PDF has non-embedded type 1 fonts.
- pdfium_test fails to render the PDF.
Status: Untriaged (was: Unconfirmed)
Regressed in r382426 -> https://pdfium.googlesource.com/pdfium.git/+log/bf067b4..4161c5c
Owner: weili@chromium.org
- Regressed in https://codereview.chromium.org/1801383002 in core/fxcodec/jbig2/JBig2_TrdProc.cpp (So not actually font related - the "text" is an image)
- pdfium_test can render the PDF if I force the non-linearized rendering path.
I take back the part about the text being an image. The text looks selectable, so I'm not 100% sure and I haven't looked closer.

Comment 5 by weili@chromium.org, Sep 22 2016

Status: Assigned (was: Untriaged)
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 12 2016

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium.git/+/8d5d00b7270c198f9bfafcbd764ba0eff70fd5b1

commit 8d5d00b7270c198f9bfafcbd764ba0eff70fd5b1
Author: weili <weili@chromium.org>
Date: Mon Dec 12 21:58:37 2016

Fix JBig2 decoding logic

While decoding among instances, variable "FIRSTS" should hold its
value, not be reset. This was accidently changed by earlier
refactoring.

BUG= chromium:625848 , pdfium:636 

Review-Url: https://codereview.chromium.org/2569023002

[modify] https://crrev.com/8d5d00b7270c198f9bfafcbd764ba0eff70fd5b1/core/fxcodec/jbig2/JBig2_TrdProc.cpp

Project Member

Comment 7 by bugdroid1@chromium.org, Dec 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2c27388b814cc57d257c01ea7ace56fd206bba50

commit 2c27388b814cc57d257c01ea7ace56fd206bba50
Author: pdfium-deps-roller <pdfium-deps-roller@chromium.org>
Date: Tue Dec 13 03:38:19 2016

Roll src/third_party/pdfium/ 9468639bc..8df023489 (6 commits).

https://pdfium.googlesource.com/pdfium.git/+log/9468639bc3c4..8df02348924f

$ git log 9468639bc..8df023489 --date=short --no-merges --format='%ad %ae %s'
2016-12-12 caryclark fix linux build
2016-12-12 weili Fix JBig2 decoding logic
2016-12-12 reed add flag for future Skia CL
2016-12-12 caryclark sync with latest skia api
2016-12-12 weili Fix dotted line drawing for GDI device driver
2016-12-12 reed add flag for SkClipOps

BUG= 625848 , 539533 

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2566283004
Cr-Commit-Position: refs/heads/master@{#438033}

[modify] https://crrev.com/2c27388b814cc57d257c01ea7ace56fd206bba50/DEPS

Comment 8 by weili@chromium.org, Dec 13 2016

Status: Fixed (was: Assigned)

Comment 9 by weili@chromium.org, Dec 28 2016

Cc: kkaluri@chromium.org weili@chromium.org
Issue 676831 has been merged into this issue.

Comment 10 by npm@chromium.org, Feb 7 2017

Cc: ligim...@chromium.org
 Issue 687271  has been merged into this issue.
Labels: M-56
As per issue :  687271  , this is broken only in M56. Please request a merge to M56, we can take the patch if there is any upcoming respins.

Sign in to add a comment