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

content/browser/renderer_host/media/audio_renderer_host_unittest.cc

 // 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.
 
 #include "base/bind.h"
 #include "base/environment.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/process_util.h"
 #include "base/sync_socket.h"
@@ skipping 31 matching lines @@
 
 class MockAudioRendererHost : public AudioRendererHost {
  public:
   explicit MockAudioRendererHost(
       media::AudioManager* audio_manager,
       content::MediaObserver* media_observer)
       : AudioRendererHost(audio_manager, media_observer),
         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));
   MOCK_METHOD1(OnStreamPlaying, void(int stream_id));
   MOCK_METHOD1(OnStreamPaused, void(int stream_id));
   MOCK_METHOD1(OnStreamError, void(int stream_id));
   MOCK_METHOD2(OnStreamVolume, void(int stream_id, double volume));
 
   base::SharedMemory* shared_memory() { return shared_memory_.get(); }
   uint32 shared_memory_length() { return shared_memory_length_; }
 
   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.
   virtual bool Send(IPC::Message* message) {
     CHECK(message);
 
     // In this method we dispatch the messages to the according handlers as if
     // we are the renderer.
     bool handled = true;
     IPC_BEGIN_MESSAGE_MAP(MockAudioRendererHost, *message)
@@ skipping 323 matching lines @@
     EnableRealDevice();
 
   Create();
   Play();
   SimulateError();
   Close();
 }
 
 
 // TODO(hclam): Add tests for data conversation in low latency mode.