Use scoped_ptr<>.Pass() to pass ownership in the remoting protocol code.

remoting/protocol/v2_authenticator.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/v2_authenticator.h"
 
 #include "base/base64.h"
 #include "base/logging.h"
 #include "crypto/rsa_private_key.h"
 #include "remoting/base/constants.h"
@@ skipping 111 matching lines @@
       case P224EncryptedKeyExchange::kResultSuccess:
         auth_key_ = key_exchange_impl_.GetKey();
         state_ = ACCEPTED;
         return;
     }
   }
 
   state_ = MESSAGE_READY;
 }
 
-buzz::XmlElement* V2Authenticator::GetNextMessage() {
+scoped_ptr<buzz::XmlElement> V2Authenticator::GetNextMessage() {
   DCHECK_EQ(state(), MESSAGE_READY);
 
-  buzz::XmlElement* message = CreateEmptyAuthenticatorMessage();
+  scoped_ptr<buzz::XmlElement> message = CreateEmptyAuthenticatorMessage();
 
   DCHECK(!pending_messages_.empty());
   while (!pending_messages_.empty()) {
     const std::string& spake_message = pending_messages_.front();
     std::string base64_message;
     if (!base::Base64Encode(spake_message, &base64_message)) {
       LOG(DFATAL) << "Cannot perform base64 encode on certificate";
       continue;
     }
 
@@ skipping 10 matching lines @@
     if (!base::Base64Encode(local_cert_, &base64_cert)) {
       LOG(DFATAL) << "Cannot perform base64 encode on certificate";
     }
     certificate_tag->SetBodyText(base64_cert);
     message->AddElement(certificate_tag);
   }
 
   if (state_ != ACCEPTED) {
     state_ = WAITING_MESSAGE;
   }
-  return message;
+  return message.Pass();
 }
 
-ChannelAuthenticator* V2Authenticator::CreateChannelAuthenticator() const {
+scoped_ptr<ChannelAuthenticator>
+V2Authenticator::CreateChannelAuthenticator() const {
   DCHECK_EQ(state(), ACCEPTED);
   CHECK(!auth_key_.empty());
 
   if (is_host_side()) {
-    return SslHmacChannelAuthenticator::CreateForHost(
-        local_cert_, local_private_key_.get(), auth_key_);
+    return scoped_ptr<ChannelAuthenticator>(
+        SslHmacChannelAuthenticator::CreateForHost(
+            local_cert_, local_private_key_.get(), auth_key_).release());

[Wez, 2012/01/19 03:18:17]

.Pass()?

[Sergey Ulanov, 2012/01/19 20:11:42]

Done.

   } else {
-    return SslHmacChannelAuthenticator::CreateForClient(
-        remote_cert_, auth_key_);
+    return scoped_ptr<ChannelAuthenticator>(
+        SslHmacChannelAuthenticator::CreateForClient(
+            remote_cert_, auth_key_).release());

[Wez, 2012/01/19 03:18:17]

.Pass()?

[Sergey Ulanov, 2012/01/19 20:11:42]

Done.

   }
 }
 
 bool V2Authenticator::is_host_side() const {
   return local_private_key_.get() != NULL;
 }
 
 }  // namespace protocol
 }  // namespace remoting