Check dbg_statement.Succeeded() in RemoveDownloadsBetween.

If the crash in CreateDownload moves here, then records are being removed from the table even though their ids are not being removed from returned_ids_. If the crash stays in CreateDownload, then there must be some other place where records are being removed from the table even though their ids are not being removed from returned_ids_.

BUG=103610

Review URL: http://codereview.chromium.org/8528037

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110947 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/history/download_database.cc

@@ -234,6 +234,7 @@ void DownloadDatabase::RemoveDownloadsBetween(base::Time delete_begin,
       int64 id_to_delete = dbg_statement.ColumnInt64(0);
       returned_ids_.erase(id_to_delete);
     }
+    CHECK(dbg_statement.Succeeded());
   }
 
   // This does not use an index. We currently aren't likely to have enough