MessageLoopProxy cleanups in remoting client.

remoting/client/frame_consumer_proxy.h

Index: remoting/client/frame_consumer_proxy.h
diff --git a/remoting/client/frame_consumer_proxy.h b/remoting/client/frame_consumer_proxy.h
index 0ee6e4660b805c544629cc11b92432a5dc43e402..efe9e6697f506447dc26f60f110a12a69422f460 100644
--- a/remoting/client/frame_consumer_proxy.h
+++ b/remoting/client/frame_consumer_proxy.h
@@ -15,7 +15,7 @@
 #include "remoting/client/frame_consumer.h"
 
 namespace base {
-class MessageLoopProxy;
+class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace remoting {
@@ -27,7 +27,7 @@ class FrameConsumerProxy
   // Constructs a proxy for |frame_consumer| which will trampoline invocations
   // to |frame_consumer_message_loop|.
   FrameConsumerProxy(
-      scoped_refptr<base::MessageLoopProxy> frame_consumer_message_loop);
+      scoped_refptr<base::SingleThreadTaskRunner> frame_consumer_task_runner);

[Wez, 2012/06/11 22:32:44]

nit: See comment below re re-naming.

[Sergey Ulanov, 2012/06/11 23:35:56]

Done.

 
   // FrameConsumer implementation.
   virtual void ApplyBuffer(const SkISize& view_size,
@@ -46,7 +46,7 @@ class FrameConsumerProxy
   virtual ~FrameConsumerProxy();
 
   base::WeakPtr<FrameConsumer> frame_consumer_;
-  scoped_refptr<base::MessageLoopProxy> frame_consumer_message_loop_;
+  scoped_refptr<base::SingleThreadTaskRunner> frame_consumer_task_runner_;

[Wez, 2012/06/11 22:32:44]

nit: You could just call this |task_runner_|, really; that it's the
FrameConsumer's task runner is pretty obvious from comments & usage.

[Sergey Ulanov, 2012/06/11 23:35:56]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(FrameConsumerProxy);
 };