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

Unified Diff: chrome/browser/download/chrome_download_manager_delegate_unittest.cc

Issue 10915180: Make DownloadHistory observe manager, items (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: @r159248 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/download/chrome_download_manager_delegate_unittest.cc
diff --git a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
index 66049b227c3e72e6c22b5f02841f0c04b5792aeb..64cf199e87fd766864b31cb3f63f4e3e67685c49 100644
--- a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
+++ b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
@@ -304,6 +304,10 @@ void ChromeDownloadManagerDelegateTest::SetUp() {
CHECK(profile());
delegate_ = new TestChromeDownloadManagerDelegate(profile());
+ EXPECT_CALL(*download_manager_.get(), GetAllDownloads(_))
+ .WillRepeatedly(Return());
+ EXPECT_CALL(*download_manager_.get(), AddObserver(_))
+ .WillRepeatedly(Return());
delegate_->SetDownloadManager(download_manager_.get());
pref_service_ = profile()->GetTestingPrefService();
contents()->SetDelegate(&web_contents_delegate_);
@@ -330,6 +334,12 @@ content::MockDownloadItem*
ChromeDownloadManagerDelegateTest::CreateActiveDownloadItem(int32 id) {
content::MockDownloadItem* item =
new ::testing::NiceMock<content::MockDownloadItem>();
+ EXPECT_CALL(*item, GetState())
+ .WillRepeatedly(Return(DownloadItem::IN_PROGRESS));
+ EXPECT_CALL(*item, AddObserver(_))
+ .WillRepeatedly(Return());
+ EXPECT_CALL(*item, RemoveObserver(_))
+ .WillRepeatedly(Return());
ON_CALL(*item, GetFullPath())
.WillByDefault(ReturnRefOfCopy(FilePath()));
ON_CALL(*item, GetHash())

Powered by Google App Engine
This is Rietveld 408576698