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

Unified Diff: remoting/host/clipboard_linux.cc

Issue 10909133: Implement clipboard for Chromoting Linux hosts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix nits. Created 8 years, 3 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
« no previous file with comments | « remoting/host/chromoting_host_context.cc ('k') | remoting/host/event_executor_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/clipboard_linux.cc
diff --git a/remoting/host/clipboard_linux.cc b/remoting/host/clipboard_linux.cc
index 6a1729167fccf9aeea4617c719d28d037e3290a2..1f7468947f359984f361f4dce1d181ff33564751 100644
--- a/remoting/host/clipboard_linux.cc
+++ b/remoting/host/clipboard_linux.cc
@@ -4,37 +4,119 @@
#include "remoting/host/clipboard.h"
+#include <X11/Xlib.h>
+
+#include "base/bind.h"
#include "base/logging.h"
+#include "base/message_loop.h"
+#include "remoting/host/linux/x_server_clipboard.h"
+#include "remoting/proto/event.pb.h"
+#include "remoting/protocol/clipboard_stub.h"
namespace remoting {
-class ClipboardLinux : public Clipboard {
+// This code is expected to be called on the desktop thread only.
+class ClipboardLinux : public Clipboard,
+ public MessageLoopForIO::Watcher {
public:
ClipboardLinux();
+ virtual ~ClipboardLinux();
- // Must be called on the UI thread.
+ // Clipboard interface.
virtual void Start(
scoped_ptr<protocol::ClipboardStub> client_clipboard) OVERRIDE;
virtual void InjectClipboardEvent(
const protocol::ClipboardEvent& event) OVERRIDE;
virtual void Stop() OVERRIDE;
+ // MessageLoopForIO::Watcher interface.
+ virtual void OnFileCanReadWithoutBlocking(int fd) OVERRIDE;
+ virtual void OnFileCanWriteWithoutBlocking(int fd) OVERRIDE;
+
private:
+ void OnClipboardChanged(const std::string& mime_type,
+ const std::string& data);
+ void PumpXEvents();
+
+ scoped_ptr<protocol::ClipboardStub> client_clipboard_;
+
+ XServerClipboard x_server_clipboard_;
+ Display* display_;
+
+ MessageLoopForIO::FileDescriptorWatcher x_connection_watcher_;
+
DISALLOW_COPY_AND_ASSIGN(ClipboardLinux);
};
+ClipboardLinux::ClipboardLinux()
+ : display_(NULL) {
+}
+
+ClipboardLinux::~ClipboardLinux() {
+ Stop();
+}
+
void ClipboardLinux::Start(
scoped_ptr<protocol::ClipboardStub> client_clipboard) {
- NOTIMPLEMENTED();
+ // TODO(lambroslambrou): Share the X connection with EventExecutor.
+ display_ = XOpenDisplay(NULL);
+ if (!display_) {
+ LOG(ERROR) << "Couldn't open X display";
+ return;
+ }
+ client_clipboard_.swap(client_clipboard);
+
+ x_server_clipboard_.Init(display_,
+ base::Bind(&ClipboardLinux::OnClipboardChanged,
+ base::Unretained(this)));
+
+ MessageLoopForIO::current()->WatchFileDescriptor(
+ ConnectionNumber(display_), true, MessageLoopForIO::WATCH_READ,
+ &x_connection_watcher_, this);
+ PumpXEvents();
}
void ClipboardLinux::InjectClipboardEvent(
const protocol::ClipboardEvent& event) {
- NOTIMPLEMENTED();
+ x_server_clipboard_.SetClipboard(event.mime_type(), event.data());
}
void ClipboardLinux::Stop() {
- NOTIMPLEMENTED();
+ client_clipboard_.reset();
+ x_connection_watcher_.StopWatchingFileDescriptor();
+
+ if (display_) {
+ XCloseDisplay(display_);
+ display_ = NULL;
+ }
+}
+
+void ClipboardLinux::OnFileCanReadWithoutBlocking(int fd) {
+ PumpXEvents();
+}
+
+void ClipboardLinux::OnFileCanWriteWithoutBlocking(int fd) {
+}
+
+void ClipboardLinux::OnClipboardChanged(const std::string& mime_type,
+ const std::string& data) {
+ protocol::ClipboardEvent event;
+ event.set_mime_type(mime_type);
+ event.set_data(data);
+
+ if (client_clipboard_.get()) {
+ client_clipboard_->InjectClipboardEvent(event);
+ }
+}
+
+void ClipboardLinux::PumpXEvents() {
+ DCHECK(display_);
+
+ while (XPending(display_)) {
+ XEvent event;
+ XNextEvent(display_, &event);
+ x_server_clipboard_.ProcessXEvent(&event);
+ }
}
scoped_ptr<Clipboard> Clipboard::Create() {
« no previous file with comments | « remoting/host/chromoting_host_context.cc ('k') | remoting/host/event_executor_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698