[Chromoting] Add a filter that will stop the host sending unnecessary clipboard events to the clien…

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 7db5498373ec3b3098c19f3a2db58323c86a33c9..4da98bd3cf5af9cad74e3a3531fc5c5391b60137 100644
--- a/remoting/host/client_session.cc
+++ b/remoting/host/client_session.cc
@@ -10,6 +10,7 @@
 #include "remoting/host/capturer.h"
 #include "remoting/proto/control.pb.h"
 #include "remoting/proto/event.pb.h"
+#include "remoting/protocol/client_stub.h"
 
 namespace remoting {
 
@@ -34,6 +35,8 @@ ClientSession::ClientSession(
   connection_->set_clipboard_stub(this);
   connection_->set_host_stub(this);
   connection_->set_input_stub(this);
+  clipboard_duplicate_filter_.set_to_host(host_event_stub_);
+  clipboard_duplicate_filter_.set_to_client(connection_->client_stub());

[Wez, 2012/05/16 20:51:43]

Is client_stub() actually valid at this point, or do we need to wait and set it
only when we know we're connected?

[simonmorris, 2012/05/16 22:35:25]

I've moved this line to OnConnectionAuthenticated(), as we shouldn't send
anything to the client before then anyway.

 }
 
 ClientSession::~ClientSession() {
@@ -50,7 +53,7 @@ void ClientSession::InjectClipboardEvent(
   if (disable_input_filter_.input_stub() == NULL)
     return;
 
-  host_event_stub_->InjectClipboardEvent(event);
+  clipboard_duplicate_filter_.get_filter_to_host()->InjectClipboardEvent(event);
 }
 
 void ClientSession::InjectKeyEvent(const protocol::KeyEvent& event) {