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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc

Issue 2693123002: Make Call::OnRecoveredPacket parse RTP header and call OnRtpPacket. (Closed)
Patch Set: Update fuzzer. Created 3 years, 7 months 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 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 0) { 125 0) {
126 return false; 126 return false;
127 } 127 }
128 } 128 }
129 // Return recovered packets through callback. 129 // Return recovered packets through callback.
130 for (const auto& recovered_packet : recovered_packets_) { 130 for (const auto& recovered_packet : recovered_packets_) {
131 if (recovered_packet->returned) { 131 if (recovered_packet->returned) {
132 continue; 132 continue;
133 } 133 }
134 ++packet_counter_.num_recovered_packets; 134 ++packet_counter_.num_recovered_packets;
135 if (!recovered_packet_receiver_->OnRecoveredPacket( 135 recovered_packet_receiver_->OnRecoveredPacket(
136 recovered_packet->pkt->data, recovered_packet->pkt->length)) { 136 recovered_packet->pkt->data, recovered_packet->pkt->length);
137 return false;
138 }
139 recovered_packet->returned = true; 137 recovered_packet->returned = true;
140 // Periodically log the incoming packets. 138 // Periodically log the incoming packets.
141 int64_t now_ms = clock_->TimeInMilliseconds(); 139 int64_t now_ms = clock_->TimeInMilliseconds();
142 if (now_ms - last_recovered_packet_ms_ > kPacketLogIntervalMs) { 140 if (now_ms - last_recovered_packet_ms_ > kPacketLogIntervalMs) {
143 uint32_t media_ssrc = 141 uint32_t media_ssrc =
144 ForwardErrorCorrection::ParseSsrc(recovered_packet->pkt->data); 142 ForwardErrorCorrection::ParseSsrc(recovered_packet->pkt->data);
145 LOG(LS_VERBOSE) << "Recovered media packet with SSRC: " << media_ssrc 143 LOG(LS_VERBOSE) << "Recovered media packet with SSRC: " << media_ssrc
146 << " from FlexFEC stream with SSRC: " << ssrc_ << "."; 144 << " from FlexFEC stream with SSRC: " << ssrc_ << ".";
147 last_recovered_packet_ms_ = now_ms; 145 last_recovered_packet_ms_ = now_ms;
148 } 146 }
149 } 147 }
150 return true; 148 return true;
151 } 149 }
152 150
153 } // namespace webrtc 151 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698