Piping for audio decoding.

remoting/client/client_context.h

Index: remoting/client/client_context.h
diff --git a/remoting/client/client_context.h b/remoting/client/client_context.h
index 27a44162d3eaa402928ea48ca661e674231b2af8..af99d4cd7934b5941b685fdbabc2cdac7181d396 100644
--- a/remoting/client/client_context.h
+++ b/remoting/client/client_context.h
@@ -27,15 +27,19 @@ class ClientContext {
   void Start();
   void Stop();
 
-  base::SingleThreadTaskRunner* main_task_runner();
-  base::SingleThreadTaskRunner* decode_task_runner();
+  base::SingleThreadTaskRunner* main_task_runner() const;

[Sergey Ulanov, 2012/08/03 17:59:55]

I don't think that const is justified here. The task runners are owned by the
context, and you can mutate them by posting tasks so it's not really const. Also
see my other comment about passing ClientContext by reference.

[kxing, 2012/08/03 20:55:13]

Done.

+  base::SingleThreadTaskRunner* decode_task_runner() const;
+  base::SingleThreadTaskRunner* audio_decode_task_runner() const;
 
  private:
   scoped_refptr<base::SingleThreadTaskRunner> main_task_runner_;
 
-  // A thread that handles all decode operations.
+  // A thread that handles all video decode operations.
   base::Thread decode_thread_;
 
+  // A thread that handles all audio decode operations.
+  base::Thread audio_decode_thread_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientContext);
 };