RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/api/permissions/permissions_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/extensions/api/permissions/permissions_api.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/api/permissions/permissions_api_helpers.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/permissions_updater.h"
@@ skipping 99 matching lines @@
   auto_confirm_for_tests = should_proceed ? PROCEED : ABORT;
 }
 
 // static
 void RequestPermissionsFunction::SetIgnoreUserGestureForTests(
     bool ignore) {
   ignore_user_gesture_for_tests = ignore;
 }
 
 RequestPermissionsFunction::RequestPermissionsFunction() {}
+
+void RequestPermissionsFunction::InstallUIProceed() {
+  PermissionsUpdater perms_updater(profile());
+  perms_updater.AddPermissions(GetExtension(), requested_permissions_.get());
+
+  result_.reset(Request::Result::Create(true));
+  SendResponse(true);
+
+  Release();  // Balanced in RunImpl().
+}
+
+void RequestPermissionsFunction::InstallUIAbort(bool user_initiated) {
+  result_.reset(Request::Result::Create(false));
+  SendResponse(true);
+
+  Release();  // Balanced in RunImpl().
+}
+
 RequestPermissionsFunction::~RequestPermissionsFunction() {}
 
 bool RequestPermissionsFunction::RunImpl() {
   if (!user_gesture() && !ignore_user_gesture_for_tests) {
     error_ = kUserGestureRequiredError;
     return false;
   }
 
   scoped_ptr<Request::Params> params(Request::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
@@ skipping 55 matching lines @@
     InstallUIAbort(true);
   } else {
     CHECK_EQ(DO_NOT_SKIP, auto_confirm_for_tests);
     install_ui_.reset(new ExtensionInstallUI(profile()));
     install_ui_->ConfirmPermissions(
         this, GetExtension(), requested_permissions_.get());
   }
 
   return true;
 }
-
-void RequestPermissionsFunction::InstallUIProceed() {
-  PermissionsUpdater perms_updater(profile());
-  perms_updater.AddPermissions(GetExtension(), requested_permissions_.get());
-
-  result_.reset(Request::Result::Create(true));
-  SendResponse(true);
-
-  Release();  // Balanced in RunImpl().
-}
-
-void RequestPermissionsFunction::InstallUIAbort(bool user_initiated) {
-  result_.reset(Request::Result::Create(false));
-  SendResponse(true);
-
-  Release();  // Balanced in RunImpl().
-}