Refactor InputStub interface to pass events by value (const reference).

This will make it much easier to inject modified key/mouse events, for example,
restricting mouse-coordinates to the screen area.

BUG=None
TEST=Manual


Review URL: http://codereview.chromium.org/7803004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100939 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 15 deletions

diff --git a/remoting/client/input_handler.cc b/remoting/client/input_handler.cc
index 690ce04..7b48d89 100644
--- a/remoting/client/input_handler.cc
+++ b/remoting/client/input_handler.cc
@@ -34,22 +34,20 @@ void InputHandler::SendKeyEvent(bool press, int keycode) {
       pressed_keys_.erase(keycode);
     }
 
-    KeyEvent* event = new KeyEvent();
-    event->set_keycode(keycode);
-    event->set_pressed(press);
-
-    stub->InjectKeyEvent(event, new DeleteTask<KeyEvent>(event));
+    KeyEvent event;
+    event.set_keycode(keycode);
+    event.set_pressed(press);
+    stub->InjectKeyEvent(event);
   }
 }
 
 void InputHandler::SendMouseMoveEvent(int x, int y) {
   protocol::InputStub* stub = connection_->input_stub();
   if (stub) {
-    MouseEvent* event = new MouseEvent();
-    event->set_x(x);
-    event->set_y(y);
-
-    stub->InjectMouseEvent(event, new DeleteTask<MouseEvent>(event));
+    MouseEvent event;
+    event.set_x(x);
+    event.set_y(y);
+    stub->InjectMouseEvent(event);
   }
 }
 
@@ -57,11 +55,10 @@ void InputHandler::SendMouseButtonEvent(bool button_down,
                                         MouseEvent::MouseButton button) {
   protocol::InputStub* stub = connection_->input_stub();
   if (stub) {
-    MouseEvent* event = new MouseEvent();
-    event->set_button(button);
-    event->set_button_down(button_down);
-
-    stub->InjectMouseEvent(event, new DeleteTask<MouseEvent>(event));
+    MouseEvent event;
+    event.set_button(button);
+    event.set_button_down(button_down);
+    stub->InjectMouseEvent(event);
   }
 }