[webkit-reviews] review requested: [Bug 19539] Nested <ul>s are mishandled when converted to <ol> using execCommand('insertorderedlist') : [Attachment 51201] fixes InsertListCommand::doApply to merge lists after inserting a list element

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Mar 19 16:18:00 PDT 2010


Ryosuke Niwa <rniwa at webkit.org> has asked  for review:
Bug 19539: Nested <ul>s are mishandled when converted to <ol> using
execCommand('insertorderedlist')
https://bugs.webkit.org/show_bug.cgi?id=19539

Attachment 51201: fixes InsertListCommand::doApply to merge lists after
inserting a list element
https://bugs.webkit.org/attachment.cgi?id=51201&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
The bug was caused by the second part of InsertListCommand::doApply where it
does not merge lists after inserting a list element.  While I'm quite convinced
that we need to cleanup InsertListCommand.cpp, this bug seems to be simple
enough that we can fix it now.


More information about the webkit-reviews mailing list