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

Unified Diff: remoting/test/remote_host_info.cc

Issue 1237093004: Support for connecting to localhost on the chromoting test driver. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made changes to comments and variable name. Created 5 years, 5 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
Index: remoting/test/remote_host_info.cc
diff --git a/remoting/test/remote_host_info.cc b/remoting/test/remote_host_info.cc
index 62883902ce4bf144a884d1caabdbc465e774e6d0..75091d9cb3b7ed444a417b745ab392988b42a881 100644
--- a/remoting/test/remote_host_info.cc
+++ b/remoting/test/remote_host_info.cc
@@ -2,10 +2,19 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+
joedow 2015/07/20 16:39:53 please remove newline. Didn't I make this comment
tonychun 2015/07/20 18:13:01 This is magic. (I know I removed it, but CLs don't
+#include "base/strings/string_split.h"
joedow 2015/07/20 16:39:53 string_split isn't needed, please remove. I seem
tonychun 2015/07/20 18:13:01 This is magic.
+#include "remoting/protocol/authentication_method.h"
#include "remoting/test/remote_host_info.h"
#include "base/logging.h"
+namespace {
+const char kAppRemotingCapabilities[] =
+ "rateLimitResizeRequests desktopShape sendInitialResolution googleDrive";
+const char kFakeHostPublicKey[] = "FAKE_HOST_PUBLIC_KEY";
+}
+
namespace remoting {
namespace test {
@@ -17,7 +26,7 @@ RemoteHostInfo::~RemoteHostInfo() {
}
bool RemoteHostInfo::IsReadyForConnection() const {
- return (remote_host_status == kRemoteHostStatusReady);
+ return remote_host_status == kRemoteHostStatusReady;
}
void RemoteHostInfo::SetRemoteHostStatusFromString(
@@ -32,5 +41,24 @@ void RemoteHostInfo::SetRemoteHostStatusFromString(
}
}
+ConnectionSetupInfo RemoteHostInfo::GenerateConnectionSetupInfo(
+ const std::string& access_token,
+ const std::string& user_name) const {
+ ConnectionSetupInfo connection_setup_info;
+ connection_setup_info.access_token = access_token;
+ connection_setup_info.authorization_code = authorization_code;
+ connection_setup_info.capabilities = kAppRemotingCapabilities;
+ connection_setup_info.host_id = host_id;
+ connection_setup_info.host_jid = host_jid;
+ connection_setup_info.public_key = kFakeHostPublicKey;
+ connection_setup_info.shared_secret = shared_secret;
+ connection_setup_info.user_name = user_name;
+
+ connection_setup_info.auth_methods.push_back(
+ protocol::AuthenticationMethod::ThirdParty());
+
+ return connection_setup_info;
+}
+
} // namespace test
} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698