[Chromoting] Let the Windows IT2Me host send clipboard events to the client.

remoting/host/client_session.cc

Index: remoting/host/client_session.cc
diff --git a/remoting/host/client_session.cc b/remoting/host/client_session.cc
index 86a6753d3e55d57c65c506065572b40d4e6b8190..926d66b59bdf59403b3468499a37b0a6c7da8bc8 100644
--- a/remoting/host/client_session.cc
+++ b/remoting/host/client_session.cc
@@ -27,6 +27,7 @@ ClientSession::ClientSession(
       input_tracker_(host_event_stub_),
       remote_input_filter_(&input_tracker_),
       mouse_input_filter_(&remote_input_filter_),
+      client_clipboard_(clipboard_echo_filter_.client_filter()),
       capturer_(capturer) {
   connection_->SetEventHandler(this);
 
@@ -164,4 +165,13 @@ void ClientSession::SetDisableInputs(bool disable_inputs) {
   }
 }
 
+scoped_refptr<protocol::ClipboardProxy> ClientSession::ClientClipboard() {
+  DCHECK(CalledOnValidThread());
+
+  scoped_refptr<protocol::ClipboardProxy> client_clipboard =
+      new protocol::ClipboardProxy(base::MessageLoopProxy::current());
+  client_clipboard->Attach(client_clipboard_.AsWeakPtr());
+  return client_clipboard;

[Wez, 2012/05/30 01:15:43]

Since you're creating a new ClipboardProxy every time ClientClipboard gets
called, there's no real benefit in it being refcounted.

[simonmorris, 2012/05/30 16:54:06]

I want ClipboardThreadProxy to be refcounted so that it can safely post messages
to itself on another thread.

[Wez, 2012/05/30 17:07:47]

You can do that if instead of storing a WeakPtr to the target ClipboardStub in
the ClipboardThreadProxy, you store a WeakPtr to the ClipboardThreadProxy
itself.  Any tasks it posts to itself will be dropped by the task queue if they
are processed after the proxy('s WeakPtrFactory) has been deleted.

(The caller must then guarantee that the ClipboardStub outlive the
ClipboardThreadProxy, but that should be easy enough to arrange.)

[simonmorris, 2012/05/30 23:18:26]

As discussed offline, ClipboardThreadProxy does need to outlive ClipboardStub,
but WeakPtr can be used instead of ref-counting anyway.

+}
+
 }  // namespace remoting