MessageLoopProxy cleanups in remoting client.

remoting/client/plugin/pepper_xmpp_proxy.h

Index: remoting/client/plugin/pepper_xmpp_proxy.h
diff --git a/remoting/client/plugin/pepper_xmpp_proxy.h b/remoting/client/plugin/pepper_xmpp_proxy.h
index 40ba4bad762ab582201f3d6eddb6ed95b01afb32..d2808f9eb1d9cea2e8ac91f9eb7ae3830d9cb5bc 100644
--- a/remoting/client/plugin/pepper_xmpp_proxy.h
+++ b/remoting/client/plugin/pepper_xmpp_proxy.h
@@ -11,7 +11,7 @@
 #include "remoting/jingle_glue/xmpp_proxy.h"
 
 namespace base {
-class MessageLoopProxy;
+class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace remoting {
@@ -22,8 +22,8 @@ class PepperXmppProxy : public XmppProxy {
 
   PepperXmppProxy(
       const SendIqCallback& send_iq_callback,
-      base::MessageLoopProxy* plugin_message_loop,
-      base::MessageLoopProxy* callback_message_loop);
+      base::SingleThreadTaskRunner* plugin_task_runner,
+      base::SingleThreadTaskRunner* callback_task_runner);

[Wez, 2012/05/26 00:18:40]

nit: scoped_refptr<>?

[Wez, 2012/05/26 00:18:40]

nit: Clarify the roles of the two TaskRunners.

[Sergey Ulanov, 2012/05/30 01:07:07]

Will fix in a separate CL.

[Sergey Ulanov, 2012/05/30 01:07:07]

Done.

 
   // Registered the callback class with this object.
   //
@@ -46,8 +46,8 @@ class PepperXmppProxy : public XmppProxy {
 
   SendIqCallback send_iq_callback_;
 
-  scoped_refptr<base::MessageLoopProxy> plugin_message_loop_;
-  scoped_refptr<base::MessageLoopProxy> callback_message_loop_;
+  scoped_refptr<base::SingleThreadTaskRunner> plugin_task_runner_;
+  scoped_refptr<base::SingleThreadTaskRunner> callback_task_runner_;
 
   // Must only be access on callback_message_loop_.
   base::WeakPtr<ResponseCallback> callback_;