Reviewed by Ken.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Jul 2004 21:50:53 +0000 (21:50 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Jul 2004 21:50:53 +0000 (21:50 +0000)
        - fixed <rdar://problem/3705500> REGRESSION (125.8-146): Cmd-E on an
        HTML page puts a trailing space on Find pasteboard

        * khtml/misc/khtml_text_operations.cpp:
        (khtml::TextIterator::handleTextBox):
        In the case where a subrun doesn't extend to the end of the text box,
        the well-commented code was supposed to return from this routine, but
        the wrong variable was being compared. This would send us through another
        pass of the loop, which would end up worrying about collapsed space at the
        end of the current box, even though we only cared about the beginning of the
        current box.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/visible_text.cpp
WebCore/khtml/misc/khtml_text_operations.cpp

index 86920d4e829c509da3f6975d5cbda9c6863dbc84..3b289109aed2589788a31476f340316ab24c4a98 100644 (file)
@@ -1,3 +1,19 @@
+2004-07-13  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Ken.
+        
+        - fixed <rdar://problem/3705500> REGRESSION (125.8-146): Cmd-E on an 
+        HTML page puts a trailing space on Find pasteboard
+
+        * khtml/misc/khtml_text_operations.cpp:
+        (khtml::TextIterator::handleTextBox):
+        In the case where a subrun doesn't extend to the end of the text box,
+        the well-commented code was supposed to return from this routine, but
+        the wrong variable was being compared. This would send us through another
+        pass of the loop, which would end up worrying about collapsed space at the
+        end of the current box, even though we only cared about the beginning of the
+        current box.
+
 2004-07-12  Richard Williamson   <rjw@apple.com>
 
        Fixed 3717982.  Implemented navigator.language!
index ba2688a5a4fd3c9c8091d109a2feb6d2bba109b3..8f233ee6edfb0fd7c0d92ae38e9892d6e1e8d7e5 100644 (file)
@@ -384,7 +384,7 @@ void TextIterator::handleTextBox()
 
             // If we are doing a subrun that doesn't go to the end of the text box,
             // come back again to finish handling this text box; don't advance to the next one.
-            if (m_positionEndOffset < runEnd) {
+            if (m_positionEndOffset < textBoxEnd) {
                 return;
             }
 
index ba2688a5a4fd3c9c8091d109a2feb6d2bba109b3..8f233ee6edfb0fd7c0d92ae38e9892d6e1e8d7e5 100644 (file)
@@ -384,7 +384,7 @@ void TextIterator::handleTextBox()
 
             // If we are doing a subrun that doesn't go to the end of the text box,
             // come back again to finish handling this text box; don't advance to the next one.
-            if (m_positionEndOffset < runEnd) {
+            if (m_positionEndOffset < textBoxEnd) {
                 return;
             }