mirror of
https://github.com/mollyim/webrtc.git
synced 2025-05-19 08:37:54 +01:00
eb8c4ca608
This removes some code in the AudioDeviceWindowsCore::CoreAudioIsSupported function that was checking that every audio input and output device was functional. There are legitimate cases where some, or all, audio devices may not be accessible, and that was causing CoreAudioIsSupported to return false. If CoreAudioIsSupported returns false, a subsequent RTC_CHECK call fails, which causes the entire app to exit. After this change, the CoreAudioIsSupported() function simply checks if the Core Audio APIs are supported and no longer tries to do extra stuff unrelated to checking if the APIs are supported. Note that Core Audio is actually supported in all versions of Windows after Windows XP. There were log messages in the code saying that if CoreAudioIsSupported() returns false, WebRTC will use the Wave Audio APIs instead. But this is no longer the case. The Wave Audio APIs would only be needed for Windows XP, and this code appears to have already been removed from WebRTC. It is tempting to simply make CoreAudioIsSupported() do a "return true;" but for now I only removed the part of the logging messages that mentioned the Wave Audio APIs. I understand that there is a new Audio Device Module (ADM) called WindowsCoreAudio2, which is now recommended for use by apps. Apps are supposed to instantiate WindowsCoreAudio2 and pass it in to WebRTC. When the app supplies its own ADM, CoreAudioIsSupported() does not get invoked, which avoids the bug. To help make it clearer that using WindowsCoreAudio2 is an acceptable solution, I am removing a comment that says that kWindowsCoreAudio2 is "experimental". Bug: webrtc:11081 Change-Id: I7ed1684a276799f4c83006b45629e48814f0b18b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/161463 Commit-Queue: Henrik Andreassson <henrika@webrtc.org> Reviewed-by: Henrik Andreassson <henrika@webrtc.org> Cr-Commit-Position: refs/heads/master@{#30025} |
||
|---|---|---|
| .. | ||
| android | ||
| dummy | ||
| include | ||
| linux | ||
| mac | ||
| win | ||
| audio_device_buffer.cc | ||
| audio_device_buffer.h | ||
| audio_device_config.h | ||
| audio_device_data_observer.cc | ||
| audio_device_generic.cc | ||
| audio_device_generic.h | ||
| audio_device_impl.cc | ||
| audio_device_impl.h | ||
| audio_device_name.cc | ||
| audio_device_name.h | ||
| audio_device_unittest.cc | ||
| BUILD.gn | ||
| DEPS | ||
| fine_audio_buffer.cc | ||
| fine_audio_buffer.h | ||
| fine_audio_buffer_unittest.cc | ||
| mock_audio_device_buffer.h | ||
| OWNERS | ||