summaryrefslogtreecommitdiffstats
path: root/content/browser/download/download_item_impl_unittest.cc
diff options
context:
space:
mode:
authorbenjhayden@chromium.org <benjhayden@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-11-27 00:38:43 +0000
committerbenjhayden@chromium.org <benjhayden@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-11-27 00:38:43 +0000
commitfd6ddb8673b44079fb7257a412704f18421b2024 (patch)
tree0b10efaf191de02acc29ebc72be4d59051b6d839 /content/browser/download/download_item_impl_unittest.cc
parenta576e26ef8d6102d1ffbf0925ec862f8c1d7ac70 (diff)
downloadchromium_src-fd6ddb8673b44079fb7257a412704f18421b2024.zip
chromium_src-fd6ddb8673b44079fb7257a412704f18421b2024.tar.gz
chromium_src-fd6ddb8673b44079fb7257a412704f18421b2024.tar.bz2
Remove DownloadManager::active_downloads_
Use DownloadItem::IsInProgress() instead. Review URL: https://chromiumcodereview.appspot.com/10696096 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169540 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/download/download_item_impl_unittest.cc')
-rw-r--r--content/browser/download/download_item_impl_unittest.cc6
1 files changed, 0 insertions, 6 deletions
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc
index c29c99a..f338168 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -41,8 +41,6 @@ class MockDelegate : public DownloadItemImplDelegate {
MOCK_METHOD1(CheckForFileRemoval, void(DownloadItemImpl*));
MOCK_CONST_METHOD0(GetBrowserContext, BrowserContext*());
MOCK_METHOD1(UpdatePersistence, void(DownloadItemImpl*));
- MOCK_METHOD1(DownloadStopped, void(DownloadItemImpl*));
- MOCK_METHOD1(DownloadCompleted, void(DownloadItemImpl*));
MOCK_METHOD1(DownloadOpened, void(DownloadItemImpl*));
MOCK_METHOD1(DownloadRemoved, void(DownloadItemImpl*));
MOCK_METHOD1(ShowDownloadInBrowser, void(DownloadItemImpl*));
@@ -201,7 +199,6 @@ class DownloadItemTest : public testing::Test {
EXPECT_EQ(DownloadItem::IN_PROGRESS, item->GetState());
EXPECT_CALL(*download_file, Cancel());
- EXPECT_CALL(delegate_, DownloadStopped(item));
item->Cancel(true);
loop_.RunUntilIdle();
}
@@ -466,7 +463,6 @@ TEST_F(DownloadItemTest, CallbackAfterRename) {
item->OnAllDataSaved("");
EXPECT_CALL(*download_file, RenameAndAnnotate(final_path, _))
.WillOnce(ScheduleRenameCallback(final_path));
- EXPECT_CALL(*mock_delegate(), DownloadCompleted(item));
EXPECT_CALL(*mock_delegate(), ShouldOpenDownload(item, _))
.WillOnce(Return(true));
EXPECT_CALL(*download_file, Detach());
@@ -501,7 +497,6 @@ TEST_F(DownloadItemTest, Canceled) {
MockDownloadFile* download_file = AddDownloadFileToDownloadItem(item, NULL);
// Confirm cancel sets state properly.
- EXPECT_CALL(*mock_delegate(), DownloadStopped(item));
EXPECT_CALL(*download_file, Cancel());
item->Cancel(true);
EXPECT_EQ(DownloadItem::CANCELLED, item->GetState());
@@ -555,7 +550,6 @@ TEST_F(DownloadItemTest, DestinationError) {
EXPECT_EQ(DOWNLOAD_INTERRUPT_REASON_NONE, item->GetLastReason());
EXPECT_FALSE(observer.CheckUpdated());
- EXPECT_CALL(*mock_delegate(), DownloadStopped(item));
EXPECT_CALL(*download_file, Cancel());
as_observer->DestinationError(
DOWNLOAD_INTERRUPT_REASON_FILE_ACCESS_DENIED);