Implement chrome.experimental.downloads.search()

content/browser/download/download_manager.cc

Index: content/browser/download/download_manager.cc
diff --git a/content/browser/download/download_manager.cc b/content/browser/download/download_manager.cc
index eb91ef0f0ba093b732af52635edad42c6f934e23..16a46318c3d7dd5ded311ba3d0fd7034b96ad090 100644
--- a/content/browser/download/download_manager.cc
+++ b/content/browser/download/download_manager.cc
@@ -25,6 +25,7 @@
 #include "content/browser/download/download_item.h"
 #include "content/browser/download/download_manager_delegate.h"
 #include "content/browser/download/download_persistent_store_info.h"
+#include "content/browser/download/download_query.h"
 #include "content/browser/download/download_stats.h"
 #include "content/browser/download/download_status_updater.h"
 #include "content/browser/download/interrupt_reasons.h"
@@ -174,6 +175,11 @@ void DownloadManager::GetAllDownloads(
   }
 }
 
+void DownloadManager::Search(const download_util::DownloadQuery& query,
+                             DownloadVector* results) const {
+  query.Search(downloads_.begin(), downloads_.end(), results);
+}
+
 void DownloadManager::SearchDownloads(const string16& query,
                                       DownloadVector* result) {
   string16 query_lower(base::i18n::ToLower(query));
@@ -827,7 +833,7 @@ void DownloadManager::OnPersistentStoreQueryComplete(
   largest_db_handle_in_history_ = 0;
 
   for (size_t i = 0; i < entries->size(); ++i) {
-    DownloadItem* download = new DownloadItem(this, entries->at(i));
+    DownloadItem* download = new DownloadItem(this, entries->at(i), -1 - i);

[cbentzel, 2011/10/20 18:18:53]

Why -1 - i? 

     // TODO(rdsmith): Remove after http://crbug.com/85408 resolved.
     CHECK(!ContainsKey(history_downloads_, download->db_handle()));
     downloads_.insert(download);