NetEq: Implement logging of Delayed Packet Outage Events

webrtc/modules/audio_coding/neteq/neteq_impl.cc

Index: webrtc/modules/audio_coding/neteq/neteq_impl.cc
diff --git a/webrtc/modules/audio_coding/neteq/neteq_impl.cc b/webrtc/modules/audio_coding/neteq/neteq_impl.cc
index 3b81999dbba042d4991042a203f5de4b2076a39e..636ae876322c70f7cd4425a308eff819e92e342d 100644
--- a/webrtc/modules/audio_coding/neteq/neteq_impl.cc
+++ b/webrtc/modules/audio_coding/neteq/neteq_impl.cc
@@ -1874,7 +1874,7 @@ void NetEqImpl::UpdatePlcComponents(int fs_hz, size_t channels) {
   // Delete objects and create new ones.
   expand_.reset(expand_factory_->Create(background_noise_.get(),
                                         sync_buffer_.get(), &random_vector_,
-                                        fs_hz, channels));
+                                        &stats_, fs_hz, channels));

[minyue-webrtc, 2015/08/17 14:07:22]

you share the pointer to stats_, how to prevent data race?

[hlundin-webrtc, 2015/08/17 14:10:31]

All of NetEq is locked down with the same critsect. So, effectively, only one
thread at a time is running, and no call will be interrupted before complete.

[minyue-webrtc, 2015/08/17 14:16:08]

Fair enough, although this type of sharing does not feel like a perfect
practice.

[hlundin-webrtc, 2015/08/17 14:23:01]

I think this is an example of dependency injection, which we use in many places.

   merge_.reset(new Merge(fs_hz, channels, expand_.get(), sync_buffer_.get()));
 }