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

remoting/client/frame_consumer.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_FRAME_CONSUMER_H_
 #define REMOTING_CLIENT_FRAME_CONSUMER_H_
 
+#include "remoting/base/decoder.h"  // For UpdatedRects
+
+class Task;
+
 namespace remoting {
 
 class FrameConsumer {
  public:
   FrameConsumer() {}
   virtual ~FrameConsumer() {}
 
   // Request a frame be allocated from the FrameConsumer.
   //
   // If a frame cannot be allocated to fit the format, and height/width
@@ skipping 16 matching lines @@
                              size_t width,
                              size_t height,
                              base::TimeDelta timestamp,
                              base::TimeDelta duration,
                              scoped_refptr<media::VideoFrame>* frame_out,
                              Task* done) = 0;
 
   virtual void ReleaseFrame(media::VideoFrame* frame) = 0;
 
   // OnPartialFrameOutput() is called every time at least one rectangle of
-  // output is produced.  The |frame| is guaranteed to have valid data for
-  // every region included in the |rects| list.
+  // output is produced.  The |frame| is guaranteed to have valid data for every
+  // region included in the |rects| list.
   //
   // Both |frame| and |rects| are guaranteed to be valid until the |done|
   // callback is invoked.
   virtual void OnPartialFrameOutput(media::VideoFrame* frame,
                                     UpdatedRects* rects,
                                     Task* done) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(FrameConsumer);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_FRAME_CONSUMER_H_