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

Issue 629037 link

Starred by 4 users
Status: Fixed
Owner:
halcanary@chromium.org
Closed: Jul 2016
Cc:
ajha@chromium.org
█ kavvaru@chromium.org
nyerramilli@chromium.org
█ tsergeant@chromium.org
█ msrchandra@chromium.org
dpa...@chromium.org
█ ranjitkan@chromium.org
█ durga.behera@chromium.org
hcarmona@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Print preview for 'sync node browser' section is not displayed properly in chrome://sync-internals/

Reported by vku...@etouch.net, Jul 18 2016 
Chrome Version:54.0.2799.0 (Official Build)8acb9d5f2f71c3df8e64b13ee321a9d194e89785-refs/heads/master@{#405947} (64-bit) 
OS:Mac (10.10.5, 10.11.4)

What steps will reproduce the problem?
(1)Launch chrome and login into browser with valid credentials.
(2)Navigate to chrome://sync-internals/ ,click on 'sync node browser' tab
(3)Click on "autofill profile" option such that data appears,press cmd+p and observe the print preview.

Actual: Print preview for 'sync node browser' section is not displayed properly in chrome://sync-internals/

Expected: Print preview for 'sync node browser' section should be displayed properly in chrome://sync-internals/.

This is a regression issue broken in 'M54' and will soon update other info.

Comment 1 by vku...@etouch.net, Jul 18 2016

Cc: roc...@chromium.org
Labels: hasbisect OS-Linux
Owner: lshang@chromium.org
Status: Assigned (was: Unconfirmed)
Manual regression range:
Good Build: 54.0.2797.0
Bad Build:  54.0.2798.0

CL:
https://chromium.googlesource.com/chromium/src/+log/54.0.2797.0..54.0.2798.0?pretty=fuller&n=10000
(Unable to narrow down the range using tool since not able to sign into chromium builds)

Suspecting: 405885 or 405669 ?
Kindly help to re-assign, if your changes are not cause for this issue.

Note:Issue not seen on Win OS.
ActualPrint.png
349 KB View Download
Actual_Print.mov
3.0 MB Download
Expected_Print.mov
2.7 MB Download

Comment 2 by msrchandra@chromium.org, Jul 18 2016

Labels: ReleaseBlock-Beta
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 3 by msrchandra@chromium.org, Jul 18 2016

Cc: kavvaru@chromium.org durga.behera@chromium.org ajha@chromium.org
 Issue 629054  has been merged into this issue.

Comment 4 by msrchandra@chromium.org, Jul 18 2016

Labels: -ReleaseBlock-Beta ReleaseBlock-Dev
This issue is also seen on Chrome as per the Bug Id -- 629054.
Changing the Release Block to Dev from Beta as the issue is seen on 3 different OS.
Please change the Release block priority if not the case.
Thank You.

Comment 5 by lshang@chromium.org, Jul 19 2016

Owner: thestig@chromium.org
My CL added a callback list in sync process and the callback list has not yet been registered in this CL(which I will do in a follow up CL out for reviewing). I don't think it's related to this bug, this bug is more like a printing or displaying problem.

My guess is 405819, based on 'printing' in the CL's description.

Comment 6 by manoranj...@chromium.org, Jul 19 2016 

thestig@, could you please see if this is related to your change?

Thank you!

Comment 7 by thestig@chromium.org, Jul 21 2016

Cc: -roc...@chromium.org tsergeant@chromium.org hcarmona@chromium.org dpa...@chromium.org
Owner: ----
Status: Untriaged (was: Assigned)
lshang: Uh, did you actually look at r405819 ? That can't be the cause.

It's most likely one the of settings page changes. r405666, r405869, or r405678. +settings folks to triage.

Comment 8 by tsergeant@chromium.org, Jul 22 2016 

None of these look like obvious culprits to me, so I'll do a manual bisect and report back soon.

Comment 9 by thestig@chromium.org, Jul 22 2016 

tsergeant: Thanks for bisecting. I hope it's not r405819. ;)

Comment 10 by tsergeant@chromium.org, Jul 22 2016

Owner: halcanary@chromium.org
Status: Assigned (was: Untriaged)
Alright, looks like it's the Skia roll at r405886.

Assigning to halcanary@, who authored some SkPDF related commits in that roll.

Comment 11 by halcanary@google.com, Jul 22 2016 

simplified test case attached:

This is a problem with monospaced fonts.  

https://crrev.com/2162023002 will fix this.  I will review that next chance I get.
crbug_629037.html
205 bytes View Download

Comment 12 by ranjitkan@chromium.org, Jul 25 2016

Cc: ranjitkan@chromium.org
Labels: TE-Verified-54.0.2806.0 TE-Verified-M54
Recheched this on chrome version 54.0.2806.0 on MAC 10.11.5 and Ubuntu 14.04, issue seems to be fixed as per above comment. Screen shot attached for the same.

Adding TE-Verified labels.
Screen Shot 2016-07-25 at 3.45.39 PM.png
322 KB View Download

Comment 13 by halcanary@chromium.org, Jul 25 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment