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

chrome/browser/extensions/bundle_installer.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_EXTENSIONS_BUNDLE_INSTALLER_H_
 #define CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 59 matching lines @@
 
     std::string id;
     std::string manifest;
     std::string localized_name;
     State state;
   };
 
   typedef std::vector<Item> ItemList;
 
   BundleInstaller(Profile* profile, const ItemList& items);
-  virtual ~BundleInstaller();
 
   // Returns true if the user has approved the bundle's permissions.
   bool approved() const { return approved_; }
 
   // Gets the items in the given state.
   ItemList GetItemsWithState(Item::State state) const;
 
   // Parses the extension manifests and then prompts the user to approve their
   // permissions. One of OnBundleInstallApproved or OnBundleInstallCanceled
   // will be called when complete if |delegate| is not NULL.
@@ skipping 18 matching lines @@
   //   STATE_INSTALLED - installed bubble successful installs list
   //   STATE_FAILED    - installed bubble failed installs list
   string16 GetHeadingTextFor(Item::State state) const;
 
  private:
   friend class base::RefCountedThreadSafe<BundleInstaller>;
 
   typedef std::map<std::string, Item> ItemMap;
   typedef std::map<std::string, linked_ptr<base::DictionaryValue> > ManifestMap;
 
+  virtual ~BundleInstaller();
+
   // Displays the install bubble for |bundle| on |browser|.
   // Note: this is a platform specific implementation.
   static void ShowInstalledBubble(const BundleInstaller* bundle,
                                   Browser* browser);
 
   // Parses the manifests using WebstoreInstallHelper.
   void ParseManifests();
 
   // Notifies the delegate that the installation has been approved.
   void ReportApproved();
@@ skipping 60 matching lines @@
   scoped_ptr<ExtensionInstallUI> install_ui_;
 
   Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(BundleInstaller);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_