RefCounted types should not have public destructors, sql/ and jingle/ edition

jingle/glue/pseudotcp_adapter_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 "jingle/glue/pseudotcp_adapter.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 163 matching lines @@
                    net::Socket* client_socket,
                    net::Socket* host_socket)
       : message_loop_(message_loop),
         host_socket_(host_socket),
         client_socket_(client_socket),
         done_(false),
         write_errors_(0),
         read_errors_(0) {
   }
 
-  virtual ~TCPChannelTester() { }
-
   void Start() {
     message_loop_->PostTask(
         FROM_HERE, base::Bind(&TCPChannelTester::DoStart, this));
   }
 
   void CheckResults() {
     EXPECT_EQ(0, write_errors_);
     EXPECT_EQ(0, read_errors_);
 
     ASSERT_EQ(kTestDataSize + kMessageSize, input_buffer_->capacity());
 
     output_buffer_->SetOffset(0);
     ASSERT_EQ(kTestDataSize, output_buffer_->size());
 
     EXPECT_EQ(0, memcmp(output_buffer_->data(),
                         input_buffer_->StartOfBuffer(), kTestDataSize));
   }
 
  protected:
+  virtual ~TCPChannelTester() {}
+
   void Done() {
     done_ = true;
     message_loop_->PostTask(FROM_HERE, MessageLoop::QuitClosure());
   }
 
   void DoStart() {
     InitBuffers();
     DoRead();
     DoWrite();
   }
@@ skipping 64 matching lines @@
       }
     } else if (result > 0) {
       // Allocate memory for the next read.
       input_buffer_->SetCapacity(input_buffer_->capacity() + result);
       if (input_buffer_->capacity() == kTestDataSize + kMessageSize)
         Done();
     }
   }
 
  private:
+  friend class base::RefCountedThreadSafe<TCPChannelTester>;
+
   MessageLoop* message_loop_;
   net::Socket* host_socket_;
   net::Socket* client_socket_;
   bool done_;
 
   scoped_refptr<net::DrainableIOBuffer> output_buffer_;
   scoped_refptr<net::GrowableIOBuffer> input_buffer_;
 
   int write_errors_;
   int read_errors_;
@@ skipping 136 matching lines @@
                            client_pseudotcp_.get());
 
   tester->Start();
   message_loop_.Run();
   tester->CheckResults();
 }
 
 }  // namespace
 
 }  // namespace jingle_glue