Reviewed by Darin
[WebKit-https.git] / WebCore / ChangeLog-2005-08-23
index 5d448beacd719a239f3b53eb84f6f1b25d6cdfa2..40f2bc4bf772fbfed8f019ea3187ae1abedd0750 100644 (file)
@@ -1,3 +1,19 @@
+2004-10-05  Ken Kocienda  <kocienda@apple.com>
+
+        Reviewed by Darin
+
+        Recent checkin adding upstreamDeepEquivalent had it backwards. The helper
+        we want is downstreamDeepEquivalent, as the deepEquivalent function returns
+        an upstream position.
+
+        * khtml/editing/selection.cpp:
+        (khtml::Selection::layout): DOWNSTREAM case now uses downstreamDeepEquivalent.
+        UPSTREAM uses deepEquivalent.
+        * khtml/editing/visible_position.cpp:
+        (khtml::VisiblePosition::downstreamDeepEquivalent): New helper, replacing 
+        upstreamDeepEquivalent.
+        * khtml/editing/visible_position.h
+
 2004-10-05  David Hyatt  <hyatt@apple.com>
 
        New selection gap-filling architecture.  Makes the gap-filling much more like NSTextView and puts the responsibility