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

Issue 3009193002: Improves stereo/mono audio support on Android (Closed)

Created:
3 years, 3 months ago by henrika_webrtc
Modified:
3 years, 3 months ago
Reviewers:
hlundin-webrtc
CC:
webrtc-reviews_webrtc.org
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Improves stereo/mono audio support on Android. Fixes some issues related to calling WebRtcAudioManager.setStereoOutput(true) and WebRtcAudioManager.setStereoInput(true) and ensures that the ADM reports correct values related to stereo support given these settings. Also makes it more clear that the OpenSLES audio implementation does not support stereo (we now fail in Init()). To summarize: this change ensures that the user can ask for stereo input and/or stereo output audio on Android in combination with the Java based audio layer. By default (if no WebRtcAudioManager.setStereoXXX() APIs are called), mono will be used. BUG=webrtc:7962 Review-Url: https://codereview.webrtc.org/3009193002 Cr-Commit-Position: refs/heads/master@{#19763} Committed: https://chromium.googlesource.com/external/webrtc/+/76535de14fa3d823b30f015d76b535887699dc8f

Patch Set 1 #

Patch Set 2 : improved comments #

Total comments: 2

Patch Set 3 : nit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -14 lines) Patch
M webrtc/modules/audio_device/android/audio_device_template.h View 1 2 1 chunk +19 lines, -12 lines 0 comments Download
M webrtc/modules/audio_device/android/audio_manager.h View 1 1 chunk +8 lines, -0 lines 0 comments Download
M webrtc/modules/audio_device/android/audio_manager.cc View 1 chunk +12 lines, -0 lines 0 comments Download
M webrtc/modules/audio_device/android/audio_manager_unittest.cc View 1 chunk +10 lines, -0 lines 0 comments Download
M webrtc/modules/audio_device/android/opensles_player.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M webrtc/modules/audio_device/android/opensles_recorder.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M webrtc/modules/audio_device/audio_device_buffer.cc View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 18 (14 generated)
henrika_webrtc
FYI: linux_memcheck trybot seems flaky. Not related to this CL.
3 years, 3 months ago (2017-09-07 14:27:08 UTC) #7
hlundin-webrtc
LGTM % typos https://codereview.webrtc.org/3009193002/diff/20001/webrtc/modules/audio_device/android/audio_device_template.h File webrtc/modules/audio_device/android/audio_device_template.h (right): https://codereview.webrtc.org/3009193002/diff/20001/webrtc/modules/audio_device/android/audio_device_template.h#newcode346 webrtc/modules/audio_device/android/audio_device_template.h:346: // Instead, the native audio layer ...
3 years, 3 months ago (2017-09-11 07:43:42 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/3009193002/40001
3 years, 3 months ago (2017-09-11 07:56:03 UTC) #15
commit-bot: I haz the power
3 years, 3 months ago (2017-09-11 08:26:01 UTC) #18
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/76535de14fa3d823b30f015d7...

Powered by Google App Engine
This is Rietveld 408576698