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

Unified Diff: remoting/host/client_session.h

Issue 10413060: [Chromoting] Let the Windows IT2Me host send clipboard events to the client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't ref-count ClipboardThreadProxy. 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/host/client_session.h
diff --git a/remoting/host/client_session.h b/remoting/host/client_session.h
index fd2b9a088f51bb6786c8da610aac004ddfd31450..6516fc55bd4d838e67ee79b4f104d3700ee5999b 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -11,6 +11,7 @@
#include "base/threading/non_thread_safe.h"
#include "remoting/host/remote_input_filter.h"
#include "remoting/protocol/clipboard_echo_filter.h"
+#include "remoting/protocol/clipboard_thread_proxy.h"
#include "remoting/protocol/clipboard_stub.h"
#include "remoting/protocol/connection_to_client.h"
#include "remoting/protocol/host_event_stub.h"
@@ -121,6 +122,9 @@ class ClientSession : public protocol::HostEventStub,
// keys or mouse buttons pressed then these will be released.
void SetDisableInputs(bool disable_inputs);
+ // Creates a proxy for sending clipboard events to the client.
+ scoped_ptr<protocol::ClipboardThreadProxy> CreateClipboardProxy();
+
private:
EventHandler* event_handler_;
@@ -153,6 +157,9 @@ class ClientSession : public protocol::HostEventStub,
// back to it.
protocol::ClipboardEchoFilter clipboard_echo_filter_;
+ // Factory for weak pointers to the client clipboard.
Wez 2012/05/30 23:58:02 nit: ... to the client clipboard stub.
simonmorris 2012/05/31 00:21:18 Done.
+ base::WeakPtrFactory<ClipboardStub> client_clipboard_factory_;
Wez 2012/05/30 23:58:02 nit: Consider clarifying that this _must_ appear a
simonmorris 2012/05/31 00:21:18 Done.
+
// Capturer, used to determine current screen size for ensuring injected
// mouse events fall within the screen area.
// TODO(lambroslambrou): Move floor-control logic, and clamping to screen

Powered by Google App Engine
This is Rietveld 408576698