125597 2013-12-11 12:03:29 -0800 mo text does not appear in dumpAsText output 2016-06-24 23:59:55 -0700 1 1 1 Unclassified WebKit MathML 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 124838 155018 139436 130245 1 fred.wang webkit-unassigned alex bfulgham cfleizach darin mrobinson oldest_to_newest 958851 0 fred.wang 2013-12-11 12:03:29 -0800 Try to call DumpRenderTree on the attached testcase. You'll get the "mtext" text but not the "mo" text. 958864 1 fred.wang 2013-12-11 12:48:12 -0800 I'm moving this to Tools/Tests as I think that's something that could be fixed in rendering/RenderTreeAsText.cpp or similar. Any suggestion for how to do that would be welcome. 958865 2 218996 fred.wang 2013-12-11 12:50:27 -0800 Created attachment 218996 testcase 959462 3 darin 2013-12-12 21:11:04 -0800 RenderTreeAsText.cpp is only used when you do *not* call dumpAsText. The testRunner.dumpAsText feature is implemented with the DOM innerText function, so there is not necessarily a way to fix this without changing the behavior of the DOM. The innerText function, in turn, is based on the text iterator in TextIterator.cpp, which is the same code used for things like converting a selection to plain text on the clipboard when you select and copy something, or searching for text with the browser’s find command. So this is likely a bug in the MathML implementation and not a tools issue, assuming that we want text from inside an <mo> to be copyable and searchable, etc. The text iterator primarily iterates the DOM and not the render tree. It does not have any code to dig inside of the kind of render tree that RenderMathMLOperator::updateFromElement builds. We could add code for that, or we could simply add special case code to TextIterator::handleNonTextNode to handle RenderMathMLOperator specially. 959524 4 fred.wang 2013-12-13 02:09:49 -0800 Thank you Darin. Indeed, the patch for bug 124838 seems to break text selection of token elements. So I'll have to check what text iterator is doing and how it could handle MathML text better. 959559 5 fred.wang 2013-12-13 08:45:27 -0800 Changing the direction of the dependency since I'll probably fix the issue while doing the refactoring code of MathML token elements. Some test references are currently incorrect with the <mo> text not appearing in the expected output. 959568 6 darin 2013-12-13 09:30:12 -0800 (In reply to comment #4) > Indeed, the patch for bug 124838 seems to break text selection of token elements. That is an even-more-fundamental issue with the same underlying cause: A render tree that is not linked to a DOM tree in the usual way. Text selection is done by other editing code and is not handled by the text iterator, so text iterator changes will not fix that. We may need to use a shadow DOM approach as elements like the slider do rather than just creating a set of renderers. Not really my area of expertise. 959570 7 fred.wang 2013-12-13 09:41:16 -0800 (In reply to comment #6) > (In reply to comment #4) > > Indeed, the patch for bug 124838 seems to break text selection of token elements. > > That is an even-more-fundamental issue with the same underlying cause: A render tree that is not linked to a DOM tree in the usual way. Text selection is done by other editing code and is not handled by the text iterator, so text iterator changes will not fix that. > > We may need to use a shadow DOM approach as elements like the slider do rather than just creating a set of renderers. Not really my area of expertise. Yes, I've been able to get something for the text output but not the selection. If that's too complicated, I'll first try to do the refactoring that is necessary for 99620 on the mo element only and come back to this later. 1162383 8 fred.wang 2016-02-05 06:09:19 -0800 On Alex's MathMLLayout branch, DumpRenderTree on the testcase now generates the expected "mo" test. Actually, the MathML layout code no longer uses any anonymous nodes for RenderMathMLOperator / RenderMathMLToken, instead we just different measuring / painting functions when necessary. 1205322 9 fred.wang 2016-06-24 23:59:55 -0700 Fixed after r202420 218996 2013-12-11 12:50:27 -0800 2013-12-11 12:50:27 -0800 testcase mo-DumpAsText.html text/html 185 fred.wang PGh0bWw+CjxoZWFkPgo8c2NyaXB0PgogICAgaWYgKHdpbmRvdy50ZXN0UnVubmVyKQogICAgICAg IHRlc3RSdW5uZXIuZHVtcEFzVGV4dCgpOwo8L3NjcmlwdD4KPC9oZWFkPgo8Ym9keT4KICA8bWF0 aD4KICAgIDxtdGV4dD5tdGV4dDwvbXRleHQ+IAogICAgPG1vPm1vPC9tbz4KICA8L21hdGg+Cjwv Ym9keT4KPC9odG1sPgo=