RefCounted types should not have public destructors, chrome/ remaining parts

chrome/common/extensions/extension_permission_set.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_COMMON_EXTENSIONS_EXTENSION_PERMISSION_SET_H_
 #define CHROME_COMMON_EXTENSIONS_EXTENSION_PERMISSION_SET_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 276 matching lines @@
   // manifest, |apis| and |hosts|.
   ExtensionPermissionSet(const Extension* extension,
                          const ExtensionAPIPermissionSet& apis,
                          const URLPatternSet& explicit_hosts);
 
   // Creates a new permission set based on the specified data.
   ExtensionPermissionSet(const ExtensionAPIPermissionSet& apis,
                          const URLPatternSet& explicit_hosts,
                          const URLPatternSet& scriptable_hosts);
 
-  ~ExtensionPermissionSet();
-
   // Creates a new permission set equal to |set1| - |set2|, passing ownership of
   // the new set to the caller.
   static ExtensionPermissionSet* CreateDifference(
       const ExtensionPermissionSet* set1, const ExtensionPermissionSet* set2);
 
   // Creates a new permission set equal to the intersection of |set1| and
   // |set2|, passing ownership of the new set to the caller.
   static ExtensionPermissionSet* CreateIntersection(
       const ExtensionPermissionSet* set1, const ExtensionPermissionSet* set2);
 
@@ skipping 69 matching lines @@
 
   const URLPatternSet& effective_hosts() const { return effective_hosts_; }
 
   const URLPatternSet& explicit_hosts() const { return explicit_hosts_; }
 
   const URLPatternSet& scriptable_hosts() const { return scriptable_hosts_; }
 
  private:
   FRIEND_TEST_ALL_PREFIXES(ExtensionPermissionsTest,
                            HasLessHostPrivilegesThan);
+  friend class base::RefCountedThreadSafe<ExtensionPermissionSet>;
 
-  friend class base::RefCountedThreadSafe<ExtensionPermissionSet>;
+  ~ExtensionPermissionSet();
 
   static std::set<std::string> GetDistinctHosts(
       const URLPatternSet& host_patterns,
       bool include_rcd,
       bool exclude_file_scheme);
 
   // Initializes the set based on |extension|'s manifest data.
   void InitImplicitExtensionPermissions(const Extension* extension);
 
   // Initializes the effective host permission based on the data in this set.
@@ skipping 22 matching lines @@
 
   // The list of hosts that can be scripted by content scripts.
   // TODO(jstritar): Rename to "user_script_hosts_"?
   URLPatternSet scriptable_hosts_;
 
   // The list of hosts this effectively grants access to.
   URLPatternSet effective_hosts_;
 };
 
 #endif  // CHROME_COMMON_EXTENSIONS_EXTENSION_PERMISSION_SET_H_