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

remoting/client/rectangle_update_decoder.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.
 
 #ifndef REMOTING_CLIENT_RECTANGLE_UPDATE_DECODER_H
 #define REMOTING_CLIENT_RECTANGLE_UPDATE_DECODER_H
 
 #include "base/scoped_ptr.h"
 #include "base/task.h"
 #include "media/base/video_frame.h"
-#include "remoting/base/decoder.h"  // For UpdatedRects.
+#include "remoting/base/decoder.h"  // For UpdatedRects
 
 class MessageLoop;
 
 namespace remoting {
 
+class Decoder;
 class FrameConsumer;
 class RectangleFormat;
 class RectangleUpdatePacket;
 
 // TODO(ajwong): Re-examine this API, especially with regards to how error
 // conditions on each step are reported.  Should they be CHECKs? Logs? Other?
 class RectangleUpdateDecoder {
  public:
   RectangleUpdateDecoder(MessageLoop* message_loop,
                          FrameConsumer* consumer);
@@ skipping 24 matching lines @@
 
   // Framebuffer for the decoder.
   scoped_refptr<media::VideoFrame> frame_;
 };
 
 }  // namespace remoting
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(remoting::RectangleUpdateDecoder);
 
 #endif  // REMOTING_CLIENT_RECTANGLE_UPDATE_DECODER_H