Stop media stream source when audio capture error occurs.

content/renderer/media/local_media_stream_audio_source.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/local_media_stream_audio_source.h"
 
+#include "base/memory/weak_ptr.h"
+#include "base/single_thread_task_runner.h"
 #include "content/common/media/media_stream_options.h"
 #include "content/renderer/media/audio_device_factory.h"
 #include "content/renderer/render_frame_impl.h"
 
 namespace content {
 
 LocalMediaStreamAudioSource::LocalMediaStreamAudioSource(
     int consumer_render_frame_id,
     const StreamDeviceInfo& device_info)
     : MediaStreamAudioSource(true /* is_local_source */),
-      consumer_render_frame_id_(consumer_render_frame_id) {
+      consumer_render_frame_id_(consumer_render_frame_id),
+      task_runner_(base::ThreadTaskRunnerHandle::Get()) {
   DVLOG(1) << "LocalMediaStreamAudioSource::LocalMediaStreamAudioSource()";
   MediaStreamSource::SetDeviceInfo(device_info);
 
   // If the device buffer size was not provided, use a default.
   int frames_per_buffer = device_info.device.input.frames_per_buffer;
   if (frames_per_buffer <= 0) {
     // TODO(miu): Like in ProcessedLocalAudioSource::GetBufferSize(), we should
     // re-evaluate whether Android needs special treatment here. Or, perhaps we
     // should just DCHECK_GT(device_info...frames_per_buffer, 0)?
     // http://crbug.com/638081
@@ skipping 64 matching lines @@
   // TODO(miu): Plumbing is needed to determine the actual capture timestamp
   // of the audio, instead of just snapshotting TimeTicks::Now(), for proper
   // audio/video sync. http://crbug.com/335335
   MediaStreamAudioSource::DeliverDataToTracks(
       *audio_bus,
       base::TimeTicks::Now() -
       base::TimeDelta::FromMilliseconds(audio_delay_milliseconds));
 }
 
 void LocalMediaStreamAudioSource::OnCaptureError(const std::string& why) {
-  // As of this writing, this method doesn't get called for anything useful,
-  // and all other implementors just log the message, but don't disconnect sinks
-  // or take any other action. So, just log the error.
-  LOG(ERROR) << why;
+  VLOG(1) << why;
+
+  // Stop source when error occurs.
+  task_runner_->PostTask(
+      FROM_HERE, base::Bind(&MediaStreamSource::StopSource, AsWeakPtr()));

[miu, 2016/10/14 05:38:26]

Why do we need the task runner? Isn't OnCaptureError() always called on the main
thread?

Also, I'm not sure it's a good idea to close the source here. Instead, the
onEnded event should be sent via the tracks. 

[xjz, 2016/10/14 17:32:53]

When tab is closed, this is called on io thread, not main thread. 
When calling MediaStreamSource::StopSource, it will call setReadyState() and the
track will send onEnded event. I did this in the similar way as that is done for
video capture source. Not sure if this is the best way either.

 }
 
 }  // namespace content