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

remoting/host/client_session.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_HOST_CLIENT_SESSION_H_
 #define REMOTING_HOST_CLIENT_SESSION_H_
 
 #include <list>
 
 #include "base/time.h"
 #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_proxy.h"
 #include "remoting/protocol/clipboard_stub.h"
+#include "remoting/protocol/clipboard_weak_ptr.h"
 #include "remoting/protocol/connection_to_client.h"
 #include "remoting/protocol/host_event_stub.h"
 #include "remoting/protocol/host_stub.h"
 #include "remoting/protocol/input_event_tracker.h"
 #include "remoting/protocol/input_filter.h"
 #include "remoting/protocol/input_stub.h"
 #include "remoting/protocol/mouse_input_filter.h"
 #include "third_party/skia/include/core/SkPoint.h"
 
 namespace remoting {
@@ skipping 89 matching lines @@
 
   // Indicate that local mouse activity has been detected. This causes remote
   // inputs to be ignored for a short time so that the local user will always
   // have the upper hand in 'pointer wars'.
   void LocalMouseMoved(const SkIPoint& new_pos);
 
   // Disable handling of input events from this client. If the client has any
   // keys or mouse buttons pressed then these will be released.
   void SetDisableInputs(bool disable_inputs);
 
+  // Gets a stub for sending clipboard events to the client.
+  scoped_refptr<protocol::ClipboardProxy> ClientClipboard();

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

Since this creates a ClipboardProxy it should have a name like
CreateClipboardProxy, but see comments in implementation.

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

Done.

+
  private:
   EventHandler* event_handler_;
 
   // The connection to the client.
   scoped_ptr<protocol::ConnectionToClient> connection_;
 
   std::string client_jid_;
   bool is_authenticated_;
 
   // The host event stub to which this object delegates. This is the final
@@ skipping 12 matching lines @@
   // Filter used to manage enabling & disabling of client input events.
   protocol::InputFilter disable_input_filter_;
 
   // Filter used to disable inputs when we're not authenticated.
   protocol::InputFilter auth_input_filter_;
 
   // Filter to used to stop clipboard items sent from the client being echoed
   // back to it.
   protocol::ClipboardEchoFilter clipboard_echo_filter_;
 
+  // Clipboard stub used to provide a weak pointer to the client clipboard.
+  protocol::ClipboardWeakPtr client_clipboard_;

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

I think you can achieve the same effect without the ClipboardWeakPtr class by
creating a base::WeakPtrFactory<ClipboardStub> and passing the target
ClipboardStub to the constructor.

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

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
   // area, out of this class (crbug.com/96508).
   Capturer* capturer_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientSession);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CLIENT_SESSION_H_