[Chromoting] Moving the daemon IPC channel to ChromotingHostContext.

remoting/host/chromoting_host_context.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
 #define REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/threading/platform_thread.h"
 #include "base/threading/thread.h"
 
 namespace base {
 class SingleThreadTaskRunner;
 }  // namespace base
 
+namespace IPC {
+class ChannelProxy;
+class Listener;
+}  // namespace IPC
+
 namespace net {
 class URLRequestContextGetter;
 }  // namespace net
 
 namespace remoting {
 
 // A class that manages threads and running context for the chromoting host
 // process.  This class is virtual only for testing purposes (see below).
 class ChromotingHostContext {
  public:
@@ skipping 29 matching lines @@
   //
   // TODO(sergeyu): Do we need a separate thread for EventExecutor?
   // Can we use some other thread instead?
   virtual base::SingleThreadTaskRunner* desktop_task_runner();
 
   // Task runner for the thread that is used for blocking file
   // IO. This thread is used by the URLRequestContext to read proxy
   // configuration and by NatConfig to read policy configs.
   virtual base::SingleThreadTaskRunner* file_task_runner();
 
+  // Returns a pointer to the IPC channel connecting this process with
+  // the daemon process. NULL can be returned if no channel exists.

[simonmorris, 2012/08/16 20:43:02]

"can be" -> "is".

[alexeypa (please no reviews), 2012/08/16 20:53:52]

Done.

+  IPC::ChannelProxy* daemon_channel();
+
+  // Connects to the endpoint provided by the daemon procesess.

[simonmorris, 2012/08/16 20:43:02]

"process", or "processes"?

[alexeypa (please no reviews), 2012/08/16 20:53:52]

Done.

+  void ConnectToDaemon(const std::string& channel_name,
+                       IPC::Listener* listener);
+
+  // Disconnects from the daemon procesess.

[simonmorris, 2012/08/16 20:43:02]

same

[alexeypa (please no reviews), 2012/08/16 20:53:52]

Done.

+  void DisconnectFromDaemon();
+
   const scoped_refptr<net::URLRequestContextGetter>&
       url_request_context_getter();
 
  private:
   FRIEND_TEST_ALL_PREFIXES(ChromotingHostContextTest, StartAndStop);
 
   // A thread that hosts all network operations.
   base::Thread network_thread_;
 
   // A thread that hosts screen capture.
   base::Thread capture_thread_;
 
   // A thread that hosts all encode operations.
   base::Thread encode_thread_;
 
   // A thread that hosts input injection.
   base::Thread desktop_thread_;
 
   // Thread for blocking IO operations.
   base::Thread file_thread_;
 
+  // The IPC channel connection us to the daemon process.

[simonmorris, 2012/08/16 20:43:02]

"connection" -> "connecting"

[alexeypa (please no reviews), 2012/08/16 20:53:52]

Done.

+  scoped_ptr<IPC::ChannelProxy> daemon_channel_;
+
   scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
 
   scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingHostContext);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CHROMOTING_HOST_CONTEXT_H_