New issue
Advanced search Search tips

Issue 718785 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Spacing of span elements in print mode is broken dependent on font-size

Reported by timo.nac...@gmail.com, May 5 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/58.0.3029.81 Chrome/58.0.3029.81 Safari/537.36

Steps to reproduce the problem:
1. Put the following markup into a html document
<div style="font-size: 14pt;">
  <span>abcdefghijklmnopqrstuvwx</span><span>yz</span>
</div>
2. Observe that in normal view, there is no space between the x and the y.
3. In the Chromium Menu, open File->Print... and observe that in the print preview, there is a significant space between the x and y

What is the expected behavior?
Normal view and print preview should be identical. There should not be a space between the x and the y.

What went wrong?
An additional space occurs between the x and the y in print mode.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 58.0.3029.81  Channel: stable
OS Version: Ubuntu 16.04
Flash Version: Shockwave Flash 25.0 r0

The effect depends on the font size:
10 pt: x and y overlap in print mode
11 pt: additional space in print mode
12 pt: everything fine
13 pt: x and y overlap in print mode
14 pt: additional space in print mode
 
bug.html
1.4 KB View Download
Cc: kkaluri@chromium.org
Labels: Needs-Feedback
Tested this issue on Ubuntu 14.04 with chrome #58.0.3029.96 with provided html.

Didn't observe any spacing between x & y in print preview.

Attaching the screen-cast for your reference.

timo.nachstedt@ could you re-try the same scenario on clean profile with no apps/extensions and let us know your observations.


Thank You...

Issue 718785.mp4
2.4 MB View Download
Thanks for taking up this issue.

I tried a fresh guest account at Ubuntu and still observed the issue (see screen-cast chromium.mp4)

Furthermore, the problem is not only present for Chromium, but also for Chrome ( screen-cast chrome.mp4)
chromium.mp4
2.4 MB View Download
chrome.mp4
2.0 MB View Download
Project Member

Comment 3 by sheriffbot@chromium.org, May 8 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: bunge...@chromium.org rkalavakuntla@chromium.org
Components: UI>Browser>PrintPreview
Labels: -Type-Bug -Pri-2 M-60 hasbisect Pri-1 Type-Bug-Regression
Owner: halcanary@chromium.org
Status: Assigned (was: Unconfirmed)
Tested this issue on Ubuntu 16.04 with version stable #58.0.3029.81,latest stable #58.0.3029.96 and M-60(#60.0.3093.0 dev),Able to reproduce the issue.
Unable to reproduce the issue on Ubuntu 14.04,Mac OS X 10.12.4 and Windows 7 with version stable #58.0.3029.81.
 
This is Regression issue broken in M-54.

Manual Bisect Info:
================== 
Good Build: 54.0.2797.0 
Bad Build : 54.0.2798.0

You are probably looking for a change made after 405882 (known good), but no later than 405888 (first known bad).
CHANGE LOG URL:
---------------
 https://chromium.googlesource.com/chromium/src/+log/6a0ba49889195d656ad665c2ec4abd9a5ecca33f..88b70de70189fb00e8ff742c19960907a77e32a1

Suspecting: https://codereview.chromium.org/2150393002 from Change log 

@halcanary : Please feel free to re-assign if its not related to your change.


I'll take a look.
I thought I had fixed this in M59!  https://review.skia.org/9324
I don't now have a system that can reproduce this issue.  I need a particular version of Freetype with a particular patch included.  Working on that.

rkalavakuntla@, can you please attach an affected PDF file (from "Save As PDF" https://github.com/HalCanary/howto-save-as-pdf) from m59 or m60?  Thanks!
never mind.  I've procured them myself (see attached).

As I thought, it looks like it is fixed in m60.
crbug_718785_ubuntu16_04_2_Chrome_m58.pdf
23.8 KB Download
crbug_718785_ubuntu16_04_2_Chrome_m60.pdf
24.3 KB Download
Status: Fixed (was: Assigned)
I'm marking this as fixed in m59.  I doubt anyone wants to push a fix back to m58 (if you do, simply land this: https://review.skia.org/16866 ). 

If you can send me a PDF from m59+ that has this bug, I will re-open.

Sign in to add a comment