Cleanups in the video encoding decoding code. Reenable VP8.

remoting/host/client_connection.cc

 // 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.
 
 // TODO(hclam): Remove this header once MessageDispatcher is used.
 #include "remoting/base/multiple_array_input_stream.h"
 #include "remoting/host/client_connection.h"
 
 #include "google/protobuf/message.h"
 #include "net/base/io_buffer.h"
@@ skipping 37 matching lines @@
   if (!connection_)
     return;
 
   ChromotingHostMessage msg;
   msg.mutable_init_client()->set_width(width);
   msg.mutable_init_client()->set_height(height);
   DCHECK(msg.IsInitialized());
   video_writer_.SendMessage(msg);
 }
 
-void ClientConnection::SendUpdateStreamPacketMessage(
-    const ChromotingHostMessage& message) {
+void ClientConnection::SendVideoPacket(const VideoPacket& packet) {
   DCHECK_EQ(loop_, MessageLoop::current());
 
   // If we are disconnected then return.
   if (!connection_)
     return;
 
-  video_writer_.SendMessage(message);
+  ChromotingHostMessage* message = new ChromotingHostMessage();
+  // TODO(sergeyu): avoid memcopy here.
+  *message->mutable_video_packet() = packet;
+
+  video_writer_.SendMessage(*message);
+
+  delete message;
 }
 
 int ClientConnection::GetPendingUpdateStreamMessages() {
   DCHECK_EQ(loop_, MessageLoop::current());
   return video_writer_.GetPendingMessages();
 }
 
 void ClientConnection::Disconnect() {
   DCHECK_EQ(loop_, MessageLoop::current());
 
@@ skipping 52 matching lines @@
   DCHECK_EQ(loop_, MessageLoop::current());
   DCHECK(handler_);
   handler_->HandleMessage(this, message);
 }
 
 // OnClosed() is used as a callback for ChromotocolConnection::Close().
 void ClientConnection::OnClosed() {
 }
 
 }  // namespace remoting