RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/extensions/file_browser_handler_api.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 "chrome/browser/chromeos/extensions/file_browser_handler_api.h"
 
 #include "base/bind.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/message_loop_proxy.h"
@@ skipping 161 matching lines @@
     const GURL& file_system_root) {
   bool success = (error == base::PLATFORM_FILE_OK);
   content::BrowserThread::PostTask(content::BrowserThread::FILE, FROM_HERE,
       base::Bind(callback, success, file_system_name, file_system_root));
 }
 
 }  // namespace
 
 FileHandlerSelectFileFunction::FileHandlerSelectFileFunction() {}
 
+void FileHandlerSelectFileFunction::OnFilePathSelected(
+    bool success,
+    const FilePath& full_path) {
+  if (!success) {
+    Respond(false, std::string(), GURL(), FilePath());
+    return;
+  }
+
+  full_path_ = full_path;
+
+  BrowserContext::GetFileSystemContext(profile_)->OpenFileSystem(
+      source_url_.GetOrigin(), fileapi::kFileSystemTypeExternal, false,
+      base::Bind(&RelayOpenFileSystemCallbackToFileThread,
+          base::Bind(&FileHandlerSelectFileFunction::CreateFileOnFileThread,
+                     this)));
+};
+
+// static
+void FileHandlerSelectFileFunction::set_file_selector_for_test(
+    FileSelector* file_selector) {
+  FileHandlerSelectFileFunction::file_selector_for_test_ = file_selector;
+}
+
+// static
+void FileHandlerSelectFileFunction::set_gesture_check_disabled_for_test(
+    bool disabled) {
+  FileHandlerSelectFileFunction::gesture_check_disabled_for_test_ = disabled;
+}
+
 FileHandlerSelectFileFunction::~FileHandlerSelectFileFunction() {}
 
 bool FileHandlerSelectFileFunction::RunImpl() {
   scoped_ptr<SelectFile::Params> params(SelectFile::Params::Create(*args_));
 
   FilePath suggested_name(params->selection_params.suggested_name);
 
   if (!user_gesture() && !gesture_check_disabled_for_test_) {
     error_ = kNoUserGestureError;
     return false;
   }
 
   // If |file_selector_| is set (e.g. in test), use it instesad of creating new
   // file selector.
   FileSelector* file_selector = GetFileSelector();
   file_selector->SelectFile(suggested_name.BaseName(), GetCurrentBrowser());
   return true;
 }
 
-// static
-void FileHandlerSelectFileFunction::set_file_selector_for_test(
-    FileSelector* file_selector) {
-  FileHandlerSelectFileFunction::file_selector_for_test_ = file_selector;
-}
-
-// static
-void FileHandlerSelectFileFunction::set_gesture_check_disabled_for_test(
-    bool disabled) {
-  FileHandlerSelectFileFunction::gesture_check_disabled_for_test_ = disabled;
-}
-
-void FileHandlerSelectFileFunction::OnFilePathSelected(
-    bool success,
-    const FilePath& full_path) {
-  if (!success) {
-    Respond(false, std::string(), GURL(), FilePath());
-    return;
-  }
-
-  full_path_ = full_path;
-
-  BrowserContext::GetFileSystemContext(profile_)->OpenFileSystem(
-      source_url_.GetOrigin(), fileapi::kFileSystemTypeExternal, false,
-      base::Bind(&RelayOpenFileSystemCallbackToFileThread,
-          base::Bind(&FileHandlerSelectFileFunction::CreateFileOnFileThread,
-                     this)));
-};
-
 void FileHandlerSelectFileFunction::CreateFileOnFileThread(
     bool success,
     const std::string& file_system_name,
     const GURL& file_system_root) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
 
   if (success)
     success = DoCreateFile();
   content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
       base::Bind(&FileHandlerSelectFileFunction::OnFileCreated, this,
@@ skipping 75 matching lines @@
     result->set_function_for_test(this);
     return result;
   }
   return new FileSelectorImpl(this);
 }
 
 FileSelector* FileHandlerSelectFileFunction::file_selector_for_test_ = NULL;
 
 bool FileHandlerSelectFileFunction::gesture_check_disabled_for_test_ = false;