Reviewed by John
[WebKit-https.git] / WebCore / ChangeLog-2005-08-23
index 255f04a3785d92147c5f889e5cc11cae453ba9af..1927fbb5c8f25d79db26adcc45f3ab7b73b1b952 100644 (file)
@@ -1,3 +1,19 @@
+2004-10-14  Ken Kocienda  <kocienda@apple.com>
+
+        Reviewed by John
+
+        Fix for this bug:
+        
+        <rdar://problem/3839989> REGRESSION (Mail): Left arrow does nothing after inserting attachment
+
+        * khtml/editing/visible_position.cpp:
+        (khtml::VisiblePosition::deepEquivalent): Remove code that attempted to bridge old-style
+        position code to new-style VisiblePosition code. In retrospect, this code was misguided.
+        Since we do a good job of insulating external code from the internal workings of 
+        VisiblePosition, the "hop ahead" being done here was not doing anyone any real good, and
+        in the case of this bug, was doing harm. Simply removing this code makes the bug
+        go away and does not cause any editing layout test regresssions.
+
 2004-10-14  Ken Kocienda  <kocienda@apple.com>
 
         Reviewed by Hyatt