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

content/browser/device_orientation/provider_unittest.cc

Index: content/browser/device_orientation/provider_unittest.cc
diff --git a/content/browser/device_orientation/provider_unittest.cc b/content/browser/device_orientation/provider_unittest.cc
index b373b89d50228cca4dfa6ea1d11fd461df6b4453..e95e7b8cbc9df78714e2bedfc5e8d0482a4b3988 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: