[Chromoting] Add plumbing to carry clipboard events from the host to the client.

remoting/protocol/client_control_dispatcher.cc

 // 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.
 
 #include "remoting/protocol/client_control_dispatcher.h"
 
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/message_loop_proxy.h"
 #include "net/socket/stream_socket.h"
 #include "remoting/base/constants.h"
 #include "remoting/proto/control.pb.h"
 #include "remoting/proto/internal.pb.h"
 #include "remoting/protocol/buffered_socket_writer.h"
 #include "remoting/protocol/client_stub.h"
 #include "remoting/protocol/util.h"
 
 namespace remoting {
 namespace protocol {
 
 ClientControlDispatcher::ClientControlDispatcher()
     : ChannelDispatcherBase(kControlChannelName),
       client_stub_(NULL),
+      clipboard_stub_(NULL),
       writer_(new BufferedSocketWriter(base::MessageLoopProxy::current())) {
 }
 
 ClientControlDispatcher::~ClientControlDispatcher() {
   writer_->Close();
 }
 
 void ClientControlDispatcher::OnInitialized() {
   // TODO(garykac): Set write failed callback.
   writer_->Init(channel(), BufferedSocketWriter::WriteFailedCallback());
   reader_.Init(channel(), base::Bind(
       &ClientControlDispatcher::OnMessageReceived, base::Unretained(this)));
 }
 
 void ClientControlDispatcher::InjectClipboardEvent(
     const ClipboardEvent& event) {
   ControlMessage message;
   message.mutable_clipboard_event()->CopyFrom(event);
   writer_->Write(SerializeAndFrameMessage(message), base::Closure());
 }
 
 void ClientControlDispatcher::OnMessageReceived(
     scoped_ptr<ControlMessage> message, const base::Closure& done_task) {
   DCHECK(client_stub_);
+  DCHECK(clipboard_stub_);

[Wez, 2012/03/30 22:20:15]

not for this CL: It feels like these checks belong in OnInitialized()?

[simonmorris, 2012/03/30 22:36:07]

Could be, though the stubs are used here.

[Wez, 2012/03/31 00:38:46]

True.  Checking in OnMessageReceived() risks us missing cases where the stubs
weren't set, since we won't realise until a message is received, though. :)

   base::ScopedClosureRunner done_runner(done_task);
-  LOG(WARNING) << "Unknown control message received.";
+
+  if (message->has_clipboard_event()) {
+    clipboard_stub_->InjectClipboardEvent(message->clipboard_event());
+  } else {
+    LOG(WARNING) << "Unknown control message received.";
+  }
 }
 
 }  // namespace protocol
 }  // namespace remoting