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

remoting/protocol/clipboard_thread_proxy.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/clipboard_thread_proxy.h"
+
+#include "base/bind.h"
+#include "base/message_loop.h"
+#include "remoting/proto/event.pb.h"
+
+namespace remoting {
+namespace protocol {
+
+ClipboardThreadProxy::ClipboardThreadProxy(
+    const base::WeakPtr<ClipboardStub>& clipboard_stub,
+    scoped_refptr<base::MessageLoopProxy> clipboard_stub_message_loop)
+    : clipboard_stub_(clipboard_stub),
+      clipboard_stub_message_loop_(clipboard_stub_message_loop) {
+}
+
+void ClipboardThreadProxy::InjectClipboardEvent(const ClipboardEvent& event) {
+  if (!clipboard_stub_message_loop_->BelongsToCurrentThread()) {

[Wez, 2012/05/30 23:58:02]

Since you're no longer trampolining the same method to a different thread, you
can dispense with the thread check, and with the InjectClipboardEventStatic call
below.

[simonmorris, 2012/05/31 00:21:18]

Done.

+    clipboard_stub_message_loop_->PostTask(FROM_HERE, base::Bind(
+        &ClipboardThreadProxy::InjectClipboardEventStatic,
+        clipboard_stub_,
+        event));
+    return;
+  }
+
+  InjectClipboardEventStatic(clipboard_stub_, event);
+}
+
+void ClipboardThreadProxy::InjectClipboardEventStatic(
+    const base::WeakPtr<ClipboardStub>& clipboard_stub,
+    const ClipboardEvent& event) {
+  if (clipboard_stub) {

[Wez, 2012/05/30 23:58:02]

nit: clipboard_stub.get(), for clarity?

[simonmorris, 2012/05/31 00:21:18]

Done.

+    clipboard_stub->InjectClipboardEvent(event);
+  }
+}
+
+ClipboardThreadProxy::~ClipboardThreadProxy() {

[Wez, 2012/05/30 23:58:02]

nit: Move this above ctor to match the header.

[simonmorris, 2012/05/31 00:21:18]

Done.

+}
+
+}  // namespace protocol
+}  // namespace remoting