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

Unified Diff: content/renderer/media/webrtc_audio_device_impl.cc

Issue 12310102: Change GetVolume() to be asynchronous when being called by pulse thread (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updated the comments Created 7 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | media/audio/pulse/pulse.sigs » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/webrtc_audio_device_impl.cc
diff --git a/content/renderer/media/webrtc_audio_device_impl.cc b/content/renderer/media/webrtc_audio_device_impl.cc
index 32a5e49cd4e649e0023c26a4a5b6d6906d890c7f..64ae487fe7494c2ce56e2efbea7761d6b5a466aa 100644
--- a/content/renderer/media/webrtc_audio_device_impl.cc
+++ b/content/renderer/media/webrtc_audio_device_impl.cc
@@ -144,8 +144,8 @@ void WebRtcAudioDeviceImpl::CaptureData(const int16* audio_data,
// "higher than maximum". The input volume slider in the sound preference
// allows the user to set a scaling that is higher than 100%. It means that
// even if the reported maximum levels is N, the actual microphone level can
- // go up to 1.5*N and that corresponds to a normalized |volume| of 1.5.
- DCHECK_LE(volume, 1.5);
+ // go up to 1.5x*N and that corresponds to a normalized |volume| of 1.5x.
+ DCHECK_LE(volume, 1.6);
#endif
int output_delay_ms = 0;
« no previous file with comments | « no previous file | media/audio/pulse/pulse.sigs » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698