Call CutRow _before_ we release our reference to the nsIMdbRow. Fixes crash on shift+delete. Bug 258767, patch by jwalden@mit.edu, r=bryner.

toolkit/components/autocomplete/src/nsAutoCompleteMdbResult.cpp

@@ -215,13 +215,13 @@ nsAutoCompleteMdbResult::RemoveValueAt(PRInt32 aRowIndex, PRBool aRemoveFromDb)
   nsIMdbRow *row = mResults.ObjectAt(aRowIndex);
   NS_ENSURE_TRUE(row, NS_ERROR_INVALID_ARG);
 
-  mResults.RemoveObjectAt(aRowIndex);
-
   if (aRemoveFromDb && mTable && mEnv) {
     mdb_err err = mTable->CutRow(mEnv, row);
     NS_ENSURE_TRUE(!err, NS_ERROR_FAILURE);
   }
 
+  mResults.RemoveObjectAt(aRowIndex);
+
   return NS_OK;
 }