Cleanups in the video encoding decoding code. Reenable VP8.

remoting/base/decoder_vp8.h

Index: remoting/base/decoder_vp8.h
diff --git a/remoting/base/decoder_vp8.h b/remoting/base/decoder_vp8.h
index 5fe01694a7a6d143321eea7a8c01593f0151f29b..dfef0b7293af428f484d386ac85d311258a9dedd 100644
--- a/remoting/base/decoder_vp8.h
+++ b/remoting/base/decoder_vp8.h
@@ -14,37 +14,34 @@ namespace remoting {
 class DecoderVp8 : public Decoder {
  public:
   DecoderVp8();
-  ~DecoderVp8();
+  virtual ~DecoderVp8();
 
   // Decoder implementations.
-  virtual bool BeginDecode(scoped_refptr<media::VideoFrame> frame,
-                           UpdatedRects* update_rects,
-                           Task* partial_decode_done,
-                           Task* decode_done);
-  virtual bool PartialDecode(ChromotingHostMessage* message);
-  virtual void EndDecode();
+  virtual void Initialize(scoped_refptr<media::VideoFrame> frame,
+                          const gfx::Rect& clip, int bytes_per_src_pixel);
+
+  virtual void Reset();
+
+  // Feeds more data into the decoder.
+  virtual void DecodeBytes(const std::string& encoded_bytes);
+
+  // Returns true if decoder is ready to accept data via ProcessRectangleData.
+  virtual bool IsReadyForData();
+
+  virtual VideoPacketFormat::Encoding Encoding();
 
  private:
-  bool HandleBeginRect(ChromotingHostMessage* message);
-  bool HandleRectData(ChromotingHostMessage* message);
-  bool HandleEndRect(ChromotingHostMessage* message);
+  enum State {
+    kUninitialized,
+    kReady,
+    kError,
+  };
 
   // The internal state of the decoder.
   State state_;
 
-  // Keeps track of the updating rect.
-  int rect_x_;
-  int rect_y_;
-  int rect_width_;
-  int rect_height_;
-
-  // Tasks to call when decode is done.
-  scoped_ptr<Task> partial_decode_done_;
-  scoped_ptr<Task> decode_done_;
-
   // The video frame to write to.
   scoped_refptr<media::VideoFrame> frame_;
-  UpdatedRects* updated_rects_;
 
   vpx_codec_ctx_t* codec_;