RefCounted types should not have public destructors, base/ edition

base/files/file_path_watcher_browsertest.cc

Index: base/files/file_path_watcher_browsertest.cc
diff --git a/base/files/file_path_watcher_browsertest.cc b/base/files/file_path_watcher_browsertest.cc
index 495f4d3d44c1280dbadc4879687baa0a8304ef06..7886df8de218f00b04b1e3d8401f850d6b4da3ac 100644
--- a/base/files/file_path_watcher_browsertest.cc
+++ b/base/files/file_path_watcher_browsertest.cc
@@ -65,6 +65,9 @@ class NotificationCollector
   }
 
  private:
+  friend class base::RefCountedThreadSafe<NotificationCollector>;
+  ~NotificationCollector() {}
+
   void RecordChange(TestDelegate* delegate) {
     ASSERT_TRUE(loop_->BelongsToCurrentThread());
     ASSERT_TRUE(delegates_.count(delegate));
@@ -107,6 +110,9 @@ class TestDelegate : public FilePathWatcher::Delegate {
     ADD_FAILURE() << "Error " << path.value();
   }
 
+ protected:
+  virtual ~TestDelegate() {}
+
  private:
   scoped_refptr<NotificationCollector> collector_;
 
@@ -249,6 +255,9 @@ class Deleter : public FilePathWatcher::Delegate {
 
   scoped_ptr<FilePathWatcher> watcher_;
   MessageLoop* loop_;
+
+ private:
+  virtual ~Deleter() {}
 };
 
 // Verify that deleting a watcher during the callback doesn't crash.