Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1317)

Unified Diff: remoting/client/chromoting_client.h

Issue 10454018: MessageLoopProxy cleanups in remoting client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/client/chromoting_client.h
diff --git a/remoting/client/chromoting_client.h b/remoting/client/chromoting_client.h
index d784b5a59c5a18b3fbff36ecd647fff4584d3636..a72109eb83b6cb889fe835fd126db64cd75d13c1 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,12 @@ 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();
void Start(scoped_refptr<XmppProxy> xmpp_proxy,
scoped_ptr<protocol::TransportFactory> transport_factory);
void Stop(const base::Closure& shutdown_task);
Wez 2012/05/26 00:18:40 Add a comment clarifying the threading properties
Sergey Ulanov 2012/05/30 01:07:07 Done.
- void ClientDone();
// Return the stats recorded by this client.
ChromotingStats* GetStats();
@@ -78,7 +74,7 @@ class ChromotingClient : public protocol::ConnectionToHost::HostEventCallback,
base::Closure done;
};
- base::MessageLoopProxy* message_loop();
+ base::SingleThreadTaskRunner* task_runner();
// Initializes connection.
void Initialize();

Powered by Google App Engine
This is Rietveld 408576698