Now run EndToEndTest with the WebRTC-NewVideoJitterBuffer experiment.

webrtc/modules/video_coding/rtp_frame_reference_finder.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 25 matching lines @@
   // If we have cleared past this frame, drop it.
   if (cleared_to_seq_num_ != -1 &&
       AheadOf<uint16_t>(cleared_to_seq_num_, frame->first_seq_num())) {
     return;
   }
 
   switch (frame->codec_type()) {
     case kVideoCodecFlexfec:
     case kVideoCodecULPFEC:
     case kVideoCodecRED:
-    case kVideoCodecUnknown:
       RTC_NOTREACHED();
       break;
     case kVideoCodecVP8:
       ManageFrameVp8(std::move(frame));
       break;
     case kVideoCodecVP9:
       ManageFrameVp9(std::move(frame));
       break;
+    case kVideoCodecUnknown:

[sprang_webrtc, 2016/11/21 16:49:14]

Isn't unknown separate from generic? Can we handle it?

[philipel, 2016/11/22 14:56:23]

I treat it as generic since that is what is used with the current jitter buffer
in the end to end tests. Added a todo to take a look at this.

     case kVideoCodecH264:
     case kVideoCodecI420:
     case kVideoCodecGeneric:
       ManageFrameGeneric(std::move(frame), kNoPictureId);
       break;
   }
 }
 
 void RtpFrameReferenceFinder::PaddingReceived(uint16_t seq_num) {
   rtc::CritScope lock(&crit_);
@@ skipping 660 matching lines @@
     if (!gof_info_.empty() &&
         AheadOf<uint8_t>(gof_info_.begin()->first, fixed_tl0)) {
       return true;
     }
   }
   return false;
 }
 
 }  // namespace video_coding
 }  // namespace webrtc