Index: media/mojo/services/renderer_config_default.cc |
diff --git a/media/mojo/services/renderer_config_default.cc b/media/mojo/services/renderer_config_default.cc |
index ccae5905a4fa8889f384f41940a9f59d25fece91..0d7215ff154f93ea2a3fd1a71ae706de0217d8de 100644 |
--- a/media/mojo/services/renderer_config_default.cc |
+++ b/media/mojo/services/renderer_config_default.cc |
@@ -19,12 +19,19 @@ namespace internal { |
class DefaultRendererConfig : public PlatformRendererConfig { |
public: |
DefaultRendererConfig() { |
+ // TODO(dalecurtis): This will not work if the process is sandboxed... |
if (!media::IsMediaLibraryInitialized()) { |
base::FilePath module_dir; |
CHECK(PathService::Get(base::DIR_EXE, &module_dir)); |
CHECK(media::InitializeMediaLibrary(module_dir)); |
} |
+ // TODO(dalecurtis): We should find a single owner per process for the audio |
+ // manager or make it a lazy instance. It's not safe to call Get()/Create() |
+ // across multiple threads... |
+ // |
+ // TODO(dalecurtis): Eventually we'll want something other than a fake audio |
+ // log factory here too. We should probably at least DVLOG() such info. |
AudioManager* audio_manager = AudioManager::Get(); |
if (!audio_manager) |
audio_manager = media::AudioManager::Create(&fake_audio_log_factory_); |