Avoid race in VideoReceiveStream shutdown

webrtc/video/video_receive_stream.cc

Index: webrtc/video/video_receive_stream.cc
diff --git a/webrtc/video/video_receive_stream.cc b/webrtc/video/video_receive_stream.cc
index ad9beb35e75384801cf03d264f935a7137a9b383..4a1c55dd0971535cd30ef26866fbd713b3f4b634 100644
--- a/webrtc/video/video_receive_stream.cc
+++ b/webrtc/video/video_receive_stream.cc
@@ -267,8 +267,8 @@ void VideoReceiveStream::Start() {
 
 void VideoReceiveStream::Stop() {
   rtp_stream_receiver_.StopReceive();
-  video_receiver_.TriggerDecoderShutdown();
   decode_thread_.Stop();

[pbos-webrtc, 2016/07/13 13:06:35]

That's harder than this. TriggerDecoderShutdown() is what's causing
decode_thread_ to stop faster. With this change TriggerDecoderShutdown doesn't
make sense since it won't make any thread stop faster.

[sprang_webrtc, 2016/07/13 16:17:06]

You're right. I've added a comment to that effect here.

+  video_receiver_.TriggerDecoderShutdown();
   call_stats_->DeregisterStatsObserver(video_stream_decoder_.get());
   video_stream_decoder_.reset();
   incoming_video_stream_.reset();