Chromium Code Reviews| Index: remoting/client/chromoting_client_runtime.h |
| diff --git a/remoting/client/chromoting_client_runtime.h b/remoting/client/chromoting_client_runtime.h |
| index d3082760c6200e8934d36af9bc1ecd91cc4d0c1c..33767a23381ac7d9325474c050b88d9ffd957139 100644 |
| --- a/remoting/client/chromoting_client_runtime.h |
| +++ b/remoting/client/chromoting_client_runtime.h |
| @@ -70,7 +70,7 @@ class ChromotingClientRuntime { |
| return url_requester_; |
| } |
| - // Must call and use log_writter on the network thread. |
| + // Must call and use log_writer on the network thread. |
| ChromotingEventLogWriter* log_writer(); |
| private: |
| @@ -87,6 +87,12 @@ class ChromotingClientRuntime { |
| // References to native threads. |
| scoped_refptr<AutoThreadTaskRunner> ui_task_runner_; |
| + // TODO(nicholss): AuthThreads will be leaked because they depend on the main |
|
Yuwei
2017/04/03 19:29:22
s/AuthThreads/AutoThreads/
nicholss
2017/04/03 21:03:53
Done.
|
| + // thread. We should update this class to use regular threads like the client |
| + // plugin does. |
| + // Longer term we should migrate most of these to background tasks except the |
| + // network thread to TaskScheduler, removing the need for threads. |
| + |
| scoped_refptr<AutoThreadTaskRunner> display_task_runner_; |
| scoped_refptr<AutoThreadTaskRunner> network_task_runner_; |
| scoped_refptr<AutoThreadTaskRunner> file_task_runner_; |