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

Unified Diff: net/dns/file_path_watcher_wrapper.h

Issue 9721002: [net/dns] Removes locking from DnsConfigServiceWin and adds local computer name. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Move lowercasing to ServeFromHosts, add test. Reorganize OnHostsRead. Created 8 years, 9 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
« no previous file with comments | « net/dns/dns_hosts_unittest.cc ('k') | net/dns/serial_worker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/dns/file_path_watcher_wrapper.h
diff --git a/net/dns/file_path_watcher_wrapper.h b/net/dns/file_path_watcher_wrapper.h
index 172bbaf65a576eb40ccfa2e0d288bcb7eda961fa..2a5a776651b61964c87e1ebbf661a62f0ab0333b 100644
--- a/net/dns/file_path_watcher_wrapper.h
+++ b/net/dns/file_path_watcher_wrapper.h
@@ -34,10 +34,8 @@ class NET_EXPORT FilePathWatcherWrapper
virtual ~FilePathWatcherWrapper();
// Starts watching the file at |path|. Returns true if Watch succeeds.
- // If so, the delegate will call callback.Run(true) from OnFilePathChanged and
- // callback.Run(false) from OnFilePathError. After failure the watch is
- // cancelled and will have to be restarted. If called again, the previous
- // watch is cancelled.
+ // If so, |callback| will be called with true on change and false on error.
+ // After failure the watch is cancelled and will have to be restarted.
bool Watch(const FilePath& path,
const base::Callback<void(bool succeeded)>& callback);
« no previous file with comments | « net/dns/dns_hosts_unittest.cc ('k') | net/dns/serial_worker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698