See text-tselect-01-b.svg and text-tselect-02-f.svg for tests.
Created attachment 12555 [details] First attempt This patch should do it. Testcases may need to wait until we can select SVG text using the DOM. Cheers, Rob.
Comment on attachment 12555 [details] First attempt This is wrong: strncmp(r->renderName(), "RenderSVGInlineText", 19) == 0 || strncmp(r->renderName(), "RenderSVGText", 13) == 0 we should add a virtual method isSVGTextObject() (or similar) to RenderObject instead (if RenderSVGInlineTExt is a subclass of RenderSVGText it coudl easily be named isSVGText()) maybe there is an easier more general way to do this? It seems a little awkward to hack this into the base code.
Created attachment 12563 [details] Improved patch Addressing Eric's issue. Cheers, Rob.
Created attachment 12575 [details] Now with testcase This one contains some extra safety checks and cleanups. Also added a manual test. Cheers, Rob.
Comment on attachment 12575 [details] Now with testcase r=me
Landed in r19002.