Bug 389876 followup (r+a=mconnor)

toolkit/components/places/src/nsNavHistory.cpp

@@ -970,6 +970,7 @@ nsNavHistory::CleanUpOnQuit()
   nsresult rv = mDBConn->CreateStatement(NS_LITERAL_CSTRING(
     "SELECT user_title FROM moz_places"), getter_AddRefs(statement2));
   if (NS_SUCCEEDED(rv)) {
+    mozStorageTransaction transaction(mDBConn, PR_FALSE);
     // 1. Indexes are moved along with the renamed table. Since we're dropping
     // that table, we're also dropping it's indexes, and later re-creating them
     // for the new table.
@@ -1036,6 +1037,7 @@ nsNavHistory::CleanUpOnQuit()
     rv = mDBConn->ExecuteSimpleSQL(NS_LITERAL_CSTRING(
       "DROP TABLE moz_places_backup"));
     NS_ENSURE_SUCCESS(rv, rv);
+    transaction.Commit();
   }
   return NS_OK;
 }