Summary: | spelling underlines disappear when they shouldn't | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||
Component: | New Bugs | Assignee: | Hironori Bono <hbono> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | justin.garcia | ||||
Priority: | P2 | ||||||
Version: | 525.x (Safari 3.1) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Eric Seidel (no email)
2008-05-01 14:08:03 PDT
Created attachment 26522 [details]
A Quick Fix
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.
Comment on attachment 26522 [details]
A Quick Fix
so, deleteTextFromNode removes spelling markers from text even if the deletion doesn't effect them? we should document that problem.
do we need to make this change to InsertParagraphSeparatorCommand and BreakBlockquoteCommand?
r=me
> so, deleteTextFromNode removes spelling markers from text even if the deletion
> doesn't effect them? we should document that problem.
Oops my mistake, I misread the code, I thought it deleted what was after the caret, not what was before. Disregard that bit.
Thank you for your review and comments.
> do we need to make this change to InsertParagraphSeparatorCommand and
> BreakBlockquoteCommand?
As far as I read the InsertParagraphSeparatorCommand::doApply() and BreakBlockquoteCommand::doApply() functions, they already use the splitTextNode() function to split a text node. (This is a reason why my quick fix uses the splitTextNode() function.)
Regards,
Hironori Bono
Added missing layout test results: http://trac.webkit.org/changeset/40341 |