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

remoting/protocol/jingle_session_unittest.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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/message_loop_proxy.h"
 #include "base/time.h"
 #include "net/socket/socket.h"
 #include "net/socket/stream_socket.h"
@@ skipping 188 matching lines @@
         EXPECT_CALL(client_connection_callback_,
                     OnStateChange(Session::FAILED))
             .Times(1);
       } else {
         EXPECT_CALL(client_connection_callback_,
                     OnStateChange(Session::AUTHENTICATED))
             .Times(1);
       }
     }
 
-    Authenticator* authenticator = new FakeAuthenticator(
-        FakeAuthenticator::CLIENT, auth_round_trips, auth_action, true);
+    scoped_ptr<Authenticator> authenticator(new FakeAuthenticator(
+        FakeAuthenticator::CLIENT, auth_round_trips, auth_action, true));
 
-    client_session_.reset(client_server_->Connect(
-        kHostJid, authenticator,
+    client_session_ = client_server_->Connect(
+        kHostJid, authenticator.Pass(),
         CandidateSessionConfig::CreateDefault(),
         base::Bind(&MockSessionCallback::OnStateChange,
-                   base::Unretained(&client_connection_callback_))));
+                   base::Unretained(&client_connection_callback_)));
 
     message_loop_.RunAllPending();
 
     Mock::VerifyAndClearExpectations(&host_connection_callback_);
     Mock::VerifyAndClearExpectations(&client_connection_callback_);
 
     if (!expect_fail) {
       // Expect that the connection will be closed eventually.
       EXPECT_CALL(host_connection_callback_,
                   OnStateChange(Session::CLOSED))
@@ skipping 74 matching lines @@
     InSequence dummy;
 
     EXPECT_CALL(client_connection_callback_,
                 OnStateChange(Session::CONNECTING))
         .Times(1);
     EXPECT_CALL(client_connection_callback_,
                 OnStateChange(Session::FAILED))
         .Times(1);
   }
 
-  Authenticator* authenticator = new FakeAuthenticator(
-      FakeAuthenticator::CLIENT, 1, FakeAuthenticator::ACCEPT, true);
-  client_session_.reset(client_server_->Connect(
-      kHostJid, authenticator,
-      CandidateSessionConfig::CreateDefault(),
+  scoped_ptr<Authenticator> authenticator(new FakeAuthenticator(
+      FakeAuthenticator::CLIENT, 1, FakeAuthenticator::ACCEPT, true));
+  client_session_ = client_server_->Connect(
+      kHostJid, authenticator.Pass(), CandidateSessionConfig::CreateDefault(),
       base::Bind(&MockSessionCallback::OnStateChange,
-                 base::Unretained(&client_connection_callback_))));
+                 base::Unretained(&client_connection_callback_)));
 
   message_loop_.RunAllPending();
 }
 
 // Verify that we can connect two endpoints with single-step authentication.
 TEST_F(JingleSessionTest, Connect) {
   CreateSessionManagers(1, FakeAuthenticator::ACCEPT);
   InitiateConnection(1, FakeAuthenticator::ACCEPT, false);
 }
 
@@ skipping 108 matching lines @@
 
   DatagramConnectionTester tester(
       client_socket, host_socket, kMessageSize, kMessages, kUdpWriteDelayMs);
   tester.Start();
   message_loop_.Run();
   tester.CheckResults();
 }
 
 }  // namespace protocol
 }  // namespace remoting