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

content/browser/fileapi/chrome_blob_storage_context.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 CONTENT_BROWSER_FILEAPI_CHROME_BLOB_STORAGE_CONTEXT_H_
 #define CONTENT_BROWSER_FILEAPI_CHROME_BLOB_STORAGE_CONTEXT_H_
 #pragma once
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 12 matching lines @@
 struct ChromeBlobStorageContextDeleter;
 
 // A context class that keeps track of BlobStorageController used by the chrome.
 // There is an instance associated with each BrowserContext. There could be
 // multiple URLRequestContexts in the same browser context that refers to the
 // same instance.
 //
 // All methods, except the ctor, are expected to be called on
 // the IO thread (unless specifically called out in doc comments).
 class CONTENT_EXPORT ChromeBlobStorageContext
-    : public base::RefCountedThreadSafe<
-          ChromeBlobStorageContext, ChromeBlobStorageContextDeleter> {
+    : public base::RefCountedThreadSafe<ChromeBlobStorageContext,
+                                        ChromeBlobStorageContextDeleter> {
  public:
+  ChromeBlobStorageContext();
+
   static ChromeBlobStorageContext* GetFor(
       content::BrowserContext* browser_context);
 
-  ChromeBlobStorageContext();
-  virtual ~ChromeBlobStorageContext();
-
   void InitializeOnIOThread();
 
   webkit_blob::BlobStorageController* controller() const {
     return controller_.get();
   }
 
+ protected:
+  virtual ~ChromeBlobStorageContext();
+
  private:
+  friend class base::DeleteHelper<ChromeBlobStorageContext>;
+  friend class base::RefCountedThreadSafe<ChromeBlobStorageContext,
+                                          ChromeBlobStorageContextDeleter>;
   friend struct ChromeBlobStorageContextDeleter;
 
   void DeleteOnCorrectThread() const;
 
   scoped_ptr<webkit_blob::BlobStorageController> controller_;
 };
 
 struct ChromeBlobStorageContextDeleter {
   static void Destruct(const ChromeBlobStorageContext* context) {
     context->DeleteOnCorrectThread();
   }
 };
 
 #endif  // CONTENT_BROWSER_FILEAPI_CHROME_BLOB_STORAGE_CONTEXT_H_