Fixing regression bug 232343. Make nsTextServicesDocument::GetFirstTextNodeInPrevBlock() just return the first text node in the previous block, and not also position the iterator at that node. r=mscott@mozilla.org, sr=bryner@brianryner.com

editor/txtsvc/src/nsTextServicesDocument.cpp

@@ -4272,14 +4272,12 @@ nsTextServicesDocument::GetFirstTextNodeInPrevBlock(nsIContent **aContent)
 
   if (!mIterator->IsDone())
   {
-    content = mIterator->GetCurrentNode();
+    NS_ADDREF(*aContent = mIterator->GetCurrentNode());
   }
 
   // Restore the iterator:
 
-  result = mIterator->PositionAt(content);
-
-  return result;
+  return mIterator->PositionAt(content);
 }
 
 nsresult
@@ -4308,15 +4306,11 @@ nsTextServicesDocument::GetFirstTextNodeInNextBlock(nsIContent **aContent)
 
   if (!mIterator->IsDone())
   {
-    *aContent = mIterator->GetCurrentNode();
-    NS_ADDREF(*aContent);
+    NS_ADDREF(*aContent = mIterator->GetCurrentNode());
   }
 
   // Restore the iterator:
-
-  result = mIterator->PositionAt(content);
-
-  return result;
+  return mIterator->PositionAt(content);
 }
 
 nsresult