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

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

Issue 178153007: Avoid hitting the thread check when WebRtcAudioRenderer is going away. (Closed) Base URL: http://git.chromium.org/chromium/src.git@libjingle_get_stats
Patch Set: Created 6 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
Index: content/renderer/media/media_stream_audio_processor.cc
diff --git a/content/renderer/media/media_stream_audio_processor.cc b/content/renderer/media/media_stream_audio_processor.cc
index 7ae7d26f6d79b1c596d01b56306bc235a9e7c7ab..07021f9ecd81e4b9d9369f64c45419c5c18b7c6d 100644
--- a/content/renderer/media/media_stream_audio_processor.cc
+++ b/content/renderer/media/media_stream_audio_processor.cc
@@ -211,6 +211,11 @@ void MediaStreamAudioProcessor::OnPlayoutData(media::AudioBus* audio_bus,
audio_processing_->AnalyzeReverseStream(&render_frame_);
}
+void MediaStreamAudioProcessor::OnPlayoutDataSourceChanged() {
+ render_thread_checker_.DetachFromThread();
tommi (sloooow) - chröme 2014/03/04 20:48:39 add some documentation here that explains what thr
no longer working on chromium 2014/03/05 13:01:04 Done, I added a thread check here as well.
+ render_converter_.reset();
+}
+
void MediaStreamAudioProcessor::GetStats(AudioProcessorStats* stats) {
stats->typing_noise_detected =
(base::subtle::Acquire_Load(&typing_detected_) != false);

Powered by Google App Engine
This is Rietveld 408576698