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

chrome/browser/extensions/api/socket/tcp_socket_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 "chrome/browser/extensions/api/socket/tcp_socket.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/api/api_resource_event_notifier.h"
 #include "net/base/address_list.h"
 #include "net/base/completion_callback.h"
@@ skipping 26 matching lines @@
 
 class MockAPIResourceEventNotifier : public APIResourceEventNotifier {
  public:
   MockAPIResourceEventNotifier() : APIResourceEventNotifier(NULL, NULL,
                                                             std::string(),
                                                             0, GURL()) {}
 
   MOCK_METHOD2(OnReadComplete, void(int result_code,
                                     const std::string& message));
   MOCK_METHOD1(OnWriteComplete, void(int result_code));
+
+ protected:
+  virtual ~MockAPIResourceEventNotifier() {}
 };
 
 TEST(SocketTest, TestTCPSocketRead) {
   net::AddressList address_list;
   MockTCPSocket* tcp_client_socket = new MockTCPSocket(address_list);
   APIResourceEventNotifier* notifier = new MockAPIResourceEventNotifier();
 
   scoped_ptr<TCPSocket> socket(TCPSocket::CreateSocketForTesting(
       tcp_client_socket, "1.2.3.4", 1, notifier));
 
@@ skipping 41 matching lines @@
                                                io_buffer->size()));
 
   // Good. Original call came back unable to complete. Now pretend the socket
   // finished, and confirm that we passed the error back.
   EXPECT_CALL(*notifier, OnWriteComplete(42))
       .Times(1);
   callback.Run(42);
 }
 
 }  // namespace extensions