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

Unified Diff: remoting/protocol/key_event_tracker_unittest.cc

Issue 9465035: Move ClientSession's input logic into separate components. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 8 years, 9 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/protocol/key_event_tracker.cc ('k') | remoting/remoting.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/key_event_tracker_unittest.cc
diff --git a/remoting/protocol/key_event_tracker_unittest.cc b/remoting/protocol/key_event_tracker_unittest.cc
deleted file mode 100644
index 358057d6630e83a2c2f63e46806cd353830bec01..0000000000000000000000000000000000000000
--- a/remoting/protocol/key_event_tracker_unittest.cc
+++ /dev/null
@@ -1,84 +0,0 @@
-// 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"
-
-using ::testing::_;
-using ::testing::ExpectationSet;
-using ::testing::InSequence;
-
-namespace remoting {
-
-MATCHER_P2(EqualsKeyEvent, keycode, pressed, "") {
- return arg.keycode() == keycode && arg.pressed() == pressed;
-}
-
-class MockInputStub : public protocol::InputStub {
- public:
- MockInputStub() {}
-
- MOCK_METHOD1(InjectKeyEvent, void(const protocol::KeyEvent&));
- MOCK_METHOD1(InjectMouseEvent, void(const protocol::MouseEvent&));
- private:
- DISALLOW_COPY_AND_ASSIGN(MockInputStub);
-};
-
-static protocol::KeyEvent KeyEvent(int keycode, bool pressed) {
- protocol::KeyEvent event;
- event.set_keycode(keycode);
- 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) {
- MockInputStub mock_stub;
- protocol::KeyEventTracker 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);
-}
-
-// Verify that keys that were left pressed get released.
-TEST(KeyEventTrackerTest, ReleaseAllKeys) {
- MockInputStub mock_stub;
- protocol::KeyEventTracker 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();
-}
-
-} // namespace remoting
« no previous file with comments | « remoting/protocol/key_event_tracker.cc ('k') | remoting/remoting.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698