From bcbb9f351215a9f03f561997389a0bd3bd270a86 Mon Sep 17 00:00:00 2001
From: "mjudge%netscape.com" <mjudge%netscape.com>
Date: Thu, 2 Sep 1999 00:48:15 +0000
Subject: [PATCH] new fix for table selection into and out. works great.

---
 layout/base/src/nsRangeList.cpp | 36 +++++++++++++++++++++++++--------
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/layout/base/src/nsRangeList.cpp b/layout/base/src/nsRangeList.cpp
index 25b75c4742cb..8bf9edb1223b 100644
--- a/layout/base/src/nsRangeList.cpp
+++ b/layout/base/src/nsRangeList.cpp
@@ -2166,17 +2166,37 @@ nsDOMSelection::FixupSelectionPoints(nsIDOMRange *aRange , nsDirection *aDir, PR
       if (!found)
         return NS_ERROR_FAILURE;
     }
-  } 
-  if (FetchAnchorNode() == startNode.get() && FetchFocusNode() == endNode.get() &&
-      FetchAnchorOffset() == startOffset && FetchFocusOffset() == endOffset)
-  {
-    *aFixupState = PR_FALSE;
-    return NS_ERROR_FAILURE;//nothing to do
   }
+  if (*aDir == eDirNext)
+  {
+    if (FetchAnchorNode() == startNode.get() && FetchFocusNode() == endNode.get() &&
+      FetchAnchorOffset() == startOffset && FetchFocusOffset() == endOffset)
+    {
+      *aFixupState = PR_FALSE;
+      return NS_ERROR_FAILURE;//nothing to do
+    }
+  }
+  else
+  {
+    if (FetchAnchorNode() == endNode.get() && FetchFocusNode() == startNode.get() &&
+      FetchAnchorOffset() == endOffset && FetchFocusOffset() == startOffset)
+    {
+      *aFixupState = PR_FALSE;
+      return NS_ERROR_FAILURE;//nothing to do
+    }
+  }
+  if (mFixupState && !dirtyend && !dirtystart)//no mor fixup! all bets off
+  {
+    dirtystart = PR_TRUE;//force a reset of anchor positions
+    dirtystart = PR_TRUE;
+    *aFixupState = PR_TRUE;//redraw all selection here
+    mFixupState = PR_FALSE;//no more fixup for next time
+  }
+  else
   if ((dirtystart || dirtyend) && *aDir != mDirection) //fixup took place but new direction all bets are off
   {
     *aFixupState = PR_TRUE;
-    mFixupState = PR_FALSE;
+    //mFixupState = PR_FALSE;
   }
   else
   if (dirtystart && (FetchAnchorNode() != startNode.get() || FetchAnchorOffset() != startOffset))
@@ -2192,7 +2212,7 @@ nsDOMSelection::FixupSelectionPoints(nsIDOMRange *aRange , nsDirection *aDir, PR
   }
   else
   {
-    mFixupState = PR_FALSE;
+    mFixupState = dirtystart || dirtyend;
     *aFixupState = PR_FALSE;
   }
   if (dirtystart || dirtyend){