Cleanups in the video encoding decoding code. Reenable VP8.

1. Moved video-related protobuf messages from event.proto to video.proto. Removed those that we don't need anymore
2. Fixed naming for enums and some types.
3. Reenabled VP8.
4. Proper RGB-YUV converter for VP8 encoder.
5. Changed the capturer_fake to show more meaningful picture.

BUG=57374
TEST=unittests

Review URL: http://codereview.chromium.org/4136010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64672 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 6 deletions

diff --git a/remoting/client/rectangle_update_decoder.h b/remoting/client/rectangle_update_decoder.h
index b383c20..e6c344e 100644
--- a/remoting/client/rectangle_update_decoder.h
+++ b/remoting/client/rectangle_update_decoder.h
@@ -16,8 +16,8 @@ namespace remoting {
 
 class Decoder;
 class FrameConsumer;
-class RectangleFormat;
-class RectangleUpdatePacket;
+class VideoPacketFormat;
+class VideoPacket;
 
 // TODO(ajwong): Re-examine this API, especially with regards to how error
 // conditions on each step are reported.  Should they be CHECKs? Logs? Other?
@@ -34,14 +34,14 @@ class RectangleUpdateDecoder {
   //
   // TODO(ajwong): Should packet be a const pointer to make the lifetime
   // more clear?
-  void DecodePacket(const RectangleUpdatePacket& packet, Task* done);
+  void DecodePacket(const VideoPacket& packet, Task* done);
 
  private:
-  static bool IsValidPacket(const RectangleUpdatePacket& packet);
+  static bool IsValidPacket(const VideoPacket& packet);
 
-  void InitializeDecoder(const RectangleFormat& format, Task* done);
+  void InitializeDecoder(const VideoPacketFormat& format, Task* done);
 
-  void ProcessPacketData(const RectangleUpdatePacket& packet, Task* done);
+  void ProcessPacketData(const VideoPacket& packet, Task* done);
 
   // Pointers to infrastructure objects.  Not owned.
   MessageLoop* message_loop_;