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

Issue 12911007: Add speaker on/off control on Android for WebRTC (Closed)

Created:
7 years, 9 months ago by wjia(left Chromium)
Modified:
7 years, 9 months ago
Reviewers:
CC:
chromium-reviews, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, jam, darin-cc_chromium.org
Visibility:
Public.

Description

Add speaker on/off control on Android for WebRTC Application listens to the headset plug intent, turns on or off speaker when webrtc is running. It will help devices with dual mics to improve audio quality. BUG=222394 Contributed by leozwang@chromium.org (https://chromiumcodereview.appspot.com/12974004/) reland https://src.chromium.org/viewvc/chrome?view=rev&revision=189870 with unit tests fixed. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189983

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+156 lines, -73 lines) Patch
M content/browser/renderer_host/media/audio_input_device_manager_unittest.cc View 2 chunks +10 lines, -0 lines 0 comments Download
M content/test/webrtc_audio_device_test.cc View 2 chunks +10 lines, -0 lines 0 comments Download
M media/audio/audio_input_controller_unittest.cc View 6 chunks +16 lines, -30 lines 0 comments Download
M media/audio/audio_input_device_unittest.cc View 2 chunks +11 lines, -2 lines 0 comments Download
M media/audio/audio_input_unittest.cc View 6 chunks +14 lines, -26 lines 0 comments Download
M media/audio/audio_manager_base.h View 2 chunks +9 lines, -0 lines 0 comments Download
M media/audio/audio_manager_base.cc View 5 chunks +36 lines, -10 lines 0 comments Download
M media/base/android/java/src/org/chromium/media/AudioManagerAndroid.java View 2 chunks +50 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
wjia(left Chromium)
7 years, 9 months ago (2013-03-23 03:49:02 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 manually as r189983.

Powered by Google App Engine
This is Rietveld 408576698