New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 591137 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

PDF is not loading properly.

Project Member Reported by manoranj...@chromium.org, Mar 1 2016

Issue description

Version: <51.0.2664.0>
OS: <Win7 64-bit OS, Mac OS X 10.11.3, Linux Ubuntu 14.04>

What steps will reproduce the problem?
1. Launch chrome browser and open the attached PDF

What is the expected output? What do you see instead?
PDF should be rendered with out any issues.

This is a recent regression in M50 and below is the narrow bisect info.
https://chromium.googlesource.com/chromium/src/+log/3cf850241bbfa1787af40672c9a57d9837205ed9..b53bed5141561f3b6b41e1f8f6f36375b3922f35

thestig@, could you please look into this PDFium change (https://chromium.googlesource.com/chromium/src/+/4d6ba6e034703125e3b1218d499611ef3919feb6)?

Many thanks!
 
FORM 15 G.PDF
102 KB Download
Here is the attached screenshot for your reference.
PDF_Issue.PNG
55.3 KB View Download
Cc: jchaffraix@chromium.org
Labels: ReleaseBlock-Stable
I am seeing the same issue on Bank of America credit card statements, which renders the PDF reader useless.

To reproduce:
* Sign into bank of america.
* Go to your card account (note: it doesn't work on checking accounts)
* Click on 'Statement and documents'
* Select a statement and 'View PDF'

Due to the scope of the regression, marking as release blocker.
Cc: thestig@chromium.org tsepez@chromium.org dsinclair@chromium.org
Owner: weili@chromium.org
I bisected this locally and it appears to be Remove PageObject's m_Type and add As<Type> functions  (https://pdfium.googlesource.com/pdfium.git/+/7cf13c9c8b9b69b01e5debb5e8dc8b265983dfa8%5E%21/)

weili@ can you take a look?
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 8 2016

Comment 5 by weili@chromium.org, Mar 8 2016

Status: Fixed (was: Assigned)
Project Member

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

Cc: rsesek@chromium.org shrike@chromium.org
 Issue 601485  has been merged into this issue.
Labels: Merge-Request-50
This issue is still present in 50.0.2661.66, which is the latest version of M50 - I guess this fix has not been merged into M50??

Comment 9 by tin...@google.com, Apr 7 2016

Labels: -Merge-Request-50 Merge-Review-50 Hotlist-Merge-Review
[Automated comment] DEPS changes referenced in bugdroid comments, needs manual review.

Comment 10 by tin...@google.com, Apr 7 2016

Labels: -Merge-Review-50 Merge-Approved-50
Merge approved for M50 (branch 2661). Pls go ahead merge.
Please merge your change to M50 branch 2661 by 5:00 PM PST on April 8th,Friday to make into the desktop Stable final build cut. Thank you.
Project Member

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

Labels: -merge-approved-50 merge-merged-2661
The following revision refers to this bug:
  http://goto.ext.google.com/viewvc/chrome-internal?view=rev&revision=86277

------------------------------------------------------------------
r86277 | thestig@google.com | 2016-04-07T23:13:13.860196Z

-----------------------------------------------------------------
Thank you thestig@.
Project Member

Comment 15 by bugdroid1@chromium.org, Apr 8 2016

The following revision refers to this bug:
  http://goto.ext.google.com/viewvc/chrome-internal?view=rev&revision=86286

------------------------------------------------------------------
r86286 | thestig@google.com | 2016-04-08T00:44:42.179892Z

-----------------------------------------------------------------

Sign in to add a comment