<text> elements ignore <a> children
See attached test case.
Created attachment 12912 [details]
Firefox fails this test as well. Opera passes.
This also causes this test to fail:
Created attachment 12933 [details]
This "fixes" the bug, but crashes, due to RenderSVGInlineText not expecting an element other than a TextPositioningElement. I'm not sure if RenderSVGInline is the correct renderer for <a> to be making or not. We'd need to hear olliej's opinion on this.
This also cleans up renderer creation, to prevent text renderers from being allowed in the wrong places.
This blocks at least 3-4 w3c tests, and is likely seen often "in the wild". Adding to the SVG hit list.
Created attachment 13567 [details]
Based on Eric's patch, this also fixes the SVGInlineFlowBox routines and starts supporting xlink:show.
Created attachment 13575 [details]
I got the target attribute down in my previous patch. I checked with Opera and it seems if either target="_self" or xlink:show="new" the link opens up in a new window, so I implemented it like this in this patch.
Comment on attachment 13575 [details]
You mentioned that we now match Opera's behaviour - I guess FF/Batik handle it the same way?
I do think the way you implemented it and how Opera handles it makes sense though, so r=me in any way :-)
Landed in r20122.