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

BUG=123295
TEST=none


Review URL: http://codereview.chromium.org/10068037

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134395 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 4 deletions

diff --git a/content/browser/device_orientation/provider_unittest.cc b/content/browser/device_orientation/provider_unittest.cc
index b373b89..e95e7b8 100644
--- a/content/browser/device_orientation/provider_unittest.cc
+++ b/content/browser/device_orientation/provider_unittest.cc
@@ -73,10 +73,6 @@ class MockOrientationFactory : public base::RefCounted<MockOrientationFactory> {
     instance_ = this;
   }
 
-  ~MockOrientationFactory() {
-    instance_ = NULL;
-  }
-
   static DataFetcher* CreateDataFetcher() {
     EXPECT_TRUE(instance_);
     return new MockDataFetcher(instance_);
@@ -88,6 +84,12 @@ class MockOrientationFactory : public base::RefCounted<MockOrientationFactory> {
   }
 
  private:
+  friend class base::RefCounted<MockOrientationFactory>;
+
+  ~MockOrientationFactory() {
+    instance_ = NULL;
+  }
+
   // Owned by ProviderImpl. Holds a reference back to MockOrientationFactory.
   class MockDataFetcher : public DataFetcher {
    public: