RefCounted types should not have public destructors, webkit/

webkit/fileapi/file_system_context.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/fileapi/file_system_context.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/message_loop_proxy.h"
 #include "googleurl/src/gurl.h"
@@ skipping 52 matching lines @@
   if (quota_manager_proxy) {
     quota_manager_proxy->RegisterClient(CreateQuotaClient(
         file_message_loop, this, options.is_incognito()));
   }
 #if defined(OS_CHROMEOS)
   external_provider_.reset(
       new chromeos::CrosMountPointProvider(special_storage_policy));
 #endif
 }
 
-FileSystemContext::~FileSystemContext() {
-}
-
 bool FileSystemContext::DeleteDataForOriginOnFileThread(
     const GURL& origin_url) {
   DCHECK(file_message_loop_->BelongsToCurrentThread());
   DCHECK(sandbox_provider());
 
   // Delete temporary and persistent data.
   return
       sandbox_provider()->DeleteOriginDataOnFileThread(
           quota_manager_proxy(), origin_url, kFileSystemTypeTemporary) &&
       sandbox_provider()->DeleteOriginDataOnFileThread(
@@ skipping 51 matching lines @@
 SandboxMountPointProvider*
 FileSystemContext::sandbox_provider() const {
   return sandbox_provider_.get();
 }
 
 ExternalFileSystemMountPointProvider*
 FileSystemContext::external_provider() const {
   return external_provider_.get();
 }
 
-void FileSystemContext::DeleteOnCorrectThread() const {
-  if (!io_message_loop_->BelongsToCurrentThread() &&
-      io_message_loop_->DeleteSoon(FROM_HERE, this)) {
-    return;
-  }
-  delete this;
-}
-
 void FileSystemContext::OpenFileSystem(
     const GURL& origin_url,
     FileSystemType type,
     bool create,
     OpenFileSystemCallback callback) {
   DCHECK(!callback.is_null());
 
   FileSystemMountPointProvider* mount_point_provider =
       GetMountPointProvider(type);
   if (!mount_point_provider) {
@@ skipping 34 matching lines @@
   FilePath file_path;
   if (!CrackFileSystemURL(url, &origin_url, &file_system_type, &file_path))
     return NULL;
   FileSystemMountPointProvider* mount_point_provider =
       GetMountPointProvider(file_system_type);
   if (!mount_point_provider)
     return NULL;
   return mount_point_provider->CreateFileReader(url, offset, file_proxy, this);
 }
 
+FileSystemContext::~FileSystemContext() {}
+
+void FileSystemContext::DeleteOnCorrectThread() const {
+  if (!io_message_loop_->BelongsToCurrentThread() &&
+      io_message_loop_->DeleteSoon(FROM_HERE, this)) {
+    return;
+  }
+  delete this;
+}
+
 }  // namespace fileapi