Delete MediaStreamManager in the same way as AudioManager

content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc

Index: content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc
diff --git a/content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc b/content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc
index 8c6d1b0a09e626051966667a8d79c34da6b0f397..58bae1e1b001b993ff198ce5a095cb99cf30c17d 100644
--- a/content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc
+++ b/content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc
@@ -200,6 +200,9 @@ class MediaStreamDispatcherHostTest : public testing::Test {
     content::GetContentClient()->set_browser_for_testing(old_browser_client_);
     content::SetContentClient(old_client_);
     content_client_.reset();
+
+    // Delete the IO message loop to delete the device thread and managers.
+    message_loop_.reset();

[tommi (sloooow) - chröme, 2012/07/17 15:36:58]

Did you have to add this explicitly?  This doesn't delete the
media_stream_manager_, right (which managers does it delete?)?

[no longer working on chromium, 2012/07/17 17:18:25]

This deletes the device thread, AudioInputDeviceManager and VideoCaptureManager
(they are deleted when the IO loop goes away). So we need to do it explicit,
otherwise it will hit the DCHECKs in the destructor of MediaStreamManager.

   }
 
   scoped_refptr<MockMediaStreamDispatcherHost> host_;