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

Unified Diff: content/renderer/media/webrtc/processed_local_audio_source.cc

Issue 2219933003: Use LocalMediaStreamAudioSource for screen-casting use cases. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments on patch set 1 (mostly rewrote comments). Created 4 years, 4 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
Index: content/renderer/media/webrtc/processed_local_audio_source.cc
diff --git a/content/renderer/media/webrtc/processed_local_audio_source.cc b/content/renderer/media/webrtc/processed_local_audio_source.cc
index b25469372a3625d563595da73736689b78ec2e44..c2a13553f9942e007cae0ed42d914148ceaa63b5 100644
--- a/content/renderer/media/webrtc/processed_local_audio_source.cc
+++ b/content/renderer/media/webrtc/processed_local_audio_source.cc
@@ -350,6 +350,7 @@ int ProcessedLocalAudioSource::GetBufferSize(int sample_rate) const {
DCHECK(thread_checker_.CalledOnValidThread());
#if defined(OS_ANDROID)
// TODO(henrika): Re-evaluate whether to use same logic as other platforms.
+ // http://crbug.com/638081
return (2 * sample_rate / 100);
#endif
@@ -367,7 +368,7 @@ int ProcessedLocalAudioSource::GetBufferSize(int sample_rate) const {
// fall-back.
//
// TODO(miu): Identify where/why the buffer size might be missing, fix the
- // code, and then require it here.
+ // code, and then require it here. http://crbug.com/638081
return (sample_rate / 100);
}

Powered by Google App Engine
This is Rietveld 408576698