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

Side by Side Diff: content/renderer/media/audio_renderer_mixer_manager.cc

Issue 1892433002: Moving device description utils from AudioManager[Base] into AudioDeviceDescription; to be shared b… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: another rebase Created 4 years, 7 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 unified diff | Download patch
OLDNEW
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 <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
11 #include "build/build_config.h" 11 #include "build/build_config.h"
12 #include "content/renderer/media/audio_device_factory.h" 12 #include "content/renderer/media/audio_device_factory.h"
13 #include "media/audio/audio_manager_base.h" 13 #include "media/audio/audio_device_description.h"
14 #include "media/base/audio_hardware_config.h" 14 #include "media/base/audio_hardware_config.h"
15 #include "media/base/audio_renderer_mixer.h" 15 #include "media/base/audio_renderer_mixer.h"
16 #include "media/base/audio_renderer_mixer_input.h" 16 #include "media/base/audio_renderer_mixer_input.h"
17 17
18 namespace content { 18 namespace content {
19 19
20 AudioRendererMixerManager::AudioRendererMixerManager() {} 20 AudioRendererMixerManager::AudioRendererMixerManager() {}
21 21
22 AudioRendererMixerManager::~AudioRendererMixerManager() { 22 AudioRendererMixerManager::~AudioRendererMixerManager() {
23 // References to AudioRendererMixers may be owned by garbage collected 23 // References to AudioRendererMixers may be owned by garbage collected
24 // objects. During process shutdown they may be leaked, so, transitively, 24 // objects. During process shutdown they may be leaked, so, transitively,
25 // |mixers_| may leak (i.e., may be non-empty at this time) as well. 25 // |mixers_| may leak (i.e., may be non-empty at this time) as well.
26 } 26 }
27 27
28 media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput( 28 media::AudioRendererMixerInput* AudioRendererMixerManager::CreateInput(
29 int source_render_frame_id, 29 int source_render_frame_id,
30 int session_id, 30 int session_id,
31 const std::string& device_id, 31 const std::string& device_id,
32 const url::Origin& security_origin) { 32 const url::Origin& security_origin) {
33 // base::Unretained() is safe since AudioRendererMixerManager lives on the 33 // base::Unretained() is safe since AudioRendererMixerManager lives on the
34 // renderer thread and is destroyed on renderer thread destruction. 34 // renderer thread and is destroyed on renderer thread destruction.
35 return new media::AudioRendererMixerInput( 35 return new media::AudioRendererMixerInput(
36 base::Bind(&AudioRendererMixerManager::GetMixer, base::Unretained(this), 36 base::Bind(&AudioRendererMixerManager::GetMixer, base::Unretained(this),
37 source_render_frame_id), 37 source_render_frame_id),
38 base::Bind(&AudioRendererMixerManager::RemoveMixer, 38 base::Bind(&AudioRendererMixerManager::RemoveMixer,
39 base::Unretained(this), source_render_frame_id), 39 base::Unretained(this), source_render_frame_id),
40 // TODO(olka) remove the dependency on AudioManagerBase, 40 media::AudioDeviceDescription::UseSessionIdToSelectDevice(session_id,
41 // https://crbug.com/02581 41 device_id)
42 media::AudioManagerBase::UseSessionIdToSelectDevice(session_id, device_id)
43 ? AudioDeviceFactory::GetOutputDeviceInfo( 42 ? AudioDeviceFactory::GetOutputDeviceInfo(
44 source_render_frame_id, session_id, device_id, security_origin) 43 source_render_frame_id, session_id, device_id, security_origin)
45 .device_id() 44 .device_id()
46 : device_id, 45 : device_id,
47 security_origin); 46 security_origin);
48 } 47 }
49 48
50 media::AudioRendererMixer* AudioRendererMixerManager::GetMixer( 49 media::AudioRendererMixer* AudioRendererMixerManager::GetMixer(
51 int source_render_frame_id, 50 int source_render_frame_id,
52 const media::AudioParameters& params, 51 const media::AudioParameters& params,
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 const std::string& device_id, 140 const std::string& device_id,
142 const url::Origin& security_origin) 141 const url::Origin& security_origin)
143 : source_render_frame_id(source_render_frame_id), 142 : source_render_frame_id(source_render_frame_id),
144 params(params), 143 params(params),
145 device_id(device_id), 144 device_id(device_id),
146 security_origin(security_origin) {} 145 security_origin(security_origin) {}
147 146
148 AudioRendererMixerManager::MixerKey::MixerKey(const MixerKey& other) = default; 147 AudioRendererMixerManager::MixerKey::MixerKey(const MixerKey& other) = default;
149 148
150 } // namespace content 149 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/audio_renderer_mixer_manager.h ('k') | content/renderer/media/audio_renderer_mixer_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698