729996 - SVG: space being incorrectly inferred in <text> content

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Michael O'Rourke]

Attachment: text.svg

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/535.11 (KHTML, like Gecko) Chrome/17.0.963.56 Safari/535.11

Steps to reproduce:


Open the attached SVG file containing a line of text and an outline of what the line of text should look like.


Actual results:


The first line is incorrectly shifted to the left


Expected results:


The two lines should look identical.

When there is a line break between the last </tspan> and the </text> tag, IE is incorrectly inferring an implied space.

Per the SVG spec (section 10.15), you should "strip out all trailing space characters (i.e., strip out space characters before "</text>")".

Chrome handles this correctly.

Comment 1

[Robert Longson [:longsonr]]

Can you attach a screenshot. I don't think I have the fonts you have installed so I'm not seeing what you are? Perhaps you could also attach a screenshot from Chrome showing what it's supposed to look like.

You are seeing this on Firefox 10 aren't you?

Comment 2

[Robert Longson [:longsonr]]

Attachment: patch

need to work on a reftest

Comment 3

[Robert Longson [:longsonr]]

(In reply to Robert Longson from comment #1)
> Can you attach a screenshot. I don't think I have the fonts you have
> installed so I'm not seeing what you are? Perhaps you could also attach a
> screenshot from Chrome showing what it's supposed to look like.
> 
> You are seeing this on Firefox 10 aren't you?

Actually never mind, I worked it out.

Comment 4

[Robert Longson [:longsonr]]

Attachment: patch with test

Comment 5

[Daniel Holbert [:dholbert]]

Last good nightly: 2011-05-04
First bad nightly: 2011-05-05

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c3c4c902e9cd&tochange=31879b88cc82

Looks like a regression from bug 620286.

Comment 6

[Daniel Holbert [:dholbert]]

Attachment: testcase (no red should be visible)

(here's a clearer standalone testcase, based on longsonr's reftest, just to make this bug easy to see / test)

Comment 7

[Robert Longson [:longsonr]]

pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/c65342fe6704

Thank you for the testcase Michael.

Comment 8

[Marco Bonardo [:mak]]

https://hg.mozilla.org/mozilla-central/rev/c65342fe6704