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

Unified Diff: remoting/protocol/spake2_authenticator.cc

Issue 1864213002: Convert //remoting to use std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Mac IWYU Created 4 years, 8 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/spake2_authenticator.h ('k') | remoting/protocol/spake2_authenticator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/spake2_authenticator.cc
diff --git a/remoting/protocol/spake2_authenticator.cc b/remoting/protocol/spake2_authenticator.cc
index ab03e5d8fadf3044f839fcb3b15cf8318b429d41..55bbea3261a7e91c5f520a6b3f26eb478491d986 100644
--- a/remoting/protocol/spake2_authenticator.cc
+++ b/remoting/protocol/spake2_authenticator.cc
@@ -8,6 +8,7 @@
#include "base/base64.h"
#include "base/logging.h"
+#include "base/memory/ptr_util.h"
#include "base/sys_byteorder.h"
#include "crypto/hmac.h"
#include "crypto/secure_util.h"
@@ -40,13 +41,13 @@ const buzz::StaticQName kVerificationHashTag = {kChromotingXmlNamespace,
const buzz::StaticQName kCertificateTag = {kChromotingXmlNamespace,
"certificate"};
-scoped_ptr<buzz::XmlElement> EncodeBinaryValueToXml(
+std::unique_ptr<buzz::XmlElement> EncodeBinaryValueToXml(
const buzz::StaticQName& qname,
const std::string& content) {
std::string content_base64;
base::Base64Encode(content, &content_base64);
- scoped_ptr<buzz::XmlElement> result(new buzz::XmlElement(qname));
+ std::unique_ptr<buzz::XmlElement> result(new buzz::XmlElement(qname));
result->SetBodyText(content_base64);
return result;
}
@@ -80,24 +81,24 @@ std::string PrefixWithLength(const std::string& str) {
} // namespace
// static
-scoped_ptr<Authenticator> Spake2Authenticator::CreateForClient(
+std::unique_ptr<Authenticator> Spake2Authenticator::CreateForClient(
const std::string& local_id,
const std::string& remote_id,
const std::string& shared_secret,
Authenticator::State initial_state) {
- return make_scoped_ptr(new Spake2Authenticator(
+ return base::WrapUnique(new Spake2Authenticator(
local_id, remote_id, shared_secret, false, initial_state));
}
// static
-scoped_ptr<Authenticator> Spake2Authenticator::CreateForHost(
+std::unique_ptr<Authenticator> Spake2Authenticator::CreateForHost(
const std::string& local_id,
const std::string& remote_id,
const std::string& local_cert,
scoped_refptr<RsaKeyPair> key_pair,
const std::string& shared_secret,
Authenticator::State initial_state) {
- scoped_ptr<Spake2Authenticator> result(new Spake2Authenticator(
+ std::unique_ptr<Spake2Authenticator> result(new Spake2Authenticator(
local_id, remote_id, shared_secret, true, initial_state));
result->local_cert_ = local_cert;
result->local_key_pair_ = key_pair;
@@ -247,10 +248,10 @@ void Spake2Authenticator::ProcessMessageInternal(
state_ = MESSAGE_READY;
}
-scoped_ptr<buzz::XmlElement> Spake2Authenticator::GetNextMessage() {
+std::unique_ptr<buzz::XmlElement> Spake2Authenticator::GetNextMessage() {
DCHECK_EQ(state(), MESSAGE_READY);
- scoped_ptr<buzz::XmlElement> message = CreateEmptyAuthenticatorMessage();
+ std::unique_ptr<buzz::XmlElement> message = CreateEmptyAuthenticatorMessage();
if (!spake_message_sent_) {
if (!local_cert_.empty()) {
@@ -282,7 +283,7 @@ const std::string& Spake2Authenticator::GetAuthKey() const {
return auth_key_;
}
-scoped_ptr<ChannelAuthenticator>
+std::unique_ptr<ChannelAuthenticator>
Spake2Authenticator::CreateChannelAuthenticator() const {
DCHECK_EQ(state(), ACCEPTED);
CHECK(!auth_key_.empty());
« no previous file with comments | « remoting/protocol/spake2_authenticator.h ('k') | remoting/protocol/spake2_authenticator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698