Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "content/renderer/media/audio_renderer_mixer_manager.h" | 5 #include "content/renderer/media/audio_renderer_mixer_manager.h" |
| 6 | 6 |
| 7 #include "base/bind.h" | 7 #include "base/bind.h" |
| 8 #include "base/bind_helpers.h" | 8 #include "base/bind_helpers.h" |
| 9 #include "content/renderer/media/audio_device_factory.h" | 9 #include "content/renderer/media/audio_device_factory.h" |
| 10 #include "media/audio/audio_output_device.h" | 10 #include "media/audio/audio_output_device.h" |
| 11 #include "media/base/audio_hardware_config.h" | 11 #include "media/base/audio_hardware_config.h" |
| 12 #include "media/base/audio_renderer_mixer.h" | 12 #include "media/base/audio_renderer_mixer.h" |
| 13 #include "media/base/audio_renderer_mixer_input.h" | 13 #include "media/base/audio_renderer_mixer_input.h" |
| 14 | 14 |
| 15 namespace content { | 15 namespace content { |
| 16 | 16 |
| 17 AudioRendererMixerManager::AudioRendererMixerManager( | 17 AudioRendererMixerManager::AudioRendererMixerManager( |
| 18 media::AudioHardwareConfig* hardware_config) | 18 media::AudioHardwareConfig* hardware_config) |
| 19 : hardware_config_(hardware_config), | 19 : hardware_config_(hardware_config), |
| 20 sink_for_testing_(NULL) { | 20 sink_for_testing_(NULL) { |
| 21 } | 21 } |
| 22 | 22 |
| 23 AudioRendererMixerManager::~AudioRendererMixerManager() { | 23 AudioRendererMixerManager::~AudioRendererMixerManager() { |
| 24 DCHECK(mixers_.empty()); | 24 // ARM owners may be garbage collected objects. During process shutdown they |
|
scherkus (not reviewing)
2014/09/05 23:04:59
s/ARM/whatever-ARM-stands-for/?
DaleCurtis
2014/09/05 23:09:35
Done.
| |
| 25 // may be leaked, so, transitively, |mixers_| may leak as well. | |
| 25 } | 26 } |
| 26 | 27 |
| 27 media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput( | 28 media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput( |
| 28 int source_render_view_id, int source_render_frame_id) { | 29 int source_render_view_id, int source_render_frame_id) { |
| 29 return new media::AudioRendererMixerInput( | 30 return new media::AudioRendererMixerInput( |
| 30 base::Bind( | 31 base::Bind( |
| 31 &AudioRendererMixerManager::GetMixer, base::Unretained(this), | 32 &AudioRendererMixerManager::GetMixer, base::Unretained(this), |
| 32 source_render_view_id, | 33 source_render_view_id, |
| 33 source_render_frame_id), | 34 source_render_frame_id), |
| 34 base::Bind( | 35 base::Bind( |
| (...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 100 | 101 |
| 101 // Only remove the mixer if AudioRendererMixerManager is the last owner. | 102 // Only remove the mixer if AudioRendererMixerManager is the last owner. |
| 102 it->second.ref_count--; | 103 it->second.ref_count--; |
| 103 if (it->second.ref_count == 0) { | 104 if (it->second.ref_count == 0) { |
| 104 delete it->second.mixer; | 105 delete it->second.mixer; |
| 105 mixers_.erase(it); | 106 mixers_.erase(it); |
| 106 } | 107 } |
| 107 } | 108 } |
| 108 | 109 |
| 109 } // namespace content | 110 } // namespace content |
| OLD | NEW |