Add VideoPacket struct for video packets. Refactor Decode interface to use it.

remoting/client/chromoting_client.h

Index: remoting/client/chromoting_client.h
diff --git a/remoting/client/chromoting_client.h b/remoting/client/chromoting_client.h
index 9ef2b39d46c546efae15e72b81dbda3a85d2d3d0..7bbe306b4a281085b5372cb0fb6df5363f756c1a 100644
--- a/remoting/client/chromoting_client.h
+++ b/remoting/client/chromoting_client.h
@@ -19,9 +19,7 @@ class MessageLoop;
 
 namespace remoting {
 
-class ChromotingHostMessage;
 class ClientContext;
-class InitClientMessage;
 class InputHandler;
 class RectangleUpdateDecoder;
 
@@ -55,8 +53,6 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
   virtual void SetViewport(int x, int y, int width, int height);
 
   // ConnectionToHost::HostEventCallback implementation.
-  virtual void HandleMessage(protocol::ConnectionToHost* conn,
-                             ChromotingHostMessage* messages);
   virtual void OnConnectionOpened(protocol::ConnectionToHost* conn);
   virtual void OnConnectionClosed(protocol::ConnectionToHost* conn);
   virtual void OnConnectionFailed(protocol::ConnectionToHost* conn);
@@ -75,6 +71,9 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
 
   MessageLoop* message_loop();
 
+  // Initializes connection.
+  void Initialize();

[awong, 2010/11/09 01:54:38]

Minor annoyance unrelated to this CL, but we should settle on Init versus
Initialize.  I think Initialize() is winning.

ajwong@ajwong:~/src/git-chrome/src$ find remoting/ -name '*.h' -exec grep -H
Init\(\) {} \; | grep -v ': *//'  | wc -l
2
ajwong@ajwong:~/src/git-chrome/src$ find remoting/ -name '*.h' -exec grep -H
Initialize\(\) {} \; | grep -v ': *//'  | wc -l
6

[Sergey Ulanov, 2010/11/09 21:34:42]

Agree

+
   // Convenience method for modifying the state on this object's message loop.
   void SetConnectionState(ConnectionState s);
 
@@ -84,9 +83,6 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
 
   void OnPacketDone();
 
-  // Handles for chromotocol messages.
-  void InitClient(const InitClientMessage& msg);
-
   // The following are not owned by this class.
   ClientConfig config_;
   ClientContext* context_;