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

Unified Diff: chrome/browser/media/webrtc/webrtc_log_util_unittest.cc

Issue 2318033002: c/browser, c/common, components M-N: Change ScopedTempDir::path() to GetPath() (Closed)
Patch Set: Fix ItunesFileUtilTest 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/media/webrtc/webrtc_log_util_unittest.cc
diff --git a/chrome/browser/media/webrtc/webrtc_log_util_unittest.cc b/chrome/browser/media/webrtc/webrtc_log_util_unittest.cc
index 22aac4bf7aaf7d9a56f3f34a92478d10bd59311c..6bdd7debc7f316e1fbb881a280c251ea37d6220d 100644
--- a/chrome/browser/media/webrtc/webrtc_log_util_unittest.cc
+++ b/chrome/browser/media/webrtc/webrtc_log_util_unittest.cc
@@ -26,13 +26,13 @@ class WebRtcLogUtilTest : public testing::Test {
// when deleting old logs.
ASSERT_TRUE(dir_.CreateUniqueTempDir());
base::FilePath file;
- ASSERT_TRUE(CreateTemporaryFileInDir(dir_.path(), &file));
- ASSERT_TRUE(CreateTemporaryFileInDir(dir_.path(), &file));
+ ASSERT_TRUE(CreateTemporaryFileInDir(dir_.GetPath(), &file));
+ ASSERT_TRUE(CreateTemporaryFileInDir(dir_.GetPath(), &file));
base::Time time_expect_to_keep =
base::Time::Now() -
base::TimeDelta::FromDays(kExpectedDaysToKeepLogFiles - 1);
TouchFile(file, time_expect_to_keep, time_expect_to_keep);
- ASSERT_TRUE(CreateTemporaryFileInDir(dir_.path(), &file));
+ ASSERT_TRUE(CreateTemporaryFileInDir(dir_.GetPath(), &file));
base::Time time_expect_to_delete =
base::Time::Now() -
base::TimeDelta::FromDays(kExpectedDaysToKeepLogFiles + 1);
@@ -40,7 +40,8 @@ class WebRtcLogUtilTest : public testing::Test {
}
void VerifyFiles(int expected_files) {
- base::FileEnumerator files(dir_.path(), false, base::FileEnumerator::FILES);
+ base::FileEnumerator files(dir_.GetPath(), false,
+ base::FileEnumerator::FILES);
int file_counter = 0;
for (base::FilePath name = files.Next(); !name.empty();
name = files.Next()) {
@@ -57,14 +58,14 @@ class WebRtcLogUtilTest : public testing::Test {
};
TEST_F(WebRtcLogUtilTest, DeleteOldWebRtcLogFiles) {
- WebRtcLogUtil::DeleteOldWebRtcLogFiles(dir_.path());
+ WebRtcLogUtil::DeleteOldWebRtcLogFiles(dir_.GetPath());
VerifyFiles(2);
}
TEST_F(WebRtcLogUtilTest, DeleteOldAndRecentWebRtcLogFiles) {
base::Time time_begin_delete =
base::Time::Now() - base::TimeDelta::FromDays(1);
- WebRtcLogUtil::DeleteOldAndRecentWebRtcLogFiles(dir_.path(),
+ WebRtcLogUtil::DeleteOldAndRecentWebRtcLogFiles(dir_.GetPath(),
time_begin_delete);
VerifyFiles(1);
}

Powered by Google App Engine
This is Rietveld 408576698