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

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

Issue 24579003: Clean up remaining unused globals (on mac). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: staticmedia Created 7 years, 2 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
11 #include "base/file_util.h" 11 #include "base/file_util.h"
12 #include "base/files/file_path_watcher.h" 12 #include "base/files/file_path_watcher.h"
13 #include "base/memory/scoped_ptr.h" 13 #include "base/memory/scoped_ptr.h"
14 #include "base/memory/weak_ptr.h" 14 #include "base/memory/weak_ptr.h"
15 #include "base/single_thread_task_runner.h" 15 #include "base/single_thread_task_runner.h"
16 #include "base/timer/timer.h" 16 #include "base/timer/timer.h"
17 17
18 namespace remoting { 18 namespace remoting {
19 19
20 // The name of the command-line switch used to specify the host configuration 20 // The name of the command-line switch used to specify the host configuration
21 // file to use. 21 // file to use.
22 const char kHostConfigSwitchName[] = "host-config"; 22 const char kHostConfigSwitchName[] = "host-config";
23 23
24 const base::FilePath::CharType kDefaultHostConfigFile[] = 24 const base::FilePath::CharType kDefaultHostConfigFile[] =
25 FILE_PATH_LITERAL("host.json"); 25 FILE_PATH_LITERAL("host.json");
26 26
27 #if defined(OS_WIN)
27 // Maximum number of times to try reading the configuration file before 28 // Maximum number of times to try reading the configuration file before
28 // reporting an error. 29 // reporting an error.
29 const int kMaxRetries = 3; 30 const int kMaxRetries = 3;
31 #endif // defined(OS_WIN)
30 32
31 class ConfigFileWatcherImpl 33 class ConfigFileWatcherImpl
32 : public base::RefCountedThreadSafe<ConfigFileWatcherImpl> { 34 : public base::RefCountedThreadSafe<ConfigFileWatcherImpl> {
33 public: 35 public:
34 // Creates a configuration file watcher that lives on the |io_task_runner| 36 // Creates a configuration file watcher that lives on the |io_task_runner|
35 // thread but posts config file updates on on |main_task_runner|. 37 // thread but posts config file updates on on |main_task_runner|.
36 ConfigFileWatcherImpl( 38 ConfigFileWatcherImpl(
37 scoped_refptr<base::SingleThreadTaskRunner> main_task_runner, 39 scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
38 scoped_refptr<base::SingleThreadTaskRunner> io_task_runner, 40 scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
39 ConfigFileWatcher::Delegate* delegate); 41 ConfigFileWatcher::Delegate* delegate);
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 if (config_ != config) { 209 if (config_ != config) {
208 config_ = config; 210 config_ = config;
209 main_task_runner_->PostTask( 211 main_task_runner_->PostTask(
210 FROM_HERE, 212 FROM_HERE,
211 base::Bind(&ConfigFileWatcher::Delegate::OnConfigUpdated, delegate_, 213 base::Bind(&ConfigFileWatcher::Delegate::OnConfigUpdated, delegate_,
212 config_)); 214 config_));
213 } 215 }
214 } 216 }
215 217
216 } // namespace remoting 218 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/base/auto_thread_unittest.cc ('k') | remoting/host/register_support_host_request_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698