RefCounted types should not have public destructors, webkit/

webkit/tools/test_shell/simple_file_writer.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 "webkit/tools/test_shell/simple_file_writer.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/message_loop_proxy.h"
@@ skipping 22 matching lines @@
                 FileSystemContext* file_system_context)
       : simple_writer_(simple_writer),
         operation_(NULL),
         file_system_context_(file_system_context) {
     // The IO thread needs to be running for this class to work.
     SimpleResourceLoaderBridge::EnsureIOThread();
     io_thread_ = SimpleResourceLoaderBridge::GetIoThread();
     main_thread_ = base::MessageLoopProxy::current();
   }
 
-  virtual ~IOThreadProxy() {
-  }
-
   void Truncate(const GURL& path, int64 offset) {
     if (!io_thread_->BelongsToCurrentThread()) {
       io_thread_->PostTask(
           FROM_HERE,
           base::Bind(&IOThreadProxy::Truncate, this, path, offset));
       return;
     }
     DCHECK(!operation_);
     operation_ = GetNewOperation(path);
     operation_->Truncate(path, offset,
@@ skipping 22 matching lines @@
       return;
     }
     if (!operation_) {
       DidFailOnMainThread(base::PLATFORM_FILE_ERROR_INVALID_OPERATION);
       return;
     }
     operation_->Cancel(base::Bind(&IOThreadProxy::DidFinish, this));
   }
 
  private:
+  friend class base::RefCountedThreadSafe<IOThreadProxy>;
+  virtual ~IOThreadProxy() {}
+
   FileSystemOperationInterface* GetNewOperation(const GURL& path) {
     return file_system_context_->CreateFileSystemOperation(path, io_thread_);
   }
 
   void DidSucceedOnMainThread() {
     if (!main_thread_->BelongsToCurrentThread()) {
       main_thread_->PostTask(
           FROM_HERE,
           base::Bind(&IOThreadProxy::DidSucceedOnMainThread, this));
       return;
@@ skipping 78 matching lines @@
 }
 
 void SimpleFileWriter::DoWrite(
     const GURL& path, const GURL& blob_url, int64 offset) {
   io_thread_proxy_->Write(path, blob_url, offset);
 }
 
 void SimpleFileWriter::DoCancel() {
   io_thread_proxy_->Cancel();
 }