Fix leak; remove "explicit" keyword from multi-argument constructor.

chrome/browser/extensions/api/socket/socket_api_controller.cc

Index: chrome/browser/extensions/api/socket/socket_api_controller.cc
diff --git a/chrome/browser/extensions/api/socket/socket_api_controller.cc b/chrome/browser/extensions/api/socket/socket_api_controller.cc
index 21952973ef560f3f3809fcf7a86e1cbf79b163db..1c168960dd53cc6f6e86dd8b868df600c082d5e5 100644
--- a/chrome/browser/extensions/api/socket/socket_api_controller.cc
+++ b/chrome/browser/extensions/api/socket/socket_api_controller.cc
@@ -22,8 +22,8 @@ namespace extensions {
 // we need to manage it in the context of an extension.
 class Socket {
  public:
-  explicit Socket(const Profile* profile, const std::string& src_extension_id,
-                  const GURL& src_url);
+  Socket(const Profile* profile, const std::string& src_extension_id,
+         const GURL& src_url);
   ~Socket();
 
   bool Connect(const net::IPEndPoint& ip_end_point);
@@ -64,6 +64,7 @@ Socket::~Socket() {
   if (is_connected_) {
     Close();
   }
+  delete udp_client_socket_;

[cbentzel, 2011/12/06 18:56:32]

Could you use a scoped_ptr instead?

 }
 
 void Socket::OnIOComplete(int result) {