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

webrtc/call/flexfec_receive_stream_impl.cc

Index: webrtc/call/flexfec_receive_stream_impl.cc
diff --git a/webrtc/call/flexfec_receive_stream_impl.cc b/webrtc/call/flexfec_receive_stream_impl.cc
index a1e9bc64ecccb2f70525812bc564d575f6161a96..1f1dcacfaba68a96064402d6fc742ab9c54d91c6 100644
--- a/webrtc/call/flexfec_receive_stream_impl.cc
+++ b/webrtc/call/flexfec_receive_stream_impl.cc
@@ -168,11 +168,8 @@ FlexfecReceiveStreamImpl::~FlexfecReceiveStreamImpl() {
 }
 
 void FlexfecReceiveStreamImpl::OnRtpPacket(const RtpPacketReceived& packet) {
-  {
-    rtc::CritScope cs(&crit_);
-    if (!started_)
-      return;
-  }
+  if (!started_)

[danilchap, 2017/07/24 13:40:23]

prefer .load() over operator bool to be explicit it is an atomic

[eladalon, 2017/07/24 16:09:02]

Done.

+    return;
 
   if (!receiver_)
     return;
@@ -191,12 +188,10 @@ void FlexfecReceiveStreamImpl::OnRtpPacket(const RtpPacketReceived& packet) {
 }
 
 void FlexfecReceiveStreamImpl::Start() {
-  rtc::CritScope cs(&crit_);
   started_ = true;

[danilchap, 2017/07/24 13:40:23]

same here: prefer store

[eladalon, 2017/07/24 16:09:02]

Done.

 }
 
 void FlexfecReceiveStreamImpl::Stop() {
-  rtc::CritScope cs(&crit_);
   started_ = false;
 }