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

Unified Diff: chrome/browser/chromeos/drive/download_handler_unittest.cc

Issue 2318023002: //chrome/browser/chromeos: Change ScopedTempDir::path() to GetPath() (Closed)
Patch Set: Address comment and also remove non-trivial cases Created 4 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/chromeos/drive/download_handler_unittest.cc
diff --git a/chrome/browser/chromeos/drive/download_handler_unittest.cc b/chrome/browser/chromeos/drive/download_handler_unittest.cc
index 07897789db04dc6f4c2898a44c6d7cac3cd2b3cf..8fdaef79825430e64763dc2d07309c2dc278a468 100644
--- a/chrome/browser/chromeos/drive/download_handler_unittest.cc
+++ b/chrome/browser/chromeos/drive/download_handler_unittest.cc
@@ -99,7 +99,7 @@ class DownloadHandlerTest : public testing::Test {
.WillRepeatedly(testing::Return(content::DownloadItem::IN_PROGRESS));
download_handler_.reset(new DownloadHandler(&test_file_system_));
- download_handler_->Initialize(download_manager_.get(), temp_dir_.path());
+ download_handler_->Initialize(download_manager_.get(), temp_dir_.GetPath());
download_handler_->SetFreeDiskSpaceDelayForTesting(
base::TimeDelta::FromMilliseconds(0));
}
@@ -149,7 +149,7 @@ TEST_F(DownloadHandlerTest, SubstituteDriveDownloadPath) {
content::RunAllBlockingPoolTasksUntilIdle();
// Check the result.
- EXPECT_TRUE(temp_dir_.path().IsParent(substituted_path));
+ EXPECT_TRUE(temp_dir_.GetPath().IsParent(substituted_path));
ASSERT_TRUE(download_handler_->IsDriveDownload(&download_item_));
EXPECT_EQ(drive_path, download_handler_->GetTargetPath(&download_item_));
}
@@ -190,7 +190,7 @@ TEST_F(DownloadHandlerTest, SubstituteDriveDownloadPathForSavePackage) {
content::RunAllBlockingPoolTasksUntilIdle();
// Check the result of SubstituteDriveDownloadPath().
- EXPECT_TRUE(temp_dir_.path().IsParent(substituted_path));
+ EXPECT_TRUE(temp_dir_.GetPath().IsParent(substituted_path));
// |download_item_| is not a drive download yet.
EXPECT_FALSE(download_handler_->IsDriveDownload(&download_item_));

Powered by Google App Engine
This is Rietveld 408576698