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

Unified Diff: remoting/host/event_executor_linux.cc

Issue 5068001: Move move classes to the remoting::protocol namespace. Minor cleanups. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more fixes for windows Created 10 years, 1 month 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/host/event_executor_linux.cc
diff --git a/remoting/host/event_executor_linux.cc b/remoting/host/event_executor_linux.cc
index f540d4455d8f6b138464e952b5cf0383441ea924..5096fac01583d754ae95b5a1efcd09f5bd3b2711 100644
--- a/remoting/host/event_executor_linux.cc
+++ b/remoting/host/event_executor_linux.cc
@@ -15,18 +15,22 @@
namespace remoting {
-static int MouseButtonToX11ButtonNumber(MouseButton button) {
+using protocol::MouseEvent;
+using protocol::KeyEvent;
+
+static int MouseButtonToX11ButtonNumber(
+ protocol::MouseEvent::MouseButton button) {
switch (button) {
- case MouseButtonLeft:
+ case MouseEvent::BUTTON_LEFT:
return 1;
- case MouseButtonRight:
+ case MouseEvent::BUTTON_RIGHT:
return 2;
- case MouseButtonMiddle:
+ case MouseEvent::BUTTON_MIDDLE:
return 3;
- case MouseButtonUndefined:
+ case MouseEvent::BUTTON_UNDEFINED:
default:
return -1;
}
@@ -202,6 +206,7 @@ class EventExecutorLinuxPimpl {
~EventExecutorLinuxPimpl();
bool Init(); // TODO(ajwong): Do we really want this to be synchronous?
+
void HandleMouse(const MouseEvent* message);
void HandleKey(const KeyEvent* key_event);

Powered by Google App Engine
This is Rietveld 408576698