diff options
author | joedow <joedow@chromium.org> | 2016-02-16 21:30:02 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-17 05:30:58 +0000 |
commit | 709e40ef1af725648b40fe28e06801b3a56f7e14 (patch) | |
tree | 93c8d61e2c7bab3093b03c1b555404c10d1487be /remoting/host/client_session_unittest.cc | |
parent | cf186a3887a562701a0f52b216a4b05ac136d0f0 (diff) | |
download | chromium_src-709e40ef1af725648b40fe28e06801b3a56f7e14.zip chromium_src-709e40ef1af725648b40fe28e06801b3a56f7e14.tar.gz chromium_src-709e40ef1af725648b40fe28e06801b3a56f7e14.tar.bz2 |
Refactoring platform-agnostic Gnubby Auth Handler logic into a Host Extension.
This change breaks the logic in the GnubbyAuthHandlerLinux class into
platform-specific and platform-agnostic chunks. The platform-specific
logic has remained in the GnubbyAuthHandlerLinux class whereas the
platform-agnostic code was moved into a new HostExtension class
(GnubbyHostExtensionSession). This allows us to add a Gnubby host
extension if it is supported by policy and remove the logic and plumbing
for gnubby message handling from ClientSession and the many
DesktopEnvironment classes.
BUG=584463
Review URL: https://codereview.chromium.org/1689003002
Cr-Commit-Position: refs/heads/master@{#375797}
Diffstat (limited to 'remoting/host/client_session_unittest.cc')
-rw-r--r-- | remoting/host/client_session_unittest.cc | 31 |
1 files changed, 0 insertions, 31 deletions
diff --git a/remoting/host/client_session_unittest.cc b/remoting/host/client_session_unittest.cc index 3e7eacb..bd7c01a 100644 --- a/remoting/host/client_session_unittest.cc +++ b/remoting/host/client_session_unittest.cc @@ -373,37 +373,6 @@ TEST_F(ClientSessionTest, ClampMouseEvents) { } } -TEST_F(ClientSessionTest, NoGnubbyAuth) { - CreateClientSession(); - ConnectClientSession(); - NotifyVideoSize(); - - protocol::ExtensionMessage message; - message.set_type("gnubby-auth"); - message.set_data("test"); - - // Host should ignore gnubby messages when gnubby is disabled. - client_session_->DeliverClientMessage(message); -} - -TEST_F(ClientSessionTest, EnableGnubbyAuth) { - CreateClientSession(); - ConnectClientSession(); - NotifyVideoSize(); - - // Lifetime controlled by object under test. - MockGnubbyAuthHandler* gnubby_auth_handler = new MockGnubbyAuthHandler(); - client_session_->SetGnubbyAuthHandlerForTesting(gnubby_auth_handler); - - // Host should ignore gnubby messages when gnubby is disabled. - protocol::ExtensionMessage message; - message.set_type("gnubby-auth"); - message.set_data("test"); - - EXPECT_CALL(*gnubby_auth_handler, DeliverClientMessage(_)).Times(1); - client_session_->DeliverClientMessage(message); -} - // Verifies that the client's video pipeline can be reset mid-session. TEST_F(ClientSessionTest, ResetVideoPipeline) { CreateClientSession(); |