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

Issue 636240 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 630909
Owner:
Not on Chrome anymore
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Print preview is not proper for saved ‘Frame source’.

Reported by yfulgaon...@etouch.net, Aug 10 2016

Issue description

Chrome Version : 54.0.2825.0 (Official Build) 3d58aef05951368778eda5aa835969a8f04bf285-refs/heads/master@{#410913} (64-bit)
OS: Mac (10.10.5, 10.11.5), Windows (7,8,8.1,10), Linux(14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome, navigate to chrome://help, open context menu and select ‘View frame source’.
2. Save the page using Cmd S and open the saved page.
3. Hit Cmd P and observe the print preview contents.

Actual : Print preview is not proper for saved ‘Frame source’.
Expected : Print preview should be proper for saved ‘Frame source’.

This is a regression issue broken in M-54, below is the Manual Regression and Narrow Bisect info:

Manual Regression :
Good build: 54.0.2820.0
Bad build: 54.0.2821.0

Narrow Bisect : 
https://chromium.googlesource.com/chromium/src/+log/2862d3423c1e0d9c7d2817e7ec0d0efcba5ae5df..070f89c9808629a65f02a0f3c818a31c873b3e9e?pretty=fuller&n=10000

Suspecting: r410175 from narrow bisect
 
Actual_Print_Preview.mov
7.5 MB Download
Expected_Print_Preview.mov
7.6 MB Download
Labels: ReleaseBlock-Stable
Adding Release block label, please undupe if not the case.

Comment 2 by nainar@chromium.org, Aug 10 2016

Cc: thestig@chromium.org esprehn@chromium.org
Taking a look at this.
Just to update the bug with latest behavior, Issue still able to repro on Windows-10 using chrome latest canary M54-54.0.2831.0 .

@nainar: Gentle ping, Could you please provide an update on this issue. Issue is marked with a blocker label and M54 is going to be branched soon.

Issue is still observed on Canary version 54.0.2836.0 on Windows 7 and MAC 10.11.6

Thanks.!

Comment 5 by nainar@chromium.org, Aug 23 2016

Mergedinto: 630909
Status: Duplicate (was: Assigned)
Sorry, I thought I had marked this as WontFix some time ago. 
Couple of reasons why I am marking this as WontFix:
1. This change makes us interoperable with FF and IE (will confirm Edge) in terms of print. Attaching a FF sample which is the same as ours with this bug in fact. 
2. For the rest of the reasoning see the bug I am duping this against. 
Please note that this issue should be marked as a dupe of 630909 which I have marked as WontFix as well. If my assessment is wrong do let me know. Thanks!

Comment 6 by nainar@chromium.org, Aug 23 2016

Sorry, I thought I had marked this as WontFix some time ago. 
Couple of reasons why I am marking this as WontFix:
1. This change makes us interoperable with FF and IE (will confirm Edge) in terms of print. Attaching a FF sample which is the same as ours with this bug in fact. 
2. For the rest of the reasoning see the bug I am duping this against. 
Please note that this issue should be marked as a dupe of 630909 which I have marked as WontFix as well. If my assessment is wrong do let me know. Thanks!

Comment 7 by nainar@chromium.org, Aug 23 2016

FF print preview
test1.pdf
37.3 KB Download
Similar issue is seen in some other pages which have more number of links in it.
Ex:chrome://view-http-cache/ , chrome://cache

Open any of above two URLS  with long content in it >> Ctrl+p for print preview and observe.

@nainar: Please confirm whether this is same as this issue[intended] or else will raise a new bug for this.

Attaching screenshot for reference.
Actual_corrupted print preview.png
161 KB View Download
Expected_printpreview.png
355 KB View Download

Comment 9 by nainar@chromium.org, Aug 26 2016

Yup - same as the above. Keeping it here is ok. I will try to raise a bug to make sure that the pages land a fix to this on their end. 
Cc: tkonch...@chromium.org
 Issue 655118  has been merged into this issue.

Sign in to add a comment