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 15 matching lines @@
 // 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:
+  ChromeBlobStorageContext();
+
   static ChromeBlobStorageContext* GetFor(
       content::BrowserContext* browser_context);
 
-  ChromeBlobStorageContext();
-  virtual ~ChromeBlobStorageContext();
-
   void InitializeOnIOThread();
 
   webkit_blob::BlobStorageController* controller() const {
     return controller_.get();
   }
 
  private:
   friend struct ChromeBlobStorageContextDeleter;
+  friend class base::DeleteHelper<ChromeBlobStorageContext>;
+
+  virtual ~ChromeBlobStorageContext();
 
   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_