Move decoder into separate thread, clean up API layering, and redo update protocl

remoting/host/chromoting_host.h

 // Copyright (c) 2010 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_CHROMOTING_HOST_H_
 #define REMOTING_CHROMOTING_HOST_H_
 
 #include <string>
 
 #include "base/thread.h"
@@ skipping 43 matching lines @@
 //    return to the idle state. We then go to step (2) if there a new
 //    incoming connection.
 class ChromotingHost : public base::RefCountedThreadSafe<ChromotingHost>,
                        public ClientConnection::EventHandler,
                        public JingleClient::Callback {
  public:
   ChromotingHost(ChromotingHostContext* context, MutableHostConfig* config,
                  Capturer* capturer, Encoder* encoder, EventExecutor* executor);
   virtual ~ChromotingHost();
 
-  // Start the host process. This method starts the chromoting host
-  // asynchronously.
+  // Asynchronously start the host process.
+  //
+  // After this is invoked, the host process will connect to the talk
+  // network and start listening for incoming connections.
   //
   // |shutdown_task| is called if Start() has failed ot Shutdown() is called
   // and all related operations are completed.
   //
   // This method can only be called once during the lifetime of this object.
   void Start(Task* shutdown_task);
 
-  // This method is called when we need to destroy the host process.
+  // Asynchronously shutdown the host process.
   void Shutdown();
 
   // This method is called if a client is connected to this object.
   void OnClientConnected(ClientConnection* client);
 
   // This method is called if a client is disconnected from the host.
   void OnClientDisconnected(ClientConnection* client);
 
   ////////////////////////////////////////////////////////////////////////////
   // ClientConnection::EventHandler implementations
@@ skipping 13 matching lines @@
       JingleClient* jingle,
       scoped_refptr<JingleChannel> channel);
 
  private:
   enum State {
     kInitial,
     kStarted,
     kStopped,
   };
 
-  // This method connects to the talk network and start listening for incoming
-  // connections.
-  void DoStart(Task* shutdown_task);
-
-  // This method shuts down the host process.
-  void DoShutdown();
-
   // The context that the chromoting host runs on.
   ChromotingHostContext* context_;
 
   scoped_refptr<MutableHostConfig> config_;
 
   // Capturer to be used by SessionManager. Once the SessionManager is
   // constructed this is set to NULL.
   scoped_ptr<Capturer> capturer_;
 
   // Encoder to be used by the SessionManager. Once the SessionManager is
@@ skipping 30 matching lines @@
 
   // Lock is to lock the access to |state_|.
   Lock lock_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingHost);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CHROMOTING_HOST_H_