[webkit-reviews] review requested: [Bug 18835] spelling underlines disappear when they shouldn't : [Attachment 26522] A Quick Fix

[bugzilla-daemon at webkit.org]

Hironori Bono <hbono at chromium.org> has asked  for review:
Bug 18835: spelling underlines disappear when they shouldn't
https://bugs.webkit.org/show_bug.cgi?id=18835

Attachment 26522: A Quick Fix
https://bugs.webkit.org/attachment.cgi?id=26522&action=review

------- Additional Comments from Hironori Bono <hbono at chromium.org>
I accidentally noticed a cause of this bug while investigating another issue.
Somehow the InsertLineBreakCommand::doApply() function forgot copying markers
while splitting a text node with a line-break key even though there is a
CompositeEditCommand::splitTextNode() function, which splits a text node and
copy markers.
I'm not sure it is correct to call the CompositeEditCommand::splitTextNode()
function here, though.