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

content/browser/renderer_host/media/audio_renderer_host_unittest.cc

Index: content/browser/renderer_host/media/audio_renderer_host_unittest.cc
diff --git a/content/browser/renderer_host/media/audio_renderer_host_unittest.cc b/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
index 6a1287a0b59329725f4cfe92dc34a2dec4125bdf..ca6940c8b661239621cbece0f2a16b31bc431a64 100644
--- a/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
+++ b/content/browser/renderer_host/media/audio_renderer_host_unittest.cc
@@ -49,11 +49,6 @@ class MockAudioRendererHost : public AudioRendererHost {
         shared_memory_length_(0) {
   }
 
-  virtual ~MockAudioRendererHost() {
-    // Make sure all audio streams have been deleted.
-    EXPECT_EQ(0u, audio_entries_.size());
-  }
-
   // A list of mock methods.
   MOCK_METHOD2(OnStreamCreated,
                void(int stream_id, int length));
@@ -68,6 +63,11 @@ class MockAudioRendererHost : public AudioRendererHost {
   base::SyncSocket* sync_socket() { return sync_socket_.get(); }
 
  private:
+  virtual ~MockAudioRendererHost() {
+    // Make sure all audio streams have been deleted.
+    EXPECT_EQ(0u, audio_entries_.size());

[sky, 2012/04/27 15:45:23]

nit: empty

+  }
+
   // This method is used to dispatch IPC messages to the renderer. We intercept
   // these messages here and dispatch to our mock methods to verify the
   // conversation between this object and the renderer.