Refactor MediaStreamManager to never output real device id. It now always output sourceId in the fo…

content/browser/renderer_host/media/media_stream_manager_unittest.cc

Index: content/browser/renderer_host/media/media_stream_manager_unittest.cc
diff --git a/content/browser/renderer_host/media/media_stream_manager_unittest.cc b/content/browser/renderer_host/media/media_stream_manager_unittest.cc
index a67237a1ed98e405329423ab2c2461214bf9ca8f..5136e0075d5f729fc7f5afe0d83bce947141096d 100644
--- a/content/browser/renderer_host/media/media_stream_manager_unittest.cc
+++ b/content/browser/renderer_host/media/media_stream_manager_unittest.cc
@@ -77,7 +77,6 @@ class MediaStreamManagerTest : public ::testing::Test {
   }
 
   virtual ~MediaStreamManagerTest() {
-    media_stream_manager_->WillDestroyCurrentMessageLoop();
   }
 
   MOCK_METHOD1(Response, void(int index));
@@ -129,6 +128,7 @@ TEST_F(MediaStreamManagerTest, MakeAndCancelMediaAccessRequest) {
   std::string label = MakeMediaAccessRequest(0);
   // No callback is expected.
   media_stream_manager_->CancelRequest(label);
+  media_stream_manager_->WillDestroyCurrentMessageLoopForTest();

[tommi (sloooow) - chröme, 2013/10/29 17:21:17]

Why can't we simply delete the message loop and let the same thing happen as
does when chrome runs?

[perkj_chrome, 2013/10/30 08:59:29]

delete MessageLoop::current(); ?
Maybe you can help me with solving this?

 }
 
 TEST_F(MediaStreamManagerTest, MakeMultipleRequests) {