diff options
| author | asanka@chromium.org <asanka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-02 22:32:16 +0000 |
|---|---|---|
| committer | asanka@chromium.org <asanka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-02 22:32:16 +0000 |
| commit | 11be6a681bbf17383ee53105e7e7b2edfd52ade4 (patch) | |
| tree | 26761aa2d86cdccf95d6261b4fb0e9eba54d8e4a /content/browser/download/download_item_impl_unittest.cc | |
| parent | 7aa01c67cab5acc2dc52e391d90de2844ab7998d (diff) | |
| download | chromium_src-11be6a681bbf17383ee53105e7e7b2edfd52ade4.zip chromium_src-11be6a681bbf17383ee53105e7e7b2edfd52ade4.tar.gz chromium_src-11be6a681bbf17383ee53105e7e7b2edfd52ade4.tar.bz2 | |
Download filename determination refactor (2/3)
Introduce DownloadReservationTracker for tracking filenames that
are being used by active downloads. This eliminates the need to
keep marker files on-disk to prevent using the same path for
multiple downloads.
BUG=78085
TEST=unit tests
TBR=sky
Review URL: https://chromiumcodereview.appspot.com/10543127
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145208 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.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc index eb6607b..c7ac84d 100644 --- a/content/browser/download/download_item_impl_unittest.cc +++ b/content/browser/download/download_item_impl_unittest.cc @@ -389,8 +389,7 @@ TEST_F(DownloadItemTest, NotificationAfterOnIntermediatePathDetermined) { RenameDownloadFile(_,intermediate_path,false,_)) .WillOnce(ScheduleRenameCallback(new_intermediate_path)); - item->OnIntermediatePathDetermined(file_manager.get(), intermediate_path, - false /* ok_to_overwrite */); + item->OnIntermediatePathDetermined(file_manager.get(), intermediate_path); EXPECT_FALSE(observer.CheckUpdated()); RunAllPendingInMessageLoops(); EXPECT_TRUE(observer.CheckUpdated()); @@ -514,8 +513,7 @@ TEST_F(DownloadItemTest, CallbackAfterRename) { item->OnTargetPathDetermined(final_path, DownloadItem::TARGET_DISPOSITION_OVERWRITE, content::DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS); - item->OnIntermediatePathDetermined(file_manager.get(), intermediate_path, - false /* ok_to_overwrite */); + item->OnIntermediatePathDetermined(file_manager.get(), intermediate_path); RunAllPendingInMessageLoops(); // All the callbacks should have happened by now. ::testing::Mock::VerifyAndClearExpectations(file_manager.get()); |