956513 - SVG text is drawn with wrong position

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Alice0775 White]

Attachment: screenshot

When I tested bug 812795, I noticed a problem.

Steps To Reproduce:
1. Open attachment 682767 [details] of Bug 812795

Actual Results:
"TEXT ON PATH" is shifted to the right and truncated.

Expected Results:
"TEXT ON PATH" is drawn at the center

Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/cbb24a4a96af
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130629 Firefox/25.0 ID:20130629065543
Bad:
http://hg.mozilla.org/mozilla-central/rev/614e1793e958
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130629 Firefox/25.0 ID:20130629173041
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=cbb24a4a96af&tochange=614e1793e958


Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/5e226d928613
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130629 Firefox/25.0 ID:20130629052942
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/3a23afb038a5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130629 Firefox/25.0 ID:20130629060904
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=5e226d928613&tochange=3a23afb038a5

Suspected;
	3a23afb038a5	Cameron McCormack — Bug 839955 - Enable new SVG text frames. r=roc

Comment 1

[Cameron McCormack (:heycam)]

I wonder what is making this document not render at all in Chrome and Safari.  Renders as expected in IE10.

Comment 2

[Cameron McCormack (:heycam)]

Attachment: reduced test case

Here's a smaller test case that does render in Chrome/Safari.

If I remove the xml:space="preserve", move it down on to the <textPath>, or remove the white space characters that precede the <textPath>, then it renders correctly.

Comment 3

[Cameron McCormack (:heycam)]

Attachment: patch

Comment 4

[Cameron McCormack (:heycam)]

https://tbpl.mozilla.org/?tree=Try&rev=e3281d42081d

Comment 5

[Robert Longson [:longsonr]]

Comment on attachment 8355836 [details] [diff] [review]
patch

Please add the Public Domain header to the tests unless there's some reason not to.

Comment 6

[Cameron McCormack (:heycam)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/362eeec1dc96

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/362eeec1dc96