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..de097966967777a1edebc6971bca6e087d7f60e3 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 {
@@ -20,10 +20,15 @@ class PepperXmppProxy : public XmppProxy {
  public:
   typedef base::Callback<void(const std::string&)> SendIqCallback;
 
+  // |plugin_task_runner| is the thread on which |send_iq_callback| is
+  // called. Normally the callback will call JavaScript, so this has
+  // to be the task runner that corresponds to the plugin
+  // thread. |callback_task_runner| is used to call the callback
+  // attacked with AttachCallback().

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

nit: atacked -> registered

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

Done.

   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);
 
   // Registered the callback class with this object.
   //
@@ -46,8 +51,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_;