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

remoting/client/frame_consumer.h

Index: remoting/client/frame_consumer.h
diff --git a/remoting/client/frame_consumer.h b/remoting/client/frame_consumer.h
index 05c4448db149cdcaac0d0d4f4c751f6b7edf0dd5..90f5e5509b2eca5b744170814a4d3b3573c589c9 100644
--- a/remoting/client/frame_consumer.h
+++ b/remoting/client/frame_consumer.h
@@ -5,6 +5,10 @@
 #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 {
@@ -41,8 +45,8 @@ class FrameConsumer {
   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.