RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/extensions/file_handler_util.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_HANDLER_UTIL_H_
 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_HANDLER_UTIL_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 64 matching lines @@
 class FileTaskExecutor : public base::RefCountedThreadSafe<FileTaskExecutor> {
  public:
   static const char kDriveTaskExtensionPrefix[];
   static const size_t kDriveTaskExtensionPrefixLength;
 
   // Creates the appropriate FileTaskExecutor for the given |extension_id|.
   static FileTaskExecutor* Create(Profile* profile,
                                   const GURL source_url,
                                   const std::string& extension_id,
                                   const std::string& action_id);
-  virtual ~FileTaskExecutor();
 
   // Same as ExecuteAndNotify, but no notification is performed.
   virtual bool Execute(const std::vector<GURL>& file_urls);
 
   // Initiates execution of file handler task for each element of |file_urls|.
   // Return |false| if the execution cannot be initiated. Otherwise returns
   // |true| and then eventually calls |done| when all the files have
   // been handled. If there is an error during processing the list of files, the
   // caller will be informed of the failure via |done|, and the rest of
   // the files will not be processed.
   virtual bool ExecuteAndNotify(const std::vector<GURL>& file_urls,
                                 const FileTaskFinishedCallback& done) = 0;
 
  protected:
   explicit FileTaskExecutor(Profile* profile);
+  virtual ~FileTaskExecutor();
 
   // Returns the profile that this task was created with.
   Profile* profile() { return profile_; }
 
   // Returns a browser to use for the current browser.
   Browser* GetBrowser() const;
  private:
+  friend class base::RefCountedThreadSafe<FileTaskExecutor>;
 
   Profile* profile_;
 };
 
 }  // namespace file_handler_util
 
 #endif  // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_HANDLER_UTIL_H_