MessageLoopProxy cleanups in remoting client.

remoting/client/chromoting_client.h

Index: remoting/client/chromoting_client.h
diff --git a/remoting/client/chromoting_client.h b/remoting/client/chromoting_client.h
index 6856a201ae0636077ed35301893057cc948adc1b..4cabb14ab31f2dbc9213161c6c0318de7e0ea9c6 100644
--- a/remoting/client/chromoting_client.h
+++ b/remoting/client/chromoting_client.h
@@ -22,8 +22,6 @@
 #include "remoting/protocol/video_stub.h"
 #include "remoting/jingle_glue/xmpp_proxy.h"
 
-class MessageLoop;
-
 namespace remoting {
 
 namespace protocol {
@@ -43,14 +41,13 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
                    ClientContext* context,
                    protocol::ConnectionToHost* connection,
                    ChromotingView* view,
-                   RectangleUpdateDecoder* rectangle_decoder,
-                   const base::Closure& client_done);
+                   RectangleUpdateDecoder* rectangle_decoder);
   virtual ~ChromotingClient();
 
+  // Start/stop the client. Must be called on the main thread.
   void Start(scoped_refptr<XmppProxy> xmpp_proxy,
              scoped_ptr<protocol::TransportFactory> transport_factory);
   void Stop(const base::Closure& shutdown_task);
-  void ClientDone();
 
   // Return the stats recorded by this client.
   ChromotingStats* GetStats();
@@ -82,7 +79,7 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
     base::Closure done;
   };
 
-  base::MessageLoopProxy* message_loop();
+  base::SingleThreadTaskRunner* task_runner();

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

nit: It would be simpler to just take a scoped_refptr<> to the TaskRunner rather
than reach it through ClientContext every time.

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

nit: Where is SingleThreadTaskRunner defined in the #includes? I think you
probably still need a forward-define here to satisy IWYU?

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

Done.

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

Done.

 
   // Initializes connection.
   void Initialize();