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

Issue 667453 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

SVG text strokeDash render incorrectly

Reported by andreabo...@gmail.com, Nov 21 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.98 Safari/537.36

Steps to reproduce the problem:
1. open the svg attached in the file
2. observe the red stroke dash
3. compare the result with firefox rendering

What is the expected behavior?
stroke dash looks like firefox
stroke dash gaps are large 3, should be as big as the stroke that is big 3.
In chrome is half.
stroke dash on paths works correctly. on text not.

What went wrong?
The dash array look incorrect

Did this work before? Yes i do not notice. was working some time ago.

Does this work in other browsers? Yes

Chrome version: 54.0.2840.98  Channel: stable
OS Version: OS X 10.11.5
Flash Version: Shockwave Flash 23.0 r0
 
svgchromium.svg
659 bytes Download
Screen Shot 2016-11-21 at 9.16.02 PM.png
82.3 KB View Download
Screen Shot 2016-11-21 at 9.16.49 PM.png
29.3 KB View Download

Comment 1 by f...@opera.com, Nov 21 2016

Labels: -OS-Mac
Status: Available (was: Unconfirmed)
I'd be good to get a notion of when it was working previously. I guess this will be directly obvious on a HiDPI/Retina device.

Comment 2 by pdr@chromium.org, Nov 21 2016

Labels: Needs-Bisect
i tryed on both retina and non retina screen before posting. Also paths render good just text has wrong dash.

Cc: sureshkumari@chromium.org
Labels: -Needs-Bisect -Type-Bug-Regression M-57 OS-Linux OS-Mac OS-Windows Type-Bug
Status: Untriaged (was: Available)
Able to reproduce the issue on Windows-7, Mac-10.11.6 and linux Ubuntu 14.04 using chrome stable version 54.0.2840.99 and canary 57.0.2926.0 with the steps mentioned above.
This is non-regression issue,observed from earlier version of chrome M30# 30.0.1550.0 .Hence marking it as Untriaged to get it addressed.
Please find the attached screenshots for reference.

Thanks,



Chrome-667453.PNG
54.0 KB View Download
Firefox-667453.PNG
47.6 KB View Download

Comment 5 by f...@opera.com, Nov 22 2016

Owner: f...@opera.com
Status: Assigned (was: Untriaged)
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 23 2016

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

commit faa703d5f2fb405a577ee2debc0c04be68aceb79
Author: fs <fs@opera.com>
Date: Wed Nov 23 08:47:57 2016

Apply the font scale factor when generating stroke geometry for <text>

Because of the special font scale factor applied to <svg:text> to bring
it into a pseudo "host" transform, the stroke geometry would end up
being generated in/relative to the wrong coordinate space.
Apply the same scale to dash-related properties as was previously
applied to stroke-width.

BUG= 667453 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2513343005
Cr-Commit-Position: refs/heads/master@{#434135}

[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/dasharray-dashoffset-text-expected.html
[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/dasharray-dashoffset-text.html
[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/dasharray-text-expected.html
[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/dasharray-text.html
[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/non-scaling-stroke-text-decoration-dashed-expected.html
[add] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/LayoutTests/svg/stroke/non-scaling-stroke-text-decoration-dashed.html
[modify] https://crrev.com/faa703d5f2fb405a577ee2debc0c04be68aceb79/third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainter.cpp

Comment 7 by f...@opera.com, Nov 23 2016

Status: Fixed (was: Assigned)

Sign in to add a comment