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

Side by Side Diff: remoting/host/config_file_watcher_unittest.cc

Issue 18584011: Rename base::Delete to base::DeleteFile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/host/config_file_watcher.h" 5 #include "remoting/host/config_file_watcher.h"
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 "IPC thread", task_runner, base::MessageLoop::TYPE_IO); 82 "IPC thread", task_runner, base::MessageLoop::TYPE_IO);
83 83
84 // Create an instance of the config watcher. 84 // Create an instance of the config watcher.
85 watcher_.reset( 85 watcher_.reset(
86 new ConfigFileWatcher(task_runner, io_task_runner, &delegate_)); 86 new ConfigFileWatcher(task_runner, io_task_runner, &delegate_));
87 } 87 }
88 88
89 void ConfigFileWatcherTest::TearDown() { 89 void ConfigFileWatcherTest::TearDown() {
90 // Delete the test file. 90 // Delete the test file.
91 if (!config_file_.empty()) 91 if (!config_file_.empty())
92 base::Delete(config_file_, false); 92 base::DeleteFile(config_file_, false);
93 } 93 }
94 94
95 // Verifies that the initial notification is delivered. 95 // Verifies that the initial notification is delivered.
96 TEST_F(ConfigFileWatcherTest, Basic) { 96 TEST_F(ConfigFileWatcherTest, Basic) {
97 EXPECT_TRUE(file_util::CreateTemporaryFile(&config_file_)); 97 EXPECT_TRUE(file_util::CreateTemporaryFile(&config_file_));
98 98
99 std::string data("test"); 99 std::string data("test");
100 EXPECT_NE(file_util::WriteFile(config_file_, data.c_str(), 100 EXPECT_NE(file_util::WriteFile(config_file_, data.c_str(),
101 static_cast<int>(data.size())), -1); 101 static_cast<int>(data.size())), -1);
102 102
(...skipping 25 matching lines...) Expand all
128 128
129 // Modify the watched file. 129 // Modify the watched file.
130 std::string data("test"); 130 std::string data("test");
131 EXPECT_NE(file_util::WriteFile(config_file_, data.c_str(), 131 EXPECT_NE(file_util::WriteFile(config_file_, data.c_str(),
132 static_cast<int>(data.size())), -1); 132 static_cast<int>(data.size())), -1);
133 133
134 run_loop_.Run(); 134 run_loop_.Run();
135 } 135 }
136 136
137 } // namespace remoting 137 } // namespace remoting
OLDNEW
« no previous file with comments | « printing/backend/print_backend_cups.cc ('k') | remoting/host/pairing_registry_delegate_linux_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698