Move ClientSession's input logic into separate components.

remoting/protocol/input_event_tracker_unittest.cc

Index: remoting/protocol/input_event_tracker_unittest.cc
diff --git a/remoting/protocol/key_event_tracker_unittest.cc b/remoting/protocol/input_event_tracker_unittest.cc
similarity index 90%
rename from remoting/protocol/key_event_tracker_unittest.cc
rename to remoting/protocol/input_event_tracker_unittest.cc
index 358057d6630e83a2c2f63e46806cd353830bec01..20ae9e8036248f8100419866e65ecfa38ae0b6c9 100644
--- a/remoting/protocol/key_event_tracker_unittest.cc
+++ b/remoting/protocol/input_event_tracker_unittest.cc
@@ -41,9 +41,9 @@ static void PressAndRelease(protocol::InputStub* input_stub, int keycode) {
 }
 
 // Verify that keys that were pressed and released aren't re-released.
-TEST(KeyEventTrackerTest, NothingToRelease) {
+TEST(InputEventTrackerTest, NothingToRelease) {

[Jamie, 2012/03/15 18:38:56]

Since this is now testing a class that deals with mouse events as well as
keyboard, can you add some tests for that? Follow-up CL is fine.

[Wez, 2012/03/15 22:25:29]

Done.

   MockInputStub mock_stub;
-  protocol::KeyEventTracker key_tracker(&mock_stub);
+  protocol::InputEventTracker key_tracker(&mock_stub);
   {
     InSequence s;
 
@@ -54,12 +54,13 @@ TEST(KeyEventTrackerTest, NothingToRelease) {
   }
   PressAndRelease(&key_tracker, 1);
   PressAndRelease(&key_tracker, 2);
+  key_tracker.ReleaseAll();
 }
 
 // Verify that keys that were left pressed get released.
-TEST(KeyEventTrackerTest, ReleaseAllKeys) {
+TEST(InputEventTrackerTest, ReleaseAllKeys) {
   MockInputStub mock_stub;
-  protocol::KeyEventTracker key_tracker(&mock_stub);
+  protocol::InputEventTracker key_tracker(&mock_stub);
   ExpectationSet injects;
 
   {
@@ -78,7 +79,7 @@ TEST(KeyEventTrackerTest, ReleaseAllKeys) {
   key_tracker.InjectKeyEvent(KeyEvent(3, true));
   PressAndRelease(&key_tracker, 1);
   PressAndRelease(&key_tracker, 2);
-  key_tracker.ReleaseAllKeys();
+  key_tracker.ReleaseAll();
 }
 
 }  // namespace remoting