Move ClientSession's input logic into separate components.

remoting/protocol/input_event_tracker_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/key_event_tracker.h"
 
 #include "remoting/proto/event.pb.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 23 matching lines @@
   event.set_pressed(pressed);
   return event;
 }
 
 static void PressAndRelease(protocol::InputStub* input_stub, int keycode) {
   input_stub->InjectKeyEvent(KeyEvent(keycode, true));
   input_stub->InjectKeyEvent(KeyEvent(keycode, false));
 }
 
 // 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;
 
     EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(1, true)));
     EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(1, false)));
     EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(2, true)));
     EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(2, false)));
   }
   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;
 
   {
     InSequence s;
 
     injects += EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(3, true)));
     injects += EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(1, true)));
     injects += EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(1, false)));
     injects += EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(2, true)));
     injects += EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(2, false)));
   }
 
   EXPECT_CALL(mock_stub, InjectKeyEvent(EqualsKeyEvent(3, false)))
       .After(injects);
 
   key_tracker.InjectKeyEvent(KeyEvent(3, true));
   PressAndRelease(&key_tracker, 1);
   PressAndRelease(&key_tracker, 2);
-  key_tracker.ReleaseAllKeys();
+  key_tracker.ReleaseAll();
 }
 
 }  // namespace remoting