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

Unified Diff: remoting/host/daemon_process_unittest.cc

Issue 12678008: Reworked the plumbing required to pass the client resolution to the desktop resizer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback Created 7 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/host/daemon_process.cc ('k') | remoting/host/daemon_process_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/daemon_process_unittest.cc
diff --git a/remoting/host/daemon_process_unittest.cc b/remoting/host/daemon_process_unittest.cc
index 033a49fd48e460b2ac36b6f93b7feaf91d5289ef..6856a0aadb1fcc4f2cf720e4534c6b93c0213b08 100644
--- a/remoting/host/daemon_process_unittest.cc
+++ b/remoting/host/daemon_process_unittest.cc
@@ -56,7 +56,7 @@ class MockDaemonProcess : public DaemonProcess {
virtual scoped_ptr<DesktopSession> DoCreateDesktopSession(
int terminal_id,
- const DesktopSessionParams& params,
+ const ScreenResolution& resolution,
bool virtual_terminal) OVERRIDE;
virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
@@ -95,7 +95,7 @@ MockDaemonProcess::~MockDaemonProcess() {
scoped_ptr<DesktopSession> MockDaemonProcess::DoCreateDesktopSession(
int terminal_id,
- const DesktopSessionParams& params,
+ const ScreenResolution& resolution,
bool virtual_terminal) {
return scoped_ptr<DesktopSession>(DoCreateDesktopSessionPtr(terminal_id));
}
@@ -229,10 +229,10 @@ TEST_F(DaemonProcessTest, OpenClose) {
StartDaemonProcess();
int id = terminal_id_++;
- DesktopSessionParams params;
+ ScreenResolution resolution;
EXPECT_TRUE(daemon_process_->OnMessageReceived(
- ChromotingNetworkHostMsg_ConnectTerminal(id, params, false)));
+ ChromotingNetworkHostMsg_ConnectTerminal(id, resolution, false)));
EXPECT_EQ(1u, desktop_sessions().size());
EXPECT_EQ(id, desktop_sessions().front()->id());
@@ -250,10 +250,10 @@ TEST_F(DaemonProcessTest, CallCloseDesktopSession) {
StartDaemonProcess();
int id = terminal_id_++;
- DesktopSessionParams params;
+ ScreenResolution resolution;
EXPECT_TRUE(daemon_process_->OnMessageReceived(
- ChromotingNetworkHostMsg_ConnectTerminal(id, params, false)));
+ ChromotingNetworkHostMsg_ConnectTerminal(id, resolution, false)));
EXPECT_EQ(1u, desktop_sessions().size());
EXPECT_EQ(id, desktop_sessions().front()->id());
@@ -274,10 +274,10 @@ TEST_F(DaemonProcessTest, DoubleDisconnectTerminal) {
StartDaemonProcess();
int id = terminal_id_++;
- DesktopSessionParams params;
+ ScreenResolution resolution;
EXPECT_TRUE(daemon_process_->OnMessageReceived(
- ChromotingNetworkHostMsg_ConnectTerminal(id, params, false)));
+ ChromotingNetworkHostMsg_ConnectTerminal(id, resolution, false)));
EXPECT_EQ(1u, desktop_sessions().size());
EXPECT_EQ(id, desktop_sessions().front()->id());
@@ -326,15 +326,15 @@ TEST_F(DaemonProcessTest, InvalidConnectTerminal) {
StartDaemonProcess();
int id = terminal_id_++;
- DesktopSessionParams params;
+ ScreenResolution resolution;
EXPECT_TRUE(daemon_process_->OnMessageReceived(
- ChromotingNetworkHostMsg_ConnectTerminal(id, params, false)));
+ ChromotingNetworkHostMsg_ConnectTerminal(id, resolution, false)));
EXPECT_EQ(1u, desktop_sessions().size());
EXPECT_EQ(id, desktop_sessions().front()->id());
EXPECT_TRUE(daemon_process_->OnMessageReceived(
- ChromotingNetworkHostMsg_ConnectTerminal(id, params, false)));
+ ChromotingNetworkHostMsg_ConnectTerminal(id, resolution, false)));
EXPECT_TRUE(desktop_sessions().empty());
EXPECT_EQ(0, terminal_id_);
}
« no previous file with comments | « remoting/host/daemon_process.cc ('k') | remoting/host/daemon_process_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698