Make FlexfecReceiveStreamImpl::started_ into std::atomic<bool>

webrtc/call/flexfec_receive_stream_impl.h

Index: webrtc/call/flexfec_receive_stream_impl.h
diff --git a/webrtc/call/flexfec_receive_stream_impl.h b/webrtc/call/flexfec_receive_stream_impl.h
index 81c09d6fac79b3c91d2f0ad7f694310bc73d5590..408ffe825f46080d1d5c903554a688402c6ff030 100644
--- a/webrtc/call/flexfec_receive_stream_impl.h
+++ b/webrtc/call/flexfec_receive_stream_impl.h
@@ -11,6 +11,7 @@
 #ifndef WEBRTC_CALL_FLEXFEC_RECEIVE_STREAM_IMPL_H_
 #define WEBRTC_CALL_FLEXFEC_RECEIVE_STREAM_IMPL_H_
 
+#include <atomic>
 #include <memory>
 
 #include "webrtc/call/flexfec_receive_stream.h"
@@ -53,8 +54,7 @@ class FlexfecReceiveStreamImpl : public FlexfecReceiveStream,
  private:
   // Config.
   const Config config_;
-  bool started_ GUARDED_BY(crit_);
-  rtc::CriticalSection crit_;
+  std::atomic<bool> started_;

[brandtr, 2017/08/01 07:08:02]

The start/stop functionality doesn't really serve a purpose for the
FlexfecReceiveStream. I have discussed this with nisse, and he is planning on
removing it completely.

[eladalon, 2017/08/01 08:27:46]

Okay, thanks.
In the meantime, do you think should I scrap the CL, or that I should land it?

[brandtr, 2017/08/01 08:39:03]

Might as well land it, since it's a nice improvement and you have done the work
already :)

 
   // Erasure code interfacing.
   const std::unique_ptr<FlexfecReceiver> receiver_;