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

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

Issue 10914158: Revert 155351 - Make DownloadManager::GetAllDownloads return all downloads regardless of state, per… (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
===================================================================
--- chrome/browser/extensions/api/downloads/downloads_api_unittest.cc (revision 155501)
+++ chrome/browser/extensions/api/downloads/downloads_api_unittest.cc (working copy)
@@ -369,7 +369,7 @@
entries.push_back(entry);
}
GetOnRecordManager()->OnPersistentStoreQueryComplete(&entries);
- GetOnRecordManager()->GetAllDownloads(items);
+ GetOnRecordManager()->GetAllDownloads(FilePath(), items);
EXPECT_EQ(count, items->size());
if (count != items->size())
return false;
@@ -401,7 +401,7 @@
EXPECT_EQ(
1u, observer->NumDownloadsSeenInState(DownloadItem::IN_PROGRESS));
}
- GetCurrentManager()->GetAllDownloads(items);
+ GetCurrentManager()->GetAllDownloads(FilePath(), items);
ASSERT_EQ(count, items->size());
}
@@ -423,7 +423,7 @@
EXPECT_EQ(1u, observer->NumDownloadsSeenInState(DownloadItem::IN_PROGRESS));
DownloadManager::DownloadVector items;
- manager->GetAllDownloads(&items);
+ manager->GetAllDownloads(FilePath(), &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