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

Unified Diff: remoting/protocol/v1_authenticator.cc

Issue 9150014: Cleanup AuthenticatorFactory implementations. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merged Created 8 years, 11 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/v1_authenticator.h ('k') | remoting/protocol/v1_authenticator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/v1_authenticator.cc
diff --git a/remoting/protocol/v1_authenticator.cc b/remoting/protocol/v1_authenticator.cc
index c39fd278dda2d9b82b43eecf5a58cc4a7ec7e7ce..b0de0ae8e034f2d3a0010c1d9dd9b81e45717b96 100644
--- a/remoting/protocol/v1_authenticator.cc
+++ b/remoting/protocol/v1_authenticator.cc
@@ -86,11 +86,11 @@ V1ClientAuthenticator::CreateChannelAuthenticator() const {
V1HostAuthenticator::V1HostAuthenticator(
const std::string& local_cert,
- const crypto::RSAPrivateKey* local_private_key,
+ const crypto::RSAPrivateKey& local_private_key,
const std::string& shared_secret,
const std::string& remote_jid)
: local_cert_(local_cert),
- local_private_key_(local_private_key->Copy()),
+ local_private_key_(local_private_key.Copy()),
shared_secret_(shared_secret),
remote_jid_(remote_jid),
state_(WAITING_MESSAGE) {
@@ -144,25 +144,5 @@ V1HostAuthenticator::CreateChannelAuthenticator() const {
return result;
};
-V1HostAuthenticatorFactory::V1HostAuthenticatorFactory(
- const std::string& local_cert,
- const crypto::RSAPrivateKey* local_private_key,
- const std::string& shared_secret)
- : local_cert_(local_cert),
- local_private_key_(local_private_key->Copy()),
- shared_secret_(shared_secret) {
- CHECK(local_private_key_.get());
-}
-
-V1HostAuthenticatorFactory::~V1HostAuthenticatorFactory() {
-}
-
-Authenticator* V1HostAuthenticatorFactory::CreateAuthenticator(
- const std::string& remote_jid,
- const buzz::XmlElement* first_message) {
- return new V1HostAuthenticator(local_cert_, local_private_key_.get(),
- shared_secret_, remote_jid);
-}
-
} // namespace remoting
} // namespace protocol
« no previous file with comments | « remoting/protocol/v1_authenticator.h ('k') | remoting/protocol/v1_authenticator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698