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

remoting/client/chromoting_client.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.
 
 // ChromotingClient is the controller for the Client implementation.
 
 #ifndef REMOTING_CLIENT_CHROMOTING_CLIENT_H
 #define REMOTING_CLIENT_CHROMOTING_CLIENT_H
 
 #include "base/task.h"
 #include "remoting/client/host_connection.h"
 #include "remoting/client/client_config.h"
 
 class MessageLoop;
 
 namespace remoting {
 
 class ChromotingView;
 class ClientContext;
 class InputHandler;
+class ChromotingHostMessage;
+class InitClientMessage;
+class RectangleUpdateDecoder;
 
 class ChromotingClient : public HostConnection::HostEventCallback {
  public:
   // Objects passed in are not owned by this class.
   ChromotingClient(const ClientConfig& config,
                    ClientContext* context,
                    HostConnection* connection,
                    ChromotingView* view,
+                   RectangleUpdateDecoder* rectangle_decoder,
                    InputHandler* input_handler,
                    CancelableTask* client_done);
   virtual ~ChromotingClient();
 
   void Start();
   void Stop();
   void ClientDone();
 
   // Signals that the associated view may need updating.
   virtual void Repaint();
@@ skipping 18 matching lines @@
     CONNECTED,
     DISCONNECTED,
     FAILED,
   };
 
   MessageLoop* message_loop();
 
   // Convenience method for modifying the state on this object's message loop.
   void SetState(State s);
 
+  // If a message is not being processed, dispatches a single message from the
+  // |received_messages_| queue.
+  void DispatchMessage();
+
+  void OnMessageDone(ChromotingHostMessage* msg);
+
   // Handles for chromotocol messages.
-  void InitClient(ChromotingHostMessage* msg);
-  void BeginUpdate(ChromotingHostMessage* msg);
-  void HandleUpdate(ChromotingHostMessage* msg);
-  void EndUpdate(ChromotingHostMessage* msg);
+  void InitClient(const InitClientMessage& msg, Task* done);
 
   // The following are not owned by this class.
   ClientConfig config_;
   ClientContext* context_;
   HostConnection* connection_;
   ChromotingView* view_;
+  RectangleUpdateDecoder* rectangle_decoder_;
   InputHandler* input_handler_;
 
   // If non-NULL, this is called when the client is done.
   CancelableTask* client_done_;
 
   State state_;
 
+  // Contains all messages that have been received, but have not yet been
+  // processed.
+  //
+  // Used to serialize sending of messages to the client.
+  HostMessageList received_messages_;
+
+  // True if a message is being processed. Can be used to determine if it is
+  // safe to dispatch another message.
+  bool message_being_processed_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromotingClient);
 };
 
 }  // namespace remoting
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(remoting::ChromotingClient);
 
 #endif  // REMOTING_CLIENT_CHROMOTING_CLIENT_H