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

Unified Diff: content/browser/download/download_item_impl_unittest.cc

Issue 2823273004: Add new UMA stats for parallelizable download (Closed)
Patch Set: Created 3 years, 8 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: content/browser/download/download_item_impl_unittest.cc
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc
index 5b87bfb80859f539cb3e0d2e617ec8a92d2cc829..dd58f934043511f9c85d136992f55e4c37c6d784 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -309,7 +309,7 @@ class DownloadItemTest : public testing::Test {
if (create_info_->result == DOWNLOAD_INTERRUPT_REASON_NONE) {
mock_download_file = new StrictMock<MockDownloadFile>;
download_file.reset(mock_download_file);
- EXPECT_CALL(*mock_download_file, Initialize(_, _))
+ EXPECT_CALL(*mock_download_file, Initialize(_, _, _))
.WillOnce(ScheduleCallbackWithParam(DOWNLOAD_INTERRUPT_REASON_NONE));
EXPECT_CALL(*mock_download_file, FullPath())
.WillRepeatedly(ReturnRefOfCopy(base::FilePath()));
@@ -601,7 +601,7 @@ TEST_F(DownloadItemTest, NotificationAfterTogglePause) {
std::unique_ptr<DownloadRequestHandleInterface> request_handle(
new NiceMock<MockRequestHandle>);
- EXPECT_CALL(*mock_download_file, Initialize(_, _));
+ EXPECT_CALL(*mock_download_file, Initialize(_, _, _));
EXPECT_CALL(*mock_delegate(), DetermineDownloadTarget(_, _));
item->Start(std::move(download_file), std::move(request_handle),
*create_info());
@@ -1068,7 +1068,7 @@ TEST_F(DownloadItemTest, Start) {
MockDownloadFile* mock_download_file(new MockDownloadFile);
std::unique_ptr<DownloadFile> download_file(mock_download_file);
DownloadItemImpl* item = CreateDownloadItem();
- EXPECT_CALL(*mock_download_file, Initialize(_, _));
+ EXPECT_CALL(*mock_download_file, Initialize(_, _, _));
std::unique_ptr<DownloadRequestHandleInterface> request_handle(
new NiceMock<MockRequestHandle>);
EXPECT_CALL(*mock_delegate(), DetermineDownloadTarget(item, _));
@@ -1089,7 +1089,7 @@ TEST_F(DownloadItemTest, InitDownloadFileFails) {
EXPECT_CALL(*file, Cancel());
EXPECT_CALL(*request_handle, CancelRequest());
- EXPECT_CALL(*file, Initialize(_, _))
+ EXPECT_CALL(*file, Initialize(_, _, _))
.WillOnce(ScheduleCallbackWithParam(
DOWNLOAD_INTERRUPT_REASON_FILE_ACCESS_DENIED));
@@ -2111,7 +2111,7 @@ TEST_P(DownloadItemDestinationUpdateRaceTest, DownloadCancelledByUser) {
base::RunLoop download_start_loop;
DownloadFile::InitializeCallback initialize_callback;
- EXPECT_CALL(*file_, Initialize(_, _))
+ EXPECT_CALL(*file_, Initialize(_, _, _))
.WillOnce(DoAll(SaveArg<0>(&initialize_callback),
ScheduleClosure(download_start_loop.QuitClosure())));
item_->Start(std::move(file_), std::move(request_handle_), *create_info());
@@ -2164,7 +2164,7 @@ TEST_P(DownloadItemDestinationUpdateRaceTest, IntermediateRenameFails) {
base::RunLoop download_start_loop;
DownloadFile::InitializeCallback initialize_callback;
- EXPECT_CALL(*file_, Initialize(_, _))
+ EXPECT_CALL(*file_, Initialize(_, _, _))
.WillOnce(DoAll(SaveArg<0>(&initialize_callback),
ScheduleClosure(download_start_loop.QuitClosure())));
@@ -2234,7 +2234,7 @@ TEST_P(DownloadItemDestinationUpdateRaceTest, IntermediateRenameSucceeds) {
base::RunLoop download_start_loop;
DownloadFile::InitializeCallback initialize_callback;
- EXPECT_CALL(*file_, Initialize(_, _))
+ EXPECT_CALL(*file_, Initialize(_, _, _))
.WillOnce(DoAll(SaveArg<0>(&initialize_callback),
ScheduleClosure(download_start_loop.QuitClosure())));

Powered by Google App Engine
This is Rietveld 408576698