RefCounted types should not have public destructors, content/browser part 1

content/browser/in_process_webkit/indexed_db_context_impl.h

 // Copyright (c) 2011 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 CONTENT_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_IMPL_H_
 #define CONTENT_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_IMPL_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 24 matching lines @@
 
 class CONTENT_EXPORT IndexedDBContextImpl
     : NON_EXPORTED_BASE(public content::IndexedDBContext) {
  public:
   // If |data_path| is empty, nothing will be saved to disk.
   IndexedDBContextImpl(const FilePath& data_path,
                        quota::SpecialStoragePolicy* special_storage_policy,
                        quota::QuotaManagerProxy* quota_manager_proxy,
                        base::MessageLoopProxy* webkit_thread_loop);
 
-  virtual ~IndexedDBContextImpl();
-
   WebKit::WebIDBFactory* GetIDBFactory();
 
   // The indexed db directory.
   static const FilePath::CharType kIndexedDBDirectory[];
 
   // The indexed db file extension.
   static const FilePath::CharType kIndexedDBExtension[];
 
   void set_clear_local_state_on_exit(bool clear_local_state) {
     clear_local_state_on_exit_ = clear_local_state;
@@ skipping 21 matching lines @@
 
   quota::QuotaManagerProxy* quota_manager_proxy();
 
   FilePath data_path() const { return data_path_; }
 
   // For unit tests allow to override the |data_path_|.
   void set_data_path_for_testing(const FilePath& data_path) {
     data_path_ = data_path;
   }
 
+ protected:
+  virtual ~IndexedDBContextImpl();
+
  private:
   FRIEND_TEST_ALL_PREFIXES(IndexedDBTest, ClearLocalState);
   FRIEND_TEST_ALL_PREFIXES(IndexedDBTest, ClearSessionOnlyDatabases);
   FRIEND_TEST_ALL_PREFIXES(IndexedDBTest, SaveSessionState);
   friend class IndexedDBQuotaClientTest;
 
   typedef std::map<GURL, int64> OriginToSizeMap;
   class IndexedDBGetUsageAndQuotaCallback;
 
   FilePath GetIndexedDBFilePath(const string16& origin_id) const;
@@ skipping 29 matching lines @@
   scoped_refptr<quota::QuotaManagerProxy> quota_manager_proxy_;
   scoped_ptr<std::set<GURL> > origin_set_;
   OriginToSizeMap origin_size_map_;
   OriginToSizeMap space_available_map_;
   std::map<GURL, unsigned int> connection_count_;
 
   DISALLOW_COPY_AND_ASSIGN(IndexedDBContextImpl);
 };
 
 #endif  // CONTENT_BROWSER_IN_PROCESS_WEBKIT_INDEXED_DB_CONTEXT_IMPL_H_