Revert 98656 - Make a new integer field in sql::MetaTable (a per-profile db) containing a counter...

content/browser/download/download_file_manager.cc

Index: content/browser/download/download_file_manager.cc
===================================================================
--- content/browser/download/download_file_manager.cc	(revision 98659)
+++ content/browser/download/download_file_manager.cc	(working copy)
@@ -65,9 +65,9 @@
     return;
   }
 
-  DownloadId global_id(download_manager, info->download_id);
-  DCHECK(GetDownloadFile(global_id) == NULL);
-  downloads_[global_id] = download_file.release();
+  int32 id = info->download_id;
+  DCHECK(GetDownloadFile(id) == NULL);
+  downloads_[id] = download_file.release();
 
   // The file is now ready, we can un-pause the request and start saving data.
   info->request_handle.ResumeRequest();
@@ -77,11 +77,11 @@
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       NewRunnableMethod(download_manager,
-                        &DownloadManager::StartDownload, info->download_id));
+                        &DownloadManager::StartDownload, id));
 }
 
-DownloadFile* DownloadFileManager::GetDownloadFile(DownloadId global_id) {
-  DownloadFileMap::iterator it = downloads_.find(global_id);
+DownloadFile* DownloadFileManager::GetDownloadFile(int id) {
+  DownloadFileMap::iterator it = downloads_.find(id);
   return it == downloads_.end() ? NULL : it->second;
 }
 
@@ -102,17 +102,21 @@
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   for (DownloadFileMap::iterator i = downloads_.begin();
        i != downloads_.end(); ++i) {
-    DownloadId global_id = i->first;
+    int id = i->first;
     DownloadFile* download_file = i->second;
     DownloadManager* manager = download_file->GetDownloadManager();
     if (manager) {
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
           NewRunnableMethod(manager, &DownloadManager::UpdateDownload,
-                            global_id.local(), download_file->bytes_so_far()));
+                            id, download_file->bytes_so_far()));
     }
   }
 }
 
+int DownloadFileManager::GetNextId() {
+  return next_id_.GetNext();
+}
+
 void DownloadFileManager::StartDownload(DownloadCreateInfo* info) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(info);
@@ -140,8 +144,7 @@
 // download (in the UI thread), we may receive a few more updates before the IO
 // thread gets the cancel message: we just delete the data since the
 // DownloadFile has been deleted.
-void DownloadFileManager::UpdateDownload(
-    DownloadId global_id, DownloadBuffer* buffer) {
+void DownloadFileManager::UpdateDownload(int id, DownloadBuffer* buffer) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   std::vector<DownloadBuffer::Contents> contents;
   {
@@ -149,7 +152,7 @@
     contents.swap(buffer->contents);
   }
 
-  DownloadFile* download_file = GetDownloadFile(global_id);
+  DownloadFile* download_file = GetDownloadFile(id);
   for (size_t i = 0; i < contents.size(); ++i) {
     net::IOBuffer* data = contents[i].first;
     const int data_len = contents[i].second;
@@ -160,16 +163,16 @@
 }
 
 void DownloadFileManager::OnResponseCompleted(
-    DownloadId global_id,
+    int id,
     DownloadBuffer* buffer,
     int os_error,
     const std::string& security_info) {
-  VLOG(20) << __FUNCTION__ << "()" << " id = " << global_id
+  VLOG(20) << __FUNCTION__ << "()" << " id = " << id
            << " os_error = " << os_error
            << " security_info = \"" << security_info << "\"";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   delete buffer;
-  DownloadFile* download_file = GetDownloadFile(global_id);
+  DownloadFile* download_file = GetDownloadFile(id);
   if (!download_file)
     return;
 
@@ -177,7 +180,7 @@
 
   DownloadManager* download_manager = download_file->GetDownloadManager();
   if (!download_manager) {
-    CancelDownload(global_id);
+    CancelDownload(id);
     return;
   }
 
@@ -189,7 +192,7 @@
       BrowserThread::UI, FROM_HERE,
       NewRunnableMethod(
         download_manager, &DownloadManager::OnResponseCompleted,
-        global_id.local(), download_file->bytes_so_far(), os_error, hash));
+        id, download_file->bytes_so_far(), os_error, hash));
   // We need to keep the download around until the UI thread has finalized
   // the name.
 }
@@ -197,10 +200,10 @@
 // This method will be sent via a user action, or shutdown on the UI thread, and
 // run on the download thread. Since this message has been sent from the UI
 // thread, the download may have already completed and won't exist in our map.
-void DownloadFileManager::CancelDownload(DownloadId global_id) {
-  VLOG(20) << __FUNCTION__ << "()" << " id = " << global_id;
+void DownloadFileManager::CancelDownload(int id) {
+  VLOG(20) << __FUNCTION__ << "()" << " id = " << id;
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  DownloadFileMap::iterator it = downloads_.find(global_id);
+  DownloadFileMap::iterator it = downloads_.find(id);
   if (it == downloads_.end())
     return;
 
@@ -209,24 +212,24 @@
            << " download_file = " << download_file->DebugString();
   download_file->Cancel();
 
-  EraseDownload(global_id);
+  EraseDownload(id);
 }
 
-void DownloadFileManager::CompleteDownload(DownloadId global_id) {
+void DownloadFileManager::CompleteDownload(int id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  if (!ContainsKey(downloads_, global_id))
+  if (!ContainsKey(downloads_, id))
     return;
 
-  DownloadFile* download_file = downloads_[global_id];
+  DownloadFile* download_file = downloads_[id];
 
   VLOG(20) << " " << __FUNCTION__ << "()"
-           << " id = " << global_id
+           << " id = " << id
            << " download_file = " << download_file->DebugString();
 
   download_file->Detach();
 
-  EraseDownload(global_id);
+  EraseDownload(id);
 }
 
 void DownloadFileManager::OnDownloadManagerShutdown(DownloadManager* manager) {
@@ -246,7 +249,7 @@
 
   for (std::set<DownloadFile*>::iterator i = to_remove.begin();
        i != to_remove.end(); ++i) {
-    downloads_.erase(DownloadId((*i)->GetDownloadManager(), (*i)->id()));
+    downloads_.erase((*i)->id());
     delete *i;
   }
 }
@@ -260,12 +263,12 @@
 // 1. tmp -> foo.crdownload (not final, safe)
 // 2. tmp-> Unconfirmed.xxx.crdownload (not final, dangerous)
 void DownloadFileManager::RenameInProgressDownloadFile(
-    DownloadId global_id, const FilePath& full_path) {
-  VLOG(20) << __FUNCTION__ << "()" << " id = " << global_id
+    int id, const FilePath& full_path) {
+  VLOG(20) << __FUNCTION__ << "()" << " id = " << id
            << " full_path = \"" << full_path.value() << "\"";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  DownloadFile* download_file = GetDownloadFile(global_id);
+  DownloadFile* download_file = GetDownloadFile(id);
   if (!download_file)
     return;
 
@@ -275,7 +278,7 @@
   if (!download_file->Rename(full_path)) {
     // Error. Between the time the UI thread generated 'full_path' to the time
     // this code runs, something happened that prevents us from renaming.
-    CancelDownloadOnRename(global_id);
+    CancelDownloadOnRename(id);
   }
 }
 
@@ -287,15 +290,13 @@
 // 1. foo.crdownload -> foo (final, safe)
 // 2. Unconfirmed.xxx.crdownload -> xxx (final, validated)
 void DownloadFileManager::RenameCompletingDownloadFile(
-    DownloadId global_id,
-    const FilePath& full_path,
-    bool overwrite_existing_file) {
-  VLOG(20) << __FUNCTION__ << "()" << " id = " << global_id
+    int id, const FilePath& full_path, bool overwrite_existing_file) {
+  VLOG(20) << __FUNCTION__ << "()" << " id = " << id
            << " overwrite_existing_file = " << overwrite_existing_file
            << " full_path = \"" << full_path.value() << "\"";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  DownloadFile* download_file = GetDownloadFile(global_id);
+  DownloadFile* download_file = GetDownloadFile(id);
   if (!download_file)
     return;
 
@@ -325,7 +326,7 @@
   if (!download_file->Rename(new_path)) {
     // Error. Between the time the UI thread generated 'full_path' to the time
     // this code runs, something happened that prevents us from renaming.
-    CancelDownloadOnRename(global_id);
+    CancelDownloadOnRename(id);
     return;
   }
 
@@ -335,17 +336,19 @@
   download_file->AnnotateWithSourceInformation();
 #endif
 
-  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, NewRunnableMethod(
-      download_manager, &DownloadManager::OnDownloadRenamedToFinalName,
-      global_id.local(), new_path, uniquifier));
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      NewRunnableMethod(
+          download_manager, &DownloadManager::OnDownloadRenamedToFinalName, id,
+          new_path, uniquifier));
 }
 
 // Called only from RenameInProgressDownloadFile and
 // RenameCompletingDownloadFile on the FILE thread.
-void DownloadFileManager::CancelDownloadOnRename(DownloadId global_id) {
+void DownloadFileManager::CancelDownloadOnRename(int id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  DownloadFile* download_file = GetDownloadFile(global_id);
+  DownloadFile* download_file = GetDownloadFile(id);
   if (!download_file)
     return;
 
@@ -361,22 +364,22 @@
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       NewRunnableMethod(download_manager,
-                        &DownloadManager::CancelDownload, global_id.local()));
+                        &DownloadManager::CancelDownload, id));
 }
 
-void DownloadFileManager::EraseDownload(DownloadId global_id) {
+void DownloadFileManager::EraseDownload(int id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  if (!ContainsKey(downloads_, global_id))
+  if (!ContainsKey(downloads_, id))
     return;
 
-  DownloadFile* download_file = downloads_[global_id];
+  DownloadFile* download_file = downloads_[id];
 
   VLOG(20) << " " << __FUNCTION__ << "()"
-           << " id = " << global_id
+           << " id = " << id
            << " download_file = " << download_file->DebugString();
 
-  downloads_.erase(global_id);
+  downloads_.erase(id);
 
   delete download_file;