sql::MetaTable.next_download_id, DownloadManager::GetNextId()

chrome/browser/history/history_backend.cc

Index: chrome/browser/history/history_backend.cc
diff --git a/chrome/browser/history/history_backend.cc b/chrome/browser/history/history_backend.cc
index 0da92cc271b117c70c16d1173e516c31b6de8071..70ee22e47975d193a15feb8e5822e75b693667e0 100644
--- a/chrome/browser/history/history_backend.cc
+++ b/chrome/browser/history/history_backend.cc
@@ -1076,6 +1076,15 @@ void HistoryBackend::GetMostRecentKeywordSearchTerms(
 
 // Downloads -------------------------------------------------------------------
 
+void HistoryBackend::GetNextDownloadId(
+    scoped_refptr<DownloadNextIdRequest> request) {
+  if (request->canceled()) return;
+  if (db_.get()) {
+    request->value = db_->next_download_id();
+  }
+  request->ForwardResult(DownloadNextIdRequest::TupleType(request->value));

[Randy Smith (Not in Mondays), 2011/07/28 21:03:16]

This seems like it'll leave an uninitialized value in the case where db_.get()
is false.  Shouldn't we put some kind of value in?  (You should probably try and
figure out in which cases that can happen so we know what the right way to
handle it is.)

[benjhayden, 2011/08/03 17:44:46]

Done.

For this cl, while DownloadManager says next_id_ += next_id; setting it to 0
turns DownloadManager::OnHistoryGetNextId() into a noop.
I think that setting it to 0 will be ok even for future CLs because, if there's
no database, then there won't be any download items loaded from the database,
and new download items won't be written to the database, because there's no
database. That logic works as long as the db_ isn't initialized after
GetNextDownloadId, which we can ensure in ProfileImpl.

[Randy Smith (Not in Mondays), 2011/08/03 21:10:29]

Yes--no collisions to avoid.  There may be circumstances in which db_ is null
when there is a database; there are comments sprinkled through the code about
"the database being offline".  But I believe in those circumstances we don't
talk to the database (can't initialize from it, can't save to it) so we're ok.  

[benjhayden, 2011/08/04 17:15:00]

Done.

+}
+
 // Get all the download entries from the database.
 void HistoryBackend::QueryDownloads(
     scoped_refptr<DownloadQueryRequest> request) {