RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/gdata/gdata_file_system_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 <errno.h>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 1196 matching lines @@
   // RefCountedThreadSafe, and a place where we can fetch results from various
   // operations.
   class CallbackHelper
     : public base::RefCountedThreadSafe<CallbackHelper> {
    public:
     CallbackHelper()
         : last_error_(base::PLATFORM_FILE_OK),
           quota_bytes_total_(0),
           quota_bytes_used_(0),
           file_proto_(NULL) {}
-    virtual ~CallbackHelper() {}
 
     virtual void GetFileCallback(base::PlatformFileError error,
                                  const FilePath& file_path,
                                  const std::string& mime_type,
                                  GDataFileType file_type) {
       last_error_ = error;
       download_path_ = file_path;
       mime_type_ = mime_type;
       file_type_ = file_type;
     }
@@ skipping 50 matching lines @@
     base::PlatformFileError last_error_;
     FilePath download_path_;
     FilePath opened_file_path_;
     std::string mime_type_;
     GDataFileType file_type_;
     int64 quota_bytes_total_;
     int64 quota_bytes_used_;
     scoped_ptr<GDataEntryProto> entry_proto_;
     scoped_ptr<GDataFileProto> file_proto_;
     scoped_ptr<GDataDirectoryProto> directory_proto_;
+
+   protected:
+    virtual ~CallbackHelper() {}
+
+   private:
+    friend class base::RefCountedThreadSafe<CallbackHelper>;
   };
 
   MessageLoopForUI message_loop_;
   // The order of the test threads is important, do not change the order.
   // See also content/browser/browser_thread_imple.cc.
   content::TestBrowserThread ui_thread_;
   content::TestBrowserThread io_thread_;
   const base::SequencedWorkerPool::SequenceToken sequence_token_;
   scoped_ptr<TestingProfile> profile_;
   scoped_refptr<CallbackHelper> callback_helper_;
@@ skipping 2222 matching lines @@
 
   // Try to close the same file twice.
   file_system_->CloseFile(kFileInRoot, close_file_callback);
   message_loop_.Run();
 
   // It must fail.
   EXPECT_EQ(base::PLATFORM_FILE_ERROR_NOT_FOUND, callback_helper_->last_error_);
 }
 
 }   // namespace gdata