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

chrome/browser/extensions/extension_context_menu_model.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/extension_context_menu_model.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/prefs/pref_service.h"
@@ skipping 29 matching lines @@
       extension_id_(extension->id()),
       browser_(browser),
       profile_(browser->profile()) {
   extension_action_ = extension->browser_action();
   if (!extension_action_)
     extension_action_ = extension->page_action();
 
   InitCommonCommands();
 }
 
-ExtensionContextMenuModel::~ExtensionContextMenuModel() {
-}
-
-void ExtensionContextMenuModel::InitCommonCommands() {
-  const Extension* extension = GetExtension();
-
-  // The extension pointer should only be null if the extension was uninstalled,
-  // and since the menu just opened, it should still be installed.
-  DCHECK(extension);
-
-  AddItem(NAME, UTF8ToUTF16(extension->name()));
-  AddSeparator();
-  AddItemWithStringId(CONFIGURE, IDS_EXTENSIONS_OPTIONS_MENU_ITEM);
-  AddItemWithStringId(DISABLE, IDS_EXTENSIONS_DISABLE);
-  AddItem(UNINSTALL, l10n_util::GetStringUTF16(IDS_EXTENSIONS_UNINSTALL));
-  if (extension->browser_action())
-    AddItemWithStringId(HIDE, IDS_EXTENSIONS_HIDE_BUTTON);
-  AddSeparator();
-  AddItemWithStringId(MANAGE, IDS_MANAGE_EXTENSIONS);
-}
-
 bool ExtensionContextMenuModel::IsCommandIdChecked(int command_id) const {
   return false;
 }
 
 bool ExtensionContextMenuModel::IsCommandIdEnabled(int command_id) const {
   const Extension* extension = this->GetExtension();
   if (!extension)
     return false;
 
   if (command_id == CONFIGURE) {
@@ skipping 65 matching lines @@
     profile_->GetExtensionService()->UninstallExtension(extension_id_, false,
                                                         NULL);
 
   Release();
 }
 
 void ExtensionContextMenuModel::ExtensionUninstallCanceled() {
   Release();
 }
 
+ExtensionContextMenuModel::~ExtensionContextMenuModel() {}
+
+void ExtensionContextMenuModel::InitCommonCommands() {
+  const Extension* extension = GetExtension();
+
+  // The extension pointer should only be null if the extension was uninstalled,
+  // and since the menu just opened, it should still be installed.
+  DCHECK(extension);
+
+  AddItem(NAME, UTF8ToUTF16(extension->name()));
+  AddSeparator();
+  AddItemWithStringId(CONFIGURE, IDS_EXTENSIONS_OPTIONS_MENU_ITEM);
+  AddItemWithStringId(DISABLE, IDS_EXTENSIONS_DISABLE);
+  AddItem(UNINSTALL, l10n_util::GetStringUTF16(IDS_EXTENSIONS_UNINSTALL));
+  if (extension->browser_action())
+    AddItemWithStringId(HIDE, IDS_EXTENSIONS_HIDE_BUTTON);
+  AddSeparator();
+  AddItemWithStringId(MANAGE, IDS_MANAGE_EXTENSIONS);
+}
+
 const Extension* ExtensionContextMenuModel::GetExtension() const {
   ExtensionService* extension_service = profile_->GetExtensionService();
   return extension_service->GetExtensionById(extension_id_, false);
 }