Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(582)

Unified Diff: remoting/protocol/clipboard_thread_proxy.cc

Issue 10413060: [Chromoting] Let the Windows IT2Me host send clipboard events to the client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't ref-count ClipboardThreadProxy. Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/protocol/clipboard_thread_proxy.cc
diff --git a/remoting/protocol/clipboard_thread_proxy.cc b/remoting/protocol/clipboard_thread_proxy.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2c39543c34eae36fa5806ab8a39487bab9961189
--- /dev/null
+++ b/remoting/protocol/clipboard_thread_proxy.cc
@@ -0,0 +1,45 @@
+// 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 metho
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

Powered by Google App Engine
This is Rietveld 408576698