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

Unified Diff: remoting/host/event_executor_win.cc

Issue 3175028: Add mouse event support to Chromoting client (Pepper and X11). (Closed)
Patch Set: Remove win float/int conversion Created 10 years, 4 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/client/x11_view.cc ('k') | remoting/remoting.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/event_executor_win.cc
diff --git a/remoting/host/event_executor_win.cc b/remoting/host/event_executor_win.cc
index 258389e1ff342e6645058259931d89096ee8dc19..1bf0ff04df607f0a8d688d155ccbdfe61e7643b1 100644
--- a/remoting/host/event_executor_win.cc
+++ b/remoting/host/event_executor_win.cc
@@ -357,36 +357,15 @@ void EventExecutorWin::HandleInputEvents(ClientMessageList* messages) {
for (size_t i = 0; i < messages->size(); ++i) {
ChromotingClientMessage* msg = (*messages)[i];
if (msg->has_mouse_set_position_event()) {
- mouse_event(MOUSEEVENTF_MOVE | MOUSEEVENTF_ABSOLUTE,
- static_cast<int>((msg->mouse_set_position_event().x() * 65535)),
- static_cast<int>((msg->mouse_set_position_event().y() * 65535)),
- 0, 0);
+ // TODO(garykac) Updated Windows host mouse support in following cl.
} else if (msg->has_mouse_move_event()) {
- mouse_event(MOUSEEVENTF_MOVE,
- msg->mouse_move_event().offset_x(),
- msg->mouse_move_event().offset_y(), 0, 0);
+ // TODO(garykac) Updated Windows host mouse support in following cl.
} else if (msg->has_mouse_wheel_event()) {
- // TODO(hclam): Handle wheel events.
+ // TODO(garykac) Updated Windows host wheel support in following cl.
} else if (msg->has_mouse_down_event()) {
- if (msg->mouse_down_event().button() ==
- MouseDownEvent::LEFT) {
- mouse_event(MOUSEEVENTF_LEFTDOWN, 0, 0, 0, 0);
- } else if (msg->mouse_down_event().button() ==
- MouseDownEvent::RIGHT) {
- mouse_event(MOUSEEVENTF_RIGHTDOWN, 0, 0, 0, 0);
- } else {
- // TODO(hclam): Handle other buttons.
- }
+ // TODO(garykac) Updated Windows host mouse support in following cl.
} else if (msg->has_mouse_up_event()) {
- if (msg->mouse_up_event().button() ==
- MouseUpEvent::LEFT) {
- mouse_event(MOUSEEVENTF_LEFTUP, 0, 0, 0, 0);
- } else if (msg->mouse_up_event().button() ==
- MouseUpEvent::RIGHT) {
- mouse_event(MOUSEEVENTF_RIGHTUP, 0, 0, 0, 0);
- } else {
- // TODO(hclam): Handle other buttons.
- }
+ // TODO(garykac) Updated Windows host mouse support in following cl.
} else if (msg->has_key_event()) {
base::KeyboardCode key_code =
WindowsKeyCodeForPosixKeyCode(msg->key_event().key());
« no previous file with comments | « remoting/client/x11_view.cc ('k') | remoting/remoting.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698