Created attachment 4643 [details] remove virtual containsOnlyWhitespace from DOM::NodeImpl
Comment on attachment 4643 [details] remove virtual containsOnlyWhitespace from DOM::NodeImpl This part of your patch is not safe: + !curr->element() || !static_cast<TextImpl *>(curr->element())- It may be possible for things other than TextNodes to create RenderText objects (at least in an SVG-enabled world).
Comment on attachment 4643 [details] remove virtual containsOnlyWhitespace from DOM::NodeImpl Good point. I thought there was a check of isTextNode on that line. I'll fix it.
Created attachment 5145 [details] adds call to isTextNode missing from earlier patch
Comment on attachment 5145 [details] adds call to isTextNode missing from earlier patch r=me
Mass moving XML DOM bugs to the "DOM" Component.