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

Issue 594111 link

Starred by 3 users
Status: Fixed
Owner:
tsepez@chromium.org
Closed: Mar 2016
Cc:
nyerramilli@chromium.org
thestig@chromium.org
█ ranjitkan@chromium.org
dsinclair@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : In Print Preview pages are seen blank.

Reported by yfulgaon...@etouch.net, Mar 11 2016 
Chrome version : 51.0.2674.0  f9be006501afecf6f58b3d638999e03613314c84-refs/heads/master@{#380495} 32/64 bit
OS : Windows(Win 7 aero enabled)

Steps : 
1. Launch Chrome, go to settings and click on 'Show Advanced settings'.
2. Hit Ctrl A and give print command using Ctrl P.
3. Scroll down the page and observe.

Actual : Only first page contents are seen in Print Preview and remaining 2 pages are seen blank.
Expected : All pages should be properly seen in Print Preview.

This is a regression issue broken in 'M-51' and below is the manual regression and narrow bisect info.
Good build : 51.0.2673.0 
Bad build : 51.0.2674.0 

Note : Will soon update the bisect info. and issue status on Mac/Linux OS.
Actual_print.mp4
891 KB Download

Comment 1 by yfulgaon...@etouch.net, Mar 11 2016

Labels: hasbisect OS-Linux
Owner: danakj@chromium.org
Status: Assigned (was: Unconfirmed)
Below is the narrow bisect info : 

Narrow Bisect :
https://chromium.googlesource.com/chromium/src/+log/e7e19cb2ab8b42feb7fa9d25fd313ec14d51b28b..82df1afea37d14685cc18ea1ab651b6859afd2e4?pretty=fuller&n=10000

Suspecting : r380441  from Narrow Bisect

@danakj : Could you please help to reassign if your change is not the cause for this issue

Note : Above issue is not seen on Mac OS.
Expected_print.mp4
713 KB Download

Comment 2 by ranjitkan@chromium.org, Mar 11 2016

Labels: ReleaseBlock-Dev
Adding release block label, please undo if not the case.

Comment 3 by danakj@chromium.org, Mar 11 2016

Owner: yfulgaon...@etouch.net
My CL renames a method, please read the CLs you blame. :)

Comment 4 by manoranj...@chromium.org, Mar 11 2016

Owner: tsepez@chromium.org
Had an offline chant with Tom and seems like this is the culprit: https://chromium.googlesource.com/chromium/src/+/1e379580207e1bc923773ed8cf2f2fa9cc020394

Comment 5 by tsepez@chromium.org, Mar 11 2016 

reproduced on Linux/ToT (410861d3b5) when setting GYP_DEFINES="pdf_enable_xfa=1" (as did the CL in question).  Its fine otherwise.

Comment 6 by tsepez@chromium.org, Mar 11 2016

Cc: thestig@chromium.org dsinclair@chromium.org

Comment 7 by yfulgaon...@etouch.net, Mar 14 2016 

Rechecked above issue in latest canary build (51.0.2676.0) on windows machine, it seems to be fixed and working as intended.

Attaching the screen cast of the same
canary_behavior.mp4
1.2 MB Download
canary_behavior.png
231 KB View Download

Comment 8 by ligim...@chromium.org, Mar 14 2016

Status: WontFix (was: Assigned)
Closing as per #7. Please reopen if needed.

Comment 9 by manoranj...@chromium.org, Mar 14 2016

Status: Fixed (was: WontFix)
This issue got 'Fixed' because of this revert (https://chromium.googlesource.com/chromium/src/+/1e379580207e1bc923773ed8cf2f2fa9cc020394).

Comment 10 by tsepez@chromium.org, Mar 14 2016

Labels: -ReleaseBlock-Dev
Status: Assigned (was: Fixed)
No longer a release blocker due to rollback, but we need to find the underlying cause to allow rollforward in the future.
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 16 2016 

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

commit b291827bfaa43278f8c7327de59de7cdaca21d1c
Author: ochang <ochang@chromium.org>
Date: Wed Mar 16 19:05:54 2016

Roll PDFium 3cc1802..30410ce

https://pdfium.googlesource.com/pdfium.git/+log/3cc1802..30410ce

TBR=tsepez@chromium.org
BUG= 594111 

Review URL: https://codereview.chromium.org/1807933002

Cr-Commit-Position: refs/heads/master@{#381507}

[modify] https://crrev.com/b291827bfaa43278f8c7327de59de7cdaca21d1c/DEPS

Comment 12 by tsepez@chromium.org, Mar 16 2016 

We're going to be re-landing this shortly.  Please watch for a re-occurrence (though I think it should be fixed).
►

Sign in to add a comment