philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. |
| 3 | * |
| 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 |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
| 9 | */ |
| 10 | |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 11 | #include "modules/video_coding/packet_buffer.h" |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 12 | |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 13 | #include <string.h> |
Jonas Olsson | a4d8737 | 2019-07-05 19:08:33 +0200 | [diff] [blame] | 14 | |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 15 | #include <algorithm> |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 16 | #include <cstdint> |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 17 | #include <limits> |
philipel | 17deeb4 | 2016-08-11 15:09:26 +0200 | [diff] [blame] | 18 | #include <utility> |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 19 | #include <vector> |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 20 | |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 21 | #include "absl/types/variant.h" |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 22 | #include "api/array_view.h" |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 23 | #include "api/rtp_packet_info.h" |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 24 | #include "api/video/encoded_frame.h" |
Danil Chapovalov | 0682ca9 | 2019-11-28 16:50:02 +0100 | [diff] [blame^] | 25 | #include "api/video/video_frame_type.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 26 | #include "common_video/h264/h264_common.h" |
Danil Chapovalov | 0682ca9 | 2019-11-28 16:50:02 +0100 | [diff] [blame^] | 27 | #include "modules/rtp_rtcp/source/rtp_depacketizer_av1.h" |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 28 | #include "modules/rtp_rtcp/source/rtp_header_extensions.h" |
| 29 | #include "modules/rtp_rtcp/source/rtp_packet_received.h" |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 30 | #include "modules/rtp_rtcp/source/rtp_video_header.h" |
| 31 | #include "modules/video_coding/codecs/h264/include/h264_globals.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 32 | #include "modules/video_coding/frame_object.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 33 | #include "rtc_base/checks.h" |
| 34 | #include "rtc_base/logging.h" |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 35 | #include "rtc_base/numerics/mod_ops.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 36 | #include "system_wrappers/include/clock.h" |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 37 | #include "system_wrappers/include/field_trial.h" |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 38 | |
| 39 | namespace webrtc { |
| 40 | namespace video_coding { |
| 41 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 42 | PacketBuffer::Packet::Packet(const RtpPacketReceived& rtp_packet, |
| 43 | const RTPVideoHeader& video_header, |
| 44 | int64_t ntp_time_ms, |
| 45 | int64_t receive_time_ms) |
| 46 | : marker_bit(rtp_packet.Marker()), |
| 47 | payload_type(rtp_packet.PayloadType()), |
| 48 | seq_num(rtp_packet.SequenceNumber()), |
| 49 | timestamp(rtp_packet.Timestamp()), |
| 50 | ntp_time_ms(ntp_time_ms), |
| 51 | times_nacked(-1), |
| 52 | video_header(video_header), |
| 53 | packet_info(rtp_packet.Ssrc(), |
| 54 | rtp_packet.Csrcs(), |
| 55 | rtp_packet.Timestamp(), |
| 56 | /*audio_level=*/absl::nullopt, |
| 57 | rtp_packet.GetExtension<AbsoluteCaptureTimeExtension>(), |
| 58 | receive_time_ms) {} |
| 59 | |
philipel | b4d3108 | 2016-07-11 08:46:29 -0700 | [diff] [blame] | 60 | PacketBuffer::PacketBuffer(Clock* clock, |
| 61 | size_t start_buffer_size, |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 62 | size_t max_buffer_size) |
philipel | b4d3108 | 2016-07-11 08:46:29 -0700 | [diff] [blame] | 63 | : clock_(clock), |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 64 | max_size_(max_buffer_size), |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 65 | first_seq_num_(0), |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 66 | first_packet_received_(false), |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 67 | is_cleared_to_first_seq_num_(false), |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 68 | buffer_(start_buffer_size), |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 69 | sps_pps_idr_is_h264_keyframe_( |
| 70 | field_trial::IsEnabled("WebRTC-SpsPpsIdrIsH264Keyframe")) { |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 71 | RTC_DCHECK_LE(start_buffer_size, max_buffer_size); |
| 72 | // Buffer size must always be a power of 2. |
| 73 | RTC_DCHECK((start_buffer_size & (start_buffer_size - 1)) == 0); |
| 74 | RTC_DCHECK((max_buffer_size & (max_buffer_size - 1)) == 0); |
| 75 | } |
| 76 | |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 77 | PacketBuffer::~PacketBuffer() { |
| 78 | Clear(); |
| 79 | } |
philipel | 17deeb4 | 2016-08-11 15:09:26 +0200 | [diff] [blame] | 80 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 81 | PacketBuffer::InsertResult PacketBuffer::InsertPacket( |
| 82 | PacketBuffer::Packet* packet) { |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 83 | PacketBuffer::InsertResult result; |
| 84 | rtc::CritScope lock(&crit_); |
philipel | 3184f8e | 2017-05-18 08:08:53 -0700 | [diff] [blame] | 85 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 86 | uint16_t seq_num = packet->seq_num; |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 87 | size_t index = seq_num % buffer_.size(); |
Ilya Nikolaevskiy | d397a0d | 2018-02-21 15:57:09 +0100 | [diff] [blame] | 88 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 89 | if (!first_packet_received_) { |
| 90 | first_seq_num_ = seq_num; |
| 91 | first_packet_received_ = true; |
| 92 | } else if (AheadOf(first_seq_num_, seq_num)) { |
| 93 | // If we have explicitly cleared past this packet then it's old, |
| 94 | // don't insert it, just silently ignore it. |
| 95 | if (is_cleared_to_first_seq_num_) { |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 96 | delete[] packet->data; |
| 97 | packet->data = nullptr; |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 98 | return result; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 99 | } |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 100 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 101 | first_seq_num_ = seq_num; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 102 | } |
| 103 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 104 | if (buffer_[index].used) { |
| 105 | // Duplicate packet, just delete the payload. |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 106 | if (buffer_[index].seq_num() == packet->seq_num) { |
| 107 | delete[] packet->data; |
| 108 | packet->data = nullptr; |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 109 | return result; |
| 110 | } |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 111 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 112 | // The packet buffer is full, try to expand the buffer. |
| 113 | while (ExpandBufferSize() && buffer_[seq_num % buffer_.size()].used) { |
| 114 | } |
| 115 | index = seq_num % buffer_.size(); |
| 116 | |
| 117 | // Packet buffer is still full since we were unable to expand the buffer. |
| 118 | if (buffer_[index].used) { |
| 119 | // Clear the buffer, delete payload, and return false to signal that a |
| 120 | // new keyframe is needed. |
| 121 | RTC_LOG(LS_WARNING) << "Clear PacketBuffer and request key frame."; |
| 122 | Clear(); |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 123 | delete[] packet->data; |
| 124 | packet->data = nullptr; |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 125 | result.buffer_cleared = true; |
| 126 | return result; |
| 127 | } |
| 128 | } |
| 129 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 130 | int64_t now_ms = clock_->TimeInMilliseconds(); |
| 131 | last_received_packet_ms_ = now_ms; |
| 132 | if (packet->video_header.frame_type == VideoFrameType::kVideoFrameKey) |
| 133 | last_received_keyframe_packet_ms_ = now_ms; |
| 134 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 135 | StoredPacket& new_entry = buffer_[index]; |
| 136 | new_entry.continuous = false; |
| 137 | new_entry.used = true; |
| 138 | new_entry.data = std::move(*packet); |
| 139 | packet->data = nullptr; |
| 140 | |
| 141 | UpdateMissingPackets(seq_num); |
| 142 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 143 | result.frames = FindFrames(seq_num); |
| 144 | return result; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 145 | } |
| 146 | |
| 147 | void PacketBuffer::ClearTo(uint16_t seq_num) { |
| 148 | rtc::CritScope lock(&crit_); |
philipel | c5fb468 | 2017-08-02 04:28:57 -0700 | [diff] [blame] | 149 | // We have already cleared past this sequence number, no need to do anything. |
| 150 | if (is_cleared_to_first_seq_num_ && |
| 151 | AheadOf<uint16_t>(first_seq_num_, seq_num)) { |
| 152 | return; |
| 153 | } |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 154 | |
| 155 | // If the packet buffer was cleared between a frame was created and returned. |
| 156 | if (!first_packet_received_) |
| 157 | return; |
| 158 | |
philipel | c5fb468 | 2017-08-02 04:28:57 -0700 | [diff] [blame] | 159 | // Avoid iterating over the buffer more than once by capping the number of |
| 160 | // iterations to the |size_| of the buffer. |
| 161 | ++seq_num; |
| 162 | size_t diff = ForwardDiff<uint16_t>(first_seq_num_, seq_num); |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 163 | size_t iterations = std::min(diff, buffer_.size()); |
philipel | c5fb468 | 2017-08-02 04:28:57 -0700 | [diff] [blame] | 164 | for (size_t i = 0; i < iterations; ++i) { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 165 | size_t index = first_seq_num_ % buffer_.size(); |
| 166 | if (AheadOf<uint16_t>(seq_num, buffer_[index].seq_num())) { |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 167 | delete[] buffer_[index].data.data; |
| 168 | buffer_[index].data.data = nullptr; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 169 | buffer_[index].used = false; |
philipel | c5fb468 | 2017-08-02 04:28:57 -0700 | [diff] [blame] | 170 | } |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 171 | ++first_seq_num_; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 172 | } |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 173 | |
philipel | c5fb468 | 2017-08-02 04:28:57 -0700 | [diff] [blame] | 174 | // If |diff| is larger than |iterations| it means that we don't increment |
| 175 | // |first_seq_num_| until we reach |seq_num|, so we set it here. |
| 176 | first_seq_num_ = seq_num; |
| 177 | |
| 178 | is_cleared_to_first_seq_num_ = true; |
philipel | bc5a408 | 2017-12-06 10:41:08 +0100 | [diff] [blame] | 179 | auto clear_to_it = missing_packets_.upper_bound(seq_num); |
| 180 | if (clear_to_it != missing_packets_.begin()) { |
| 181 | --clear_to_it; |
| 182 | missing_packets_.erase(missing_packets_.begin(), clear_to_it); |
| 183 | } |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 184 | } |
| 185 | |
Johannes Kron | a370556 | 2019-08-26 16:37:11 +0200 | [diff] [blame] | 186 | void PacketBuffer::ClearInterval(uint16_t start_seq_num, |
| 187 | uint16_t stop_seq_num) { |
| 188 | size_t iterations = ForwardDiff<uint16_t>(start_seq_num, stop_seq_num + 1); |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 189 | RTC_DCHECK_LE(iterations, buffer_.size()); |
Johannes Kron | a370556 | 2019-08-26 16:37:11 +0200 | [diff] [blame] | 190 | uint16_t seq_num = start_seq_num; |
| 191 | for (size_t i = 0; i < iterations; ++i) { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 192 | size_t index = seq_num % buffer_.size(); |
| 193 | RTC_DCHECK_EQ(buffer_[index].seq_num(), seq_num); |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 194 | delete[] buffer_[index].data.data; |
| 195 | buffer_[index].data.data = nullptr; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 196 | buffer_[index].used = false; |
Johannes Kron | a370556 | 2019-08-26 16:37:11 +0200 | [diff] [blame] | 197 | |
| 198 | ++seq_num; |
| 199 | } |
| 200 | } |
| 201 | |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 202 | void PacketBuffer::Clear() { |
| 203 | rtc::CritScope lock(&crit_); |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 204 | for (StoredPacket& entry : buffer_) { |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 205 | delete[] entry.data.data; |
| 206 | entry.data.data = nullptr; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 207 | entry.used = false; |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 208 | } |
| 209 | |
| 210 | first_packet_received_ = false; |
| 211 | is_cleared_to_first_seq_num_ = false; |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 212 | last_received_packet_ms_.reset(); |
| 213 | last_received_keyframe_packet_ms_.reset(); |
| 214 | newest_inserted_seq_num_.reset(); |
| 215 | missing_packets_.clear(); |
| 216 | } |
| 217 | |
Danil Chapovalov | ce1ffcd | 2019-10-22 17:12:42 +0200 | [diff] [blame] | 218 | PacketBuffer::InsertResult PacketBuffer::InsertPadding(uint16_t seq_num) { |
| 219 | PacketBuffer::InsertResult result; |
| 220 | rtc::CritScope lock(&crit_); |
| 221 | UpdateMissingPackets(seq_num); |
| 222 | result.frames = FindFrames(static_cast<uint16_t>(seq_num + 1)); |
| 223 | return result; |
philipel | 3184f8e | 2017-05-18 08:08:53 -0700 | [diff] [blame] | 224 | } |
| 225 | |
Danil Chapovalov | 0040b66 | 2018-06-18 10:48:16 +0200 | [diff] [blame] | 226 | absl::optional<int64_t> PacketBuffer::LastReceivedPacketMs() const { |
philipel | 3184f8e | 2017-05-18 08:08:53 -0700 | [diff] [blame] | 227 | rtc::CritScope lock(&crit_); |
| 228 | return last_received_packet_ms_; |
| 229 | } |
| 230 | |
Danil Chapovalov | 0040b66 | 2018-06-18 10:48:16 +0200 | [diff] [blame] | 231 | absl::optional<int64_t> PacketBuffer::LastReceivedKeyframePacketMs() const { |
philipel | 3184f8e | 2017-05-18 08:08:53 -0700 | [diff] [blame] | 232 | rtc::CritScope lock(&crit_); |
| 233 | return last_received_keyframe_packet_ms_; |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 234 | } |
| 235 | |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 236 | bool PacketBuffer::ExpandBufferSize() { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 237 | if (buffer_.size() == max_size_) { |
Mirko Bonadei | 675513b | 2017-11-09 11:09:25 +0100 | [diff] [blame] | 238 | RTC_LOG(LS_WARNING) << "PacketBuffer is already at max size (" << max_size_ |
Johannes Kron | bd3f305 | 2019-08-01 15:45:54 +0200 | [diff] [blame] | 239 | << "), failed to increase size."; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 240 | return false; |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 241 | } |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 242 | |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 243 | size_t new_size = std::min(max_size_, 2 * buffer_.size()); |
| 244 | std::vector<StoredPacket> new_buffer(new_size); |
| 245 | for (StoredPacket& entry : buffer_) { |
| 246 | if (entry.used) { |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 247 | new_buffer[entry.seq_num() % new_size] = std::move(entry); |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 248 | } |
| 249 | } |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 250 | buffer_ = std::move(new_buffer); |
Mirko Bonadei | 675513b | 2017-11-09 11:09:25 +0100 | [diff] [blame] | 251 | RTC_LOG(LS_INFO) << "PacketBuffer size expanded to " << new_size; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 252 | return true; |
| 253 | } |
| 254 | |
philipel | aee3e0e | 2016-11-01 11:45:34 +0100 | [diff] [blame] | 255 | bool PacketBuffer::PotentialNewFrame(uint16_t seq_num) const { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 256 | size_t index = seq_num % buffer_.size(); |
| 257 | int prev_index = index > 0 ? index - 1 : buffer_.size() - 1; |
| 258 | const StoredPacket& entry = buffer_[index]; |
| 259 | const StoredPacket& prev_entry = buffer_[prev_index]; |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 260 | |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 261 | if (!entry.used) |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 262 | return false; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 263 | if (entry.seq_num() != seq_num) |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 264 | return false; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 265 | if (entry.frame_begin()) |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 266 | return true; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 267 | if (!prev_entry.used) |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 268 | return false; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 269 | if (prev_entry.seq_num() != static_cast<uint16_t>(entry.seq_num() - 1)) |
philipel | ea142f8 | 2017-01-11 02:01:56 -0800 | [diff] [blame] | 270 | return false; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 271 | if (prev_entry.data.timestamp != entry.data.timestamp) |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 272 | return false; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 273 | if (prev_entry.continuous) |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 274 | return true; |
| 275 | |
| 276 | return false; |
| 277 | } |
| 278 | |
philipel | fd5a20f | 2016-11-15 00:57:57 -0800 | [diff] [blame] | 279 | std::vector<std::unique_ptr<RtpFrameObject>> PacketBuffer::FindFrames( |
| 280 | uint16_t seq_num) { |
| 281 | std::vector<std::unique_ptr<RtpFrameObject>> found_frames; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 282 | for (size_t i = 0; i < buffer_.size() && PotentialNewFrame(seq_num); ++i) { |
| 283 | size_t index = seq_num % buffer_.size(); |
| 284 | buffer_[index].continuous = true; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 285 | |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 286 | // If all packets of the frame is continuous, find the first packet of the |
| 287 | // frame and create an RtpFrameObject. |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 288 | if (buffer_[index].frame_end()) { |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 289 | uint16_t start_seq_num = seq_num; |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 290 | |
philipel | 5ceaaae | 2016-05-24 10:20:47 +0200 | [diff] [blame] | 291 | // Find the start index by searching backward until the packet with |
| 292 | // the |frame_begin| flag is set. |
| 293 | int start_index = index; |
philipel | 227f8b9 | 2017-08-04 06:39:31 -0700 | [diff] [blame] | 294 | size_t tested_packets = 0; |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 295 | int64_t frame_timestamp = buffer_[start_index].data.timestamp; |
philipel | 5391071 | 2017-05-18 02:24:40 -0700 | [diff] [blame] | 296 | |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 297 | // Identify H.264 keyframes by means of SPS, PPS, and IDR. |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 298 | bool is_h264 = buffer_[start_index].data.codec() == kVideoCodecH264; |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 299 | bool has_h264_sps = false; |
| 300 | bool has_h264_pps = false; |
| 301 | bool has_h264_idr = false; |
| 302 | bool is_h264_keyframe = false; |
Shyam Sadhwani | 5b2df17 | 2019-10-16 09:13:38 -0700 | [diff] [blame] | 303 | int idr_width = -1; |
| 304 | int idr_height = -1; |
philipel | 227f8b9 | 2017-08-04 06:39:31 -0700 | [diff] [blame] | 305 | while (true) { |
| 306 | ++tested_packets; |
Ilya Nikolaevskiy | 4348ce2 | 2018-12-07 16:26:56 +0100 | [diff] [blame] | 307 | |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 308 | if (!is_h264 && buffer_[start_index].frame_begin()) |
philipel | 5ceaaae | 2016-05-24 10:20:47 +0200 | [diff] [blame] | 309 | break; |
| 310 | |
Shyam Sadhwani | 2b84dad | 2019-10-02 17:22:33 -0700 | [diff] [blame] | 311 | if (is_h264) { |
philipel | 7d745e5 | 2018-08-02 14:03:53 +0200 | [diff] [blame] | 312 | const auto* h264_header = absl::get_if<RTPVideoHeaderH264>( |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 313 | &buffer_[start_index].data.video_header.video_type_header); |
philipel | 7d745e5 | 2018-08-02 14:03:53 +0200 | [diff] [blame] | 314 | if (!h264_header || h264_header->nalus_length >= kMaxNalusPerPacket) |
philipel | 09133af | 2018-05-17 14:11:09 +0200 | [diff] [blame] | 315 | return found_frames; |
| 316 | |
philipel | 7d745e5 | 2018-08-02 14:03:53 +0200 | [diff] [blame] | 317 | for (size_t j = 0; j < h264_header->nalus_length; ++j) { |
| 318 | if (h264_header->nalus[j].type == H264::NaluType::kSps) { |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 319 | has_h264_sps = true; |
philipel | 7d745e5 | 2018-08-02 14:03:53 +0200 | [diff] [blame] | 320 | } else if (h264_header->nalus[j].type == H264::NaluType::kPps) { |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 321 | has_h264_pps = true; |
philipel | 7d745e5 | 2018-08-02 14:03:53 +0200 | [diff] [blame] | 322 | } else if (h264_header->nalus[j].type == H264::NaluType::kIdr) { |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 323 | has_h264_idr = true; |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 324 | } |
| 325 | } |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 326 | if ((sps_pps_idr_is_h264_keyframe_ && has_h264_idr && has_h264_sps && |
| 327 | has_h264_pps) || |
| 328 | (!sps_pps_idr_is_h264_keyframe_ && has_h264_idr)) { |
| 329 | is_h264_keyframe = true; |
Shyam Sadhwani | 5b2df17 | 2019-10-16 09:13:38 -0700 | [diff] [blame] | 330 | // Store the resolution of key frame which is the packet with |
| 331 | // smallest index and valid resolution; typically its IDR or SPS |
| 332 | // packet; there may be packet preceeding this packet, IDR's |
| 333 | // resolution will be applied to them. |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 334 | if (buffer_[start_index].data.width() > 0 && |
| 335 | buffer_[start_index].data.height() > 0) { |
| 336 | idr_width = buffer_[start_index].data.width(); |
| 337 | idr_height = buffer_[start_index].data.height(); |
Shyam Sadhwani | 5b2df17 | 2019-10-16 09:13:38 -0700 | [diff] [blame] | 338 | } |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 339 | } |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 340 | } |
| 341 | |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 342 | if (tested_packets == buffer_.size()) |
philipel | 227f8b9 | 2017-08-04 06:39:31 -0700 | [diff] [blame] | 343 | break; |
| 344 | |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 345 | start_index = start_index > 0 ? start_index - 1 : buffer_.size() - 1; |
philipel | 8c61924 | 2017-02-02 08:51:29 -0800 | [diff] [blame] | 346 | |
| 347 | // In the case of H264 we don't have a frame_begin bit (yes, |
| 348 | // |frame_begin| might be set to true but that is a lie). So instead |
| 349 | // we traverese backwards as long as we have a previous packet and |
| 350 | // the timestamp of that packet is the same as this one. This may cause |
| 351 | // the PacketBuffer to hand out incomplete frames. |
| 352 | // See: https://bugs.chromium.org/p/webrtc/issues/detail?id=7106 |
philipel | 5391071 | 2017-05-18 02:24:40 -0700 | [diff] [blame] | 353 | if (is_h264 && |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 354 | (!buffer_[start_index].used || |
| 355 | buffer_[start_index].data.timestamp != frame_timestamp)) { |
philipel | 8c61924 | 2017-02-02 08:51:29 -0800 | [diff] [blame] | 356 | break; |
| 357 | } |
| 358 | |
| 359 | --start_seq_num; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 360 | } |
| 361 | |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 362 | if (is_h264) { |
| 363 | // Warn if this is an unsafe frame. |
| 364 | if (has_h264_idr && (!has_h264_sps || !has_h264_pps)) { |
Jonas Olsson | fc50110 | 2018-06-15 14:24:10 +0200 | [diff] [blame] | 365 | RTC_LOG(LS_WARNING) |
| 366 | << "Received H.264-IDR frame " |
| 367 | << "(SPS: " << has_h264_sps << ", PPS: " << has_h264_pps |
| 368 | << "). Treating as " |
| 369 | << (sps_pps_idr_is_h264_keyframe_ ? "delta" : "key") |
| 370 | << " frame since WebRTC-SpsPpsIdrIsH264Keyframe is " |
| 371 | << (sps_pps_idr_is_h264_keyframe_ ? "enabled." : "disabled"); |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 372 | } |
| 373 | |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 374 | // Now that we have decided whether to treat this frame as a key frame |
| 375 | // or delta frame in the frame buffer, we update the field that |
| 376 | // determines if the RtpFrameObject is a key frame or delta frame. |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 377 | const size_t first_packet_index = start_seq_num % buffer_.size(); |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 378 | if (is_h264_keyframe) { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 379 | buffer_[first_packet_index].data.video_header.frame_type = |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 380 | VideoFrameType::kVideoFrameKey; |
Shyam Sadhwani | 5b2df17 | 2019-10-16 09:13:38 -0700 | [diff] [blame] | 381 | if (idr_width > 0 && idr_height > 0) { |
| 382 | // IDR frame was finalized and we have the correct resolution for |
| 383 | // IDR; update first packet to have same resolution as IDR. |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 384 | buffer_[first_packet_index].data.video_header.width = idr_width; |
| 385 | buffer_[first_packet_index].data.video_header.height = idr_height; |
Shyam Sadhwani | 5b2df17 | 2019-10-16 09:13:38 -0700 | [diff] [blame] | 386 | } |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 387 | } else { |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 388 | buffer_[first_packet_index].data.video_header.frame_type = |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 389 | VideoFrameType::kVideoFrameDelta; |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 390 | } |
| 391 | |
Johnny Lee | bc7f41b | 2019-05-01 14:41:32 -0400 | [diff] [blame] | 392 | // With IPPP, if this is not a keyframe, make sure there are no gaps |
| 393 | // in the packet sequence numbers up until this point. |
| 394 | const uint8_t h264tid = |
Danil Chapovalov | 4aae11d | 2019-10-18 11:17:03 +0200 | [diff] [blame] | 395 | buffer_[start_index].data.video_header.frame_marking.temporal_id; |
Jonas Olsson | a4d8737 | 2019-07-05 19:08:33 +0200 | [diff] [blame] | 396 | if (h264tid == kNoTemporalIdx && !is_h264_keyframe && |
| 397 | missing_packets_.upper_bound(start_seq_num) != |
| 398 | missing_packets_.begin()) { |
Rasmus Brandt | 88f080a | 2017-11-02 14:28:06 +0100 | [diff] [blame] | 399 | return found_frames; |
| 400 | } |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 401 | } |
| 402 | |
Danil Chapovalov | 0682ca9 | 2019-11-28 16:50:02 +0100 | [diff] [blame^] | 403 | if (auto frame = AssembleFrame(start_seq_num, seq_num)) { |
| 404 | found_frames.push_back(std::move(frame)); |
| 405 | } else { |
| 406 | RTC_LOG(LS_ERROR) << "Failed to assemble frame from packets " |
| 407 | << start_seq_num << "-" << seq_num; |
| 408 | } |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 409 | |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 410 | missing_packets_.erase(missing_packets_.begin(), |
| 411 | missing_packets_.upper_bound(seq_num)); |
Johannes Kron | a370556 | 2019-08-26 16:37:11 +0200 | [diff] [blame] | 412 | ClearInterval(start_seq_num, seq_num); |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 413 | } |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 414 | ++seq_num; |
| 415 | } |
philipel | fd5a20f | 2016-11-15 00:57:57 -0800 | [diff] [blame] | 416 | return found_frames; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 417 | } |
| 418 | |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 419 | std::unique_ptr<RtpFrameObject> PacketBuffer::AssembleFrame( |
philipel | b5e4785 | 2019-09-20 11:30:12 +0200 | [diff] [blame] | 420 | uint16_t first_seq_num, |
| 421 | uint16_t last_seq_num) { |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 422 | const uint16_t end_seq_num = last_seq_num + 1; |
| 423 | const uint16_t num_packets = end_seq_num - first_seq_num; |
| 424 | int max_nack_count = -1; |
| 425 | int64_t min_recv_time = std::numeric_limits<int64_t>::max(); |
| 426 | int64_t max_recv_time = std::numeric_limits<int64_t>::min(); |
| 427 | size_t frame_size = 0; |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 428 | |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 429 | std::vector<rtc::ArrayView<const uint8_t>> payloads; |
| 430 | RtpPacketInfos::vector_type packet_infos; |
| 431 | payloads.reserve(num_packets); |
| 432 | packet_infos.reserve(num_packets); |
philipel | 227f8b9 | 2017-08-04 06:39:31 -0700 | [diff] [blame] | 433 | |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 434 | for (uint16_t seq_num = first_seq_num; seq_num != end_seq_num; ++seq_num) { |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 435 | const Packet& packet = GetPacket(seq_num); |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 436 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 437 | max_nack_count = std::max(max_nack_count, packet.times_nacked); |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 438 | min_recv_time = |
| 439 | std::min(min_recv_time, packet.packet_info.receive_time_ms()); |
| 440 | max_recv_time = |
| 441 | std::max(max_recv_time, packet.packet_info.receive_time_ms()); |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 442 | frame_size += packet.size_bytes; |
| 443 | payloads.emplace_back(packet.data, packet.size_bytes); |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 444 | packet_infos.push_back(packet.packet_info); |
| 445 | } |
philipel | 227f8b9 | 2017-08-04 06:39:31 -0700 | [diff] [blame] | 446 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 447 | const Packet& first_packet = GetPacket(first_seq_num); |
Danil Chapovalov | 0682ca9 | 2019-11-28 16:50:02 +0100 | [diff] [blame^] | 448 | rtc::scoped_refptr<EncodedImageBuffer> bitstream; |
| 449 | // TODO(danilchap): Hide codec-specific code paths behind an interface. |
| 450 | if (first_packet.codec() == VideoCodecType::kVideoCodecAV1) { |
| 451 | bitstream = RtpDepacketizerAv1::AssembleFrame(payloads); |
| 452 | if (!bitstream) { |
| 453 | // Failed to assemble a frame. Discard and continue. |
| 454 | return nullptr; |
| 455 | } |
| 456 | } else { |
| 457 | bitstream = EncodedImageBuffer::Create(frame_size); |
| 458 | |
| 459 | uint8_t* write_at = bitstream->data(); |
| 460 | for (rtc::ArrayView<const uint8_t> payload : payloads) { |
| 461 | memcpy(write_at, payload.data(), payload.size()); |
| 462 | write_at += payload.size(); |
| 463 | } |
| 464 | RTC_DCHECK_EQ(write_at - bitstream->data(), bitstream->size()); |
| 465 | } |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 466 | const Packet& last_packet = GetPacket(last_seq_num); |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 467 | return std::make_unique<RtpFrameObject>( |
| 468 | first_seq_num, // |
| 469 | last_seq_num, // |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 470 | last_packet.marker_bit, // |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 471 | max_nack_count, // |
| 472 | min_recv_time, // |
| 473 | max_recv_time, // |
| 474 | first_packet.timestamp, // |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 475 | first_packet.ntp_time_ms, // |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 476 | last_packet.video_header.video_timing, // |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 477 | first_packet.payload_type, // |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 478 | first_packet.codec(), // |
| 479 | last_packet.video_header.rotation, // |
| 480 | last_packet.video_header.content_type, // |
| 481 | first_packet.video_header, // |
| 482 | last_packet.video_header.color_space, // |
| 483 | first_packet.generic_descriptor, // |
| 484 | RtpPacketInfos(std::move(packet_infos)), // |
| 485 | std::move(bitstream)); |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 486 | } |
| 487 | |
Danil Chapovalov | aa3f5da | 2019-11-14 14:57:33 +0100 | [diff] [blame] | 488 | const PacketBuffer::Packet& PacketBuffer::GetPacket(uint16_t seq_num) const { |
Danil Chapovalov | 3527a4f | 2019-11-08 17:30:29 +0100 | [diff] [blame] | 489 | const StoredPacket& entry = buffer_[seq_num % buffer_.size()]; |
| 490 | RTC_DCHECK(entry.used); |
| 491 | RTC_DCHECK_EQ(seq_num, entry.seq_num()); |
| 492 | return entry.data; |
philipel | f413933 | 2016-04-20 10:26:34 +0200 | [diff] [blame] | 493 | } |
| 494 | |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 495 | void PacketBuffer::UpdateMissingPackets(uint16_t seq_num) { |
| 496 | if (!newest_inserted_seq_num_) |
Oskar Sundbom | 6bd3902 | 2017-11-16 10:54:49 +0100 | [diff] [blame] | 497 | newest_inserted_seq_num_ = seq_num; |
philipel | 2c9f9f2 | 2017-06-13 02:47:28 -0700 | [diff] [blame] | 498 | |
| 499 | const int kMaxPaddingAge = 1000; |
| 500 | if (AheadOf(seq_num, *newest_inserted_seq_num_)) { |
| 501 | uint16_t old_seq_num = seq_num - kMaxPaddingAge; |
| 502 | auto erase_to = missing_packets_.lower_bound(old_seq_num); |
| 503 | missing_packets_.erase(missing_packets_.begin(), erase_to); |
| 504 | |
| 505 | // Guard against inserting a large amount of missing packets if there is a |
| 506 | // jump in the sequence number. |
| 507 | if (AheadOf(old_seq_num, *newest_inserted_seq_num_)) |
| 508 | *newest_inserted_seq_num_ = old_seq_num; |
| 509 | |
| 510 | ++*newest_inserted_seq_num_; |
| 511 | while (AheadOf(seq_num, *newest_inserted_seq_num_)) { |
| 512 | missing_packets_.insert(*newest_inserted_seq_num_); |
| 513 | ++*newest_inserted_seq_num_; |
| 514 | } |
| 515 | } else { |
| 516 | missing_packets_.erase(seq_num); |
| 517 | } |
| 518 | } |
| 519 | |
philipel | c707ab7 | 2016-04-01 02:01:54 -0700 | [diff] [blame] | 520 | } // namespace video_coding |
| 521 | } // namespace webrtc |