LayoutTests:
authorjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Oct 2006 19:27:19 +0000 (19:27 +0000)
committerjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Oct 2006 19:27:19 +0000 (19:27 +0000)
commitf6023a3175bf8fb3359105be099d07078af6f1d1
treea6b5054530a7ddcd22961e33e9bff7609a19a926
parentdc0c47955cff9766134bfd376c0d41dc3e13fa58
LayoutTests:

        Reviewed by harrison

        <rdar://problem/4784823>
        GMail Editor: Hang occurs when removing list styling on text in a rich text message

        * editing/execCommand/remove-list-1-expected.checksum: Added.
        * editing/execCommand/remove-list-1-expected.png: Added.
        * editing/execCommand/remove-list-1-expected.txt: Added.
        * editing/execCommand/remove-list-1.html: Added.

WebCore:

        Reviewed by harrison

        <rdar://problem/4784823>
        GMail Editor: Hang occurs when removing list styling on text in a rich text message

        List removal moves the contents of every list item out of the list it's
        in. When the code tried to move the contents of an empty list item (an li
        with no child nodes, not even a placeholder br), moveParagraph didn't prune
        the li, like it would if the li had a placeholder inside it.  So the list
        removal code went into an infinite loop, continually attempting to de-list
        an empty list item.

        * editing/CompositeEditCommand.cpp:
        (WebCore::CompositeEditCommand::moveParagraphs): If the selection
        to move was empty and in an empty block that doesn't require a
        placeholder to prop itself open (like a bordered div or an li),
        remove it during the move.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@17114 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/editing/execCommand/remove-list-1-expected.checksum [new file with mode: 0644]
LayoutTests/editing/execCommand/remove-list-1-expected.png [new file with mode: 0644]
LayoutTests/editing/execCommand/remove-list-1-expected.txt [new file with mode: 0644]
LayoutTests/editing/execCommand/remove-list-1.html [new file with mode: 0644]
WebCore/ChangeLog
WebCore/editing/CompositeEditCommand.cpp