New jitter buffer experiment.

webrtc/modules/video_coding/packet_buffer.cc

Index: webrtc/modules/video_coding/packet_buffer.cc
diff --git a/webrtc/modules/video_coding/packet_buffer.cc b/webrtc/modules/video_coding/packet_buffer.cc
index 58ad31e50cefda7c959cdeffb08d22b61e795c94..351d0d2e8003cf86e245adc51be5c7a4f6bd70d9 100644
--- a/webrtc/modules/video_coding/packet_buffer.cc
+++ b/webrtc/modules/video_coding/packet_buffer.cc
@@ -219,7 +219,9 @@ void PacketBuffer::FindFrames(uint16_t seq_num) {
           new RtpFrameObject(this, start_seq_num, seq_num, frame_size,
                              max_nack_count, clock_->TimeInMilliseconds()));
 
+      crit_.Leave();
       received_frame_callback_->OnReceivedFrame(std::move(frame));
+      crit_.Enter();

[stefan-webrtc, 2016/11/08 10:41:34]

I think we should rewrite the code so that we can properly use scoped crits. For
instance you can return a list of frames to be sent out from this method and
call the callback from InsertPacket() after you have left the crit.

[philipel, 2016/11/08 12:28:39]

Done.

     }
 
     ++seq_num;
@@ -267,7 +269,6 @@ bool PacketBuffer::GetBitstream(const RtpFrameObject& frame,
 }
 
 VCMPacket* PacketBuffer::GetPacket(uint16_t seq_num) {
-  rtc::CritScope lock(&crit_);
   size_t index = seq_num % size_;
   if (!sequence_buffer_[index].used ||
       seq_num != sequence_buffer_[index].seq_num) {