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

Issue 723976 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Content of pdf file is not properly displayed after loading the page.

Reported by vku...@etouch.net, May 18 2017

Issue description

Chrome Version:60.0.3103.0 (Official Build)Revision 0d2b07cab5a9dfb8429b42ae5b8ccdfac5d56fd7-refs/heads/master@{#472587} 32/64-bit.
OS:Windows (7,8,10), Mac (10.11.6, 10.12.3), Linux (14.04 LTS)

What steps will reproduce the problem?
(1)Launch chrome and navigate to https://www.nobelprize.org/nobel_prizes/economic-sciences/laureates/2005/press-he.pdf
(2)Observe the content after loading the page.

Actual: Content of pdf file is not properly displayed after loading the page.

Expected: Content of pdf file should be properly displayed after loading the page.

This is a regression issue broken in 'M60' and below is the manual regression range
Good Build: 60.0.3081.0 
Bad Build:  60.0.3082.0 



 
Actual.mov
997 KB Download
Expected.mov
1.2 MB Download

Comment 1 by vku...@etouch.net, May 18 2017

Labels: hasbisect
Owner: jam@chromium.org
Status: Assigned (was: Unconfirmed)
Narrow bisect info:
https://chromium.googlesource.com/chromium/src/+log/bc7ede43a021f66b529c5a00dfcc864f09014762..9371e23b8b90027373ba8c27f60698ee42ef94dd?pretty=fuller&n=50

Suspecting: 467438 ?
Kindly help to reassign if it's not related to your change.
Labels: ReleaseBlock-Stable
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 3 by jam@chromium.org, May 18 2017

Owner: dsinclair@chromium.org
my cl was a test-only change

looks like there was a pdfium roll:
https://chromium.googlesource.com/chromium/src/+/8216b27dfdb3240900e60f2ccc67132eb5889507


Roll src/third_party/pdfium/ 2bbb55162..2e2a4fcd4 (1 commit)

https://pdfium.googlesource.com/pdfium.git/+log/2bbb55162f0c..2e2a4fcd4367

$ git log 2bbb55162..2e2a4fcd4 --date=short --no-merges --format='%ad %ae %s'
2017-04-25 dsinclair Guard against overflow in CFX_BilinearMatrix code.

Cc: dsinclair@chromium.org
Owner: npm@chromium.org
npm@ can you take a look?
Project Member

Comment 5 by bugdroid1@chromium.org, May 18 2017

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium/+/e6792f9901267b2b65611e94713462bc9d0b5191

commit e6792f9901267b2b65611e94713462bc9d0b5191
Author: Nicolas Pena <npm@chromium.org>
Date: Thu May 18 18:48:28 2017

Fix typo in CFX_ImageTransformer::Continue

This CL fixes a typo introduced in:
https://pdfium-review.googlesource.com/c/4495/3/core/fxge/dib/cfx_imagetransformer.cpp#395

Bug:  chromium:723976 
Change-Id: I629d1e61054805f768356fce44c25f40f9346beb
Reviewed-on: https://pdfium-review.googlesource.com/5634
Reviewed-by: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Nicolás Peña <npm@chromium.org>

[modify] https://crrev.com/e6792f9901267b2b65611e94713462bc9d0b5191/core/fxge/dib/cfx_imagetransformer.cpp

Comment 6 by npm@chromium.org, May 18 2017

Status: Fixed (was: Assigned)

Sign in to add a comment