RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/api/socket/socket_api.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 "chrome/browser/extensions/api/socket/socket_api.h"
 
 #include "base/bind.h"
 #include "chrome/browser/extensions/api/api_resource_controller.h"
 #include "chrome/browser/extensions/api/socket/socket.h"
 #include "chrome/browser/extensions/api/socket/tcp_socket.h"
 #include "chrome/browser/extensions/api/socket/udp_socket.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "net/base/io_buffer.h"
 
 namespace extensions {
 
 const char kBytesWrittenKey[] = "bytesWritten";
 const char kDataKey[] = "data";
 const char kResultCodeKey[] = "resultCode";
 const char kSocketIdKey[] = "socketId";
 const char kTCPOption[] = "tcp";
 const char kUDPOption[] = "udp";
 
 const char kSocketNotFoundError[] = "Socket not found";
 
 SocketCreateFunction::SocketCreateFunction()
-    : src_id_(-1), event_notifier_(NULL) {
+    : src_id_(-1),
+      event_notifier_(NULL) {
 }
 
+SocketCreateFunction::~SocketCreateFunction() {}
+
 bool SocketCreateFunction::Prepare() {
   std::string socket_type_string;
   size_t argument_position = 0;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(argument_position++,
                                                &socket_type_string));
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(argument_position++,
                                                &address_));
   EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(argument_position++,
                                                 &port_));
 
@@ skipping 115 matching lines @@
 
 bool SocketReadFunction::Respond() {
   return true;
 }
 
 SocketWriteFunction::SocketWriteFunction()
     : socket_id_(0),
       io_buffer_(NULL) {
 }
 
-SocketWriteFunction::~SocketWriteFunction() {
-}
+SocketWriteFunction::~SocketWriteFunction() {}
 
 bool SocketWriteFunction::Prepare() {
   EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &socket_id_));
   base::ListValue* data_list_value = NULL;
   EXTENSION_FUNCTION_VALIDATE(args_->GetList(1, &data_list_value));
 
   size_t size = data_list_value->GetSize();
   if (size != 0) {
     io_buffer_ = new net::IOBufferWithSize(size);
     uint8* data_buffer =
@@ skipping 21 matching lines @@
   DictionaryValue* result = new DictionaryValue();
   result->SetInteger(kBytesWrittenKey, bytes_written);
   result_.reset(result);
 }
 
 bool SocketWriteFunction::Respond() {
   return true;
 }
 
 }  // namespace extensions