Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1987)

Unified Diff: chrome/browser/extensions/api/downloads/downloads_api_unittest.cc

Issue 10913015: Reland DownloadManager::GetAllDownloads actually does now (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
diff --git a/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc b/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
index dd38572ca07b62e8ef8e983c5b414065ab343d7d..bad27f0cae09729c40c32bd2341b82785201149d 100644
--- a/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
+++ b/chrome/browser/extensions/api/downloads/downloads_api_unittest.cc
@@ -369,7 +369,7 @@ class DownloadExtensionTest : public ExtensionApiTest {
entries.push_back(entry);
}
GetOnRecordManager()->OnPersistentStoreQueryComplete(&entries);
- GetOnRecordManager()->GetAllDownloads(FilePath(), items);
+ GetOnRecordManager()->GetAllDownloads(items);
EXPECT_EQ(count, items->size());
if (count != items->size())
return false;
@@ -401,7 +401,7 @@ class DownloadExtensionTest : public ExtensionApiTest {
EXPECT_EQ(
1u, observer->NumDownloadsSeenInState(DownloadItem::IN_PROGRESS));
}
- GetCurrentManager()->GetAllDownloads(FilePath(), items);
+ GetCurrentManager()->GetAllDownloads(items);
ASSERT_EQ(count, items->size());
}
@@ -423,7 +423,7 @@ class DownloadExtensionTest : public ExtensionApiTest {
EXPECT_EQ(1u, observer->NumDownloadsSeenInState(DownloadItem::IN_PROGRESS));
DownloadManager::DownloadVector items;
- manager->GetAllDownloads(FilePath(), &items);
+ manager->GetAllDownloads(&items);
DownloadItem* new_item = NULL;
for (DownloadManager::DownloadVector::iterator iter = items.begin();
« no previous file with comments | « chrome/browser/extensions/api/downloads/downloads_api.cc ('k') | content/browser/download/download_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698