LayoutTests:
authorjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Jul 2006 18:39:55 +0000 (18:39 +0000)
committerjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Jul 2006 18:39:55 +0000 (18:39 +0000)
commit529c789148781081199346eaf0173b793c0d7f37
tree88c2ebef917ce1eac15202ecc39623562bdaac55
parent39e66ebc68a008e00f73d32680d33ec6fca3451f
LayoutTests:

        Reviewed by john

        <rdar://problem/4641033>
        REGRESSION: Pasting from web pages into Mail (or Blot) often loses most of the content

        * editing/pasteboard/4641033-expected.checksum: Added.
        * editing/pasteboard/4641033-expected.png: Added.
        * editing/pasteboard/4641033-expected.txt: Added.
        * editing/pasteboard/4641033.html: Added.

WebCore:

        Reviewed by john

        <rdar://problem/4641033>
        REGRESSION: Pasting from web pages into Mail (or Blot) often loses most of the content

        * editing/markup.cpp:
        (WebCore::createMarkup): We surround the currently accumulated markup with markup
        for ancestors of the startNode when the pre-order traversal leaves the trees rooted
        at those ancestors.  We assumed that any ancestors of the current node not in the
        ancestorsToClose list were those kind of ancestors.  But we don't add renderer-less
        containers to the ancestorsToClose list.  So, we were incorrectly surrounding the
        currently accumulated markup with markup for render-less containers.  Most of the
        content at apple.com was being put inside an <optgroup> inside a <select> element.
        Then on Paste createContextualFragment would drop it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15556 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/editing/pasteboard/4641033-expected.checksum [new file with mode: 0644]
LayoutTests/editing/pasteboard/4641033-expected.png [new file with mode: 0644]
LayoutTests/editing/pasteboard/4641033-expected.txt [new file with mode: 0644]
LayoutTests/editing/pasteboard/4641033.html [new file with mode: 0644]
WebCore/ChangeLog
WebCore/editing/markup.cpp