Fix some uses of scoped_ptr.release() in net/ such that the return value is not ignored.

net/socket/socks_client_socket_pool_unittest.cc

 // Copyright (c) 2010 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 "net/socket/socks_client_socket_pool.h"
 
 #include <vector>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
@@ skipping 31 matching lines @@
       if (rv == OK) {
         user_callback_ = NULL;
         OnConnect(OK);
       }
       return rv;
     }
 
     bool CancelHandle(const ClientSocketHandle* handle) {
       if (handle != handle_)
         return false;
-      socket_.reset(NULL);
+      socket_.reset();
       handle_ = NULL;
       user_callback_ = NULL;
       return true;
     }
 
    private:
     void OnConnect(int rv) {
       if (!socket_.get())
         return;
       if (rv == OK)
-        handle_->set_socket(socket_.get());
+        handle_->set_socket(socket_.release());
       else
-        socket_.reset(NULL);
+        socket_.reset();
 
-      socket_.release();
       handle_ = NULL;
 
       if (user_callback_) {
         CompletionCallback* callback = user_callback_;
         user_callback_ = NULL;
         callback->Run(rv);
       }
     }
 
     scoped_ptr<ClientSocket> socket_;
@@ skipping 287 matching lines @@
 
   requests_[0]->handle()->Reset();
   requests_[1]->handle()->Reset();
 }
 
 // It would be nice to also test the timeouts in SOCKSClientSocketPool.
 
 }  // namespace
 
 }  // namespace net