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

Unified Diff: chrome/browser/chromeos/file_manager/file_watcher_unittest.cc

Issue 2345473002: Change ScopedTempDir::path() to GetPath() in chrome/browser/chromeos/file_manager (Closed)
Patch Set: 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/file_manager/file_watcher_unittest.cc
diff --git a/chrome/browser/chromeos/file_manager/file_watcher_unittest.cc b/chrome/browser/chromeos/file_manager/file_watcher_unittest.cc
index 9b0da23bb364812a77378991c6b5401ac4855476..6ba94877551948434960fe90a328bc6d22f3e523 100644
--- a/chrome/browser/chromeos/file_manager/file_watcher_unittest.cc
+++ b/chrome/browser/chromeos/file_manager/file_watcher_unittest.cc
@@ -125,10 +125,9 @@ TEST_F(FileManagerFileWatcherTest, WatchLocalFile) {
bool on_change_error = false;
base::RunLoop run_loop;
file_watcher->WatchLocalFile(
- temp_dir.path(),
+ temp_dir.GetPath(),
CreateQuitCallback(
- &run_loop,
- CreateCopyResultCallback(&changed_path, &on_change_error)),
+ &run_loop, CreateCopyResultCallback(&changed_path, &on_change_error)),
CreateCopyResultCallback(&watcher_created));
// Spin the message loop so the base::FilePathWatcher is created.
base::RunLoop().RunUntilIdle();
@@ -137,11 +136,12 @@ TEST_F(FileManagerFileWatcherTest, WatchLocalFile) {
// Create a temporary file in the temporary directory. The file watcher
// should detect the change in the directory.
base::FilePath temp_file_path;
- ASSERT_TRUE(base::CreateTemporaryFileInDir(temp_dir.path(), &temp_file_path));
+ ASSERT_TRUE(
+ base::CreateTemporaryFileInDir(temp_dir.GetPath(), &temp_file_path));
// Wait until the directory change is notified.
run_loop.Run();
ASSERT_FALSE(on_change_error);
- ASSERT_EQ(temp_dir.path().value(), changed_path.value());
+ ASSERT_EQ(temp_dir.GetPath().value(), changed_path.value());
// This is ugly, but FileWatcher should be deleted explicitly here, and
// spin the message loop so the base::FilePathWatcher is deleted.

Powered by Google App Engine
This is Rietveld 408576698