Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(227)

Side by Side Diff: webrtc/modules/video_coding/rtp_frame_reference_finder.cc

Issue 2522493002: Now run EndToEndTest with the WebRTC-NewVideoJitterBuffer experiment. (Closed)
Patch Set: Rebase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 25 matching lines...) Expand all
36 // If we have cleared past this frame, drop it. 36 // If we have cleared past this frame, drop it.
37 if (cleared_to_seq_num_ != -1 && 37 if (cleared_to_seq_num_ != -1 &&
38 AheadOf<uint16_t>(cleared_to_seq_num_, frame->first_seq_num())) { 38 AheadOf<uint16_t>(cleared_to_seq_num_, frame->first_seq_num())) {
39 return; 39 return;
40 } 40 }
41 41
42 switch (frame->codec_type()) { 42 switch (frame->codec_type()) {
43 case kVideoCodecFlexfec: 43 case kVideoCodecFlexfec:
44 case kVideoCodecULPFEC: 44 case kVideoCodecULPFEC:
45 case kVideoCodecRED: 45 case kVideoCodecRED:
46 case kVideoCodecUnknown:
47 RTC_NOTREACHED(); 46 RTC_NOTREACHED();
48 break; 47 break;
49 case kVideoCodecVP8: 48 case kVideoCodecVP8:
50 ManageFrameVp8(std::move(frame)); 49 ManageFrameVp8(std::move(frame));
51 break; 50 break;
52 case kVideoCodecVP9: 51 case kVideoCodecVP9:
53 ManageFrameVp9(std::move(frame)); 52 ManageFrameVp9(std::move(frame));
54 break; 53 break;
54 // Since the EndToEndTests use kVicdeoCodecUnknow we treat it the same as
55 // kVideoCodecGeneric.
56 // TODO(philipel): Take a look at the EndToEndTests and see if maybe they
57 // should be changed to use kVideoCodecGeneric instead.
58 case kVideoCodecUnknown:
55 case kVideoCodecH264: 59 case kVideoCodecH264:
56 case kVideoCodecI420: 60 case kVideoCodecI420:
57 case kVideoCodecGeneric: 61 case kVideoCodecGeneric:
58 ManageFrameGeneric(std::move(frame), kNoPictureId); 62 ManageFrameGeneric(std::move(frame), kNoPictureId);
59 break; 63 break;
60 } 64 }
61 } 65 }
62 66
63 void RtpFrameReferenceFinder::PaddingReceived(uint16_t seq_num) { 67 void RtpFrameReferenceFinder::PaddingReceived(uint16_t seq_num) {
64 rtc::CritScope lock(&crit_); 68 rtc::CritScope lock(&crit_);
(...skipping 660 matching lines...) Expand 10 before | Expand all | Expand 10 after
725 if (!gof_info_.empty() && 729 if (!gof_info_.empty() &&
726 AheadOf<uint8_t>(gof_info_.begin()->first, fixed_tl0)) { 730 AheadOf<uint8_t>(gof_info_.begin()->first, fixed_tl0)) {
727 return true; 731 return true;
728 } 732 }
729 } 733 }
730 return false; 734 return false;
731 } 735 }
732 736
733 } // namespace video_coding 737 } // namespace video_coding
734 } // namespace webrtc 738 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/frame_object.cc ('k') | webrtc/modules/video_coding/rtp_frame_reference_finder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698