Reviewed by Maciej.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Oct 2003 06:26:49 +0000 (06:26 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Oct 2003 06:26:49 +0000 (06:26 +0000)
        - fixed 3457831 -- REGRESSION: copying particular text and pasting (plain) produces lots of extra text

        * khtml/khtml_part.cpp: (KHTMLPart::text): Check for the end node when following a
        link to the parent node, not just when following a link to the next child node.
        * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::attributedString): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@5268 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/khtml_part.cpp
WebCore/kwq/KWQKHTMLPart.mm

index bacc0f2..0430ec6 100644 (file)
@@ -1 +1,11 @@
+2003-10-26  Darin Adler  <darin@apple.com>
+
+        Reviewed by Maciej.
+
+        - fixed 3457831 -- REGRESSION: copying particular text and pasting (plain) produces lots of extra text
+
+        * khtml/khtml_part.cpp: (KHTMLPart::text): Check for the end node when following a
+        link to the parent node, not just when following a link to the next child node.
+        * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::attributedString): Ditto.
+
 == Rolled over to ChangeLog-2003-10-25 ==
index c37b61c..9d69cf5 100644 (file)
@@ -2366,6 +2366,7 @@ QString KHTMLPart::text(const DOM::Range &r) const
       if(next.isNull()) next = n.nextSibling();
       while( next.isNull() && !n.parentNode().isNull() ) {
         n = n.parentNode();
+        if(n == endNode) break;
         next = n.nextSibling();
         unsigned short id = n.elementId();
         switch(id) {
index 75fbc57..84b4b2e 100644 (file)
@@ -2406,6 +2406,8 @@ NSAttributedString *KWQKHTMLPart::attributedString(NodeImpl *_start, int startOf
         while (next.isNull() && !n.parentNode().isNull()) {
             QString text;
             n = n.parentNode();
+            if (n == endNode)
+                break;
             next = n.nextSibling();
 
             unsigned short _id = n.elementId();