360869 - text-anchor='middle' does not work with text content added via certain scripts

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Doug Schepers]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0

text-anchor='middle' doesn't center text that is added to an element if it is the result of mouse events (rather than load events, for example, oddly enough).  Text that is in place when the document is parsed works fine, as does text added via the same script called onload instead.

Reproducible: Always

Steps to Reproduce:
1. create empty static text element
2. add text via script evoked by click 
3. text is not centered

Actual Results:  
Text is not centered

Expected Results:  
Text should be centered

Comment 1

[Doug Schepers]

Attachment: illustrates how text isn't centered... click on red dot to activate

Comment 2

[Robert Longson [:longsonr]]

WFM with latest firefox trunk build. Broken in firefox 2.0 though.

Comment 3

[tor]

Attachment: branch patch - missing chunk from bug 307155

Comment 4

[tor]

Comment on attachment 246072 [details] [diff] [review]
branch patch - missing chunk from bug 307155

While if writing new code I'd use IsEmpty() and shuffle the code in the method a bit, I figured it was best to keep this as close to the shipping ff1.5 as possible.

Comment 5

[tor]

Comment on attachment 246072 [details] [diff] [review]
branch patch - missing chunk from bug 307155

Important fix for a regression between Firefox 1.5 and 2.0.  This same fix has been on the 1.8.0 branch for over a year.

Comment 6

[Jay Patel [:jay]]

Comment on attachment 246072 [details] [diff] [review]
branch patch - missing chunk from bug 307155

Approved for 1.8.1 branch, a=jay for drivers.  Please land this asap, thanks!

Comment 7

[tor]

Checked in on MOZILLA_1_8_BRANCH.

Comment 8

[tor]

*** Bug 362659 has been marked as a duplicate of this bug. ***