Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2166)

Unified Diff: content/test/webrtc_audio_device_test.cc

Issue 10786029: Delete MediaStreamManager in the same way as AudioManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: put Stop() in the dtor. Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/renderer_host/media/video_capture_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/webrtc_audio_device_test.cc
diff --git a/content/test/webrtc_audio_device_test.cc b/content/test/webrtc_audio_device_test.cc
index 350e0c1141835c6d3f11a9acd71ca4ac4746bfb6..cb9fff9d2e8cedda021f83b6160970f9fa7acc23 100644
--- a/content/test/webrtc_audio_device_test.cc
+++ b/content/test/webrtc_audio_device_test.cc
@@ -133,6 +133,17 @@ void WebRTCAudioDeviceTest::SetUp() {
ui_thread_.reset(new content::TestBrowserThread(content::BrowserThread::UI,
MessageLoop::current()));
+ // Create our own AudioManager and MediaStreamManager.
+ audio_manager_.reset(media::AudioManager::Create());
+
+ scoped_refptr<media_stream::AudioInputDeviceManager>
+ audio_input_device_manager(new media_stream::AudioInputDeviceManager(
+ audio_manager_.get()));
+ scoped_refptr<media_stream::VideoCaptureManager> video_capture_manager(
+ new media_stream::VideoCaptureManager());
+ media_stream_manager_.reset(new media_stream::MediaStreamManager(
+ audio_input_device_manager, video_capture_manager));
+
// Construct the resource context on the UI thread.
resource_context_.reset(new MockResourceContext);
@@ -173,6 +184,8 @@ void WebRTCAudioDeviceTest::TearDown() {
base::Unretained((this))));
WaitForIOThreadCompletion();
mock_process_.reset();
+ media_stream_manager_.reset();
+ audio_manager_.reset();
RendererWebKitPlatformSupportImpl::SetSandboxEnabledForTesting(
sandbox_was_enabled_);
}
@@ -197,16 +210,6 @@ void WebRTCAudioDeviceTest::InitializeIOThread(const char* thread_name) {
io_thread_.reset(new content::TestBrowserThread(content::BrowserThread::IO,
MessageLoop::current()));
- audio_manager_.reset(media::AudioManager::Create());
-
- scoped_refptr<media_stream::AudioInputDeviceManager>
- audio_input_device_manager(new media_stream::AudioInputDeviceManager(
- audio_manager_.get()));
- scoped_refptr<media_stream::VideoCaptureManager> video_capture_manager(
- new media_stream::VideoCaptureManager());
- media_stream_manager_.reset(new media_stream::MediaStreamManager(
- audio_input_device_manager, video_capture_manager));
-
// Populate our resource context.
test_request_context_.reset(new TestURLRequestContext());
MockResourceContext* resource_context =
@@ -224,8 +227,6 @@ void WebRTCAudioDeviceTest::InitializeIOThread(const char* thread_name) {
void WebRTCAudioDeviceTest::UninitializeIOThread() {
resource_context_.reset();
- media_stream_manager_.reset();
- audio_manager_.reset();
test_request_context_.reset();
initialize_com_.reset();
}
« no previous file with comments | « content/browser/renderer_host/media/video_capture_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698