Fix memory leak in MediaStreamImplTest.

content/renderer/media/media_stream_impl_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/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "content/renderer/media/media_stream_impl.h"
 #include "content/renderer/media/mock_media_stream_dependency_factory.h"
 #include "content/renderer/media/mock_media_stream_dispatcher.h"
 #include "content/renderer/media/mock_web_peer_connection_00_handler_client.h"
@@ skipping 16 matching lines @@
         new VideoCaptureImplManager());
     MockMediaStreamDependencyFactory* dependency_factory =
         new MockMediaStreamDependencyFactory();
     ms_impl_.reset(new MediaStreamImpl(NULL,
                                        ms_dispatcher_.get(),
                                        p2p_socket_dispatcher_.get(),
                                        vc_manager.get(),
                                        dependency_factory));
   }
 
+  void TearDown() {
+    loop_.RunAllPending();

[tommi (sloooow) - chröme, 2012/04/23 12:17:52]

Add a comment for why this is needed (maybe even mention the P2P class here)

+  }
+
  protected:
   MessageLoop loop_;
   scoped_ptr<MockMediaStreamDispatcher> ms_dispatcher_;
   scoped_ptr<content::P2PSocketDispatcher> p2p_socket_dispatcher_;
   scoped_ptr<MediaStreamImpl> ms_impl_;
 };
 
 TEST_F(MediaStreamImplTest, Basic) {
   // TODO(grunell): Add tests for WebKit::WebUserMediaClient and
   // MediaStreamDispatcherEventHandler implementations.
@@ skipping 40 matching lines @@
       static_cast<PeerConnectionHandler*>(pc_handler));
   EXPECT_EQ(1u, ms_impl_->peer_connection_handlers_.size());
   delete pc_handler;
 
   // Delete PC handler explicitly after closing to mimic WebKit behavior.
   ms_impl_->ClosePeerConnection(
       static_cast<PeerConnectionHandlerJsep*>(pc_handler_jsep));
   EXPECT_TRUE(ms_impl_->peer_connection_handlers_.empty());
   delete pc_handler_jsep;
 }