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

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

Issue 1544403002: [rtp_rtcp] rtcp::Fir cleaned and got Parse function. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 11 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
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtcp_receiver_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 523 matching lines...) Expand 10 before | Expand all | Expand 10 after
534 534
535 return rtc::scoped_ptr<rtcp::Pli>(pli); 535 return rtc::scoped_ptr<rtcp::Pli>(pli);
536 } 536 }
537 537
538 rtc::scoped_ptr<rtcp::RtcpPacket> RTCPSender::BuildFIR(const RtcpContext& ctx) { 538 rtc::scoped_ptr<rtcp::RtcpPacket> RTCPSender::BuildFIR(const RtcpContext& ctx) {
539 if (!ctx.repeat_) 539 if (!ctx.repeat_)
540 ++sequence_number_fir_; // Do not increase if repetition. 540 ++sequence_number_fir_; // Do not increase if repetition.
541 541
542 rtcp::Fir* fir = new rtcp::Fir(); 542 rtcp::Fir* fir = new rtcp::Fir();
543 fir->From(ssrc_); 543 fir->From(ssrc_);
544 fir->To(remote_ssrc_); 544 fir->WithRequestTo(remote_ssrc_, sequence_number_fir_);
545 fir->WithCommandSeqNum(sequence_number_fir_);
546 545
547 TRACE_EVENT_INSTANT0(TRACE_DISABLED_BY_DEFAULT("webrtc_rtp"), 546 TRACE_EVENT_INSTANT0(TRACE_DISABLED_BY_DEFAULT("webrtc_rtp"),
548 "RTCPSender::FIR"); 547 "RTCPSender::FIR");
549 ++packet_type_counter_.fir_packets; 548 ++packet_type_counter_.fir_packets;
550 TRACE_COUNTER_ID1(TRACE_DISABLED_BY_DEFAULT("webrtc_rtp"), "RTCP_FIRCount", 549 TRACE_COUNTER_ID1(TRACE_DISABLED_BY_DEFAULT("webrtc_rtp"), "RTCP_FIRCount",
551 ssrc_, packet_type_counter_.fir_packets); 550 ssrc_, packet_type_counter_.fir_packets);
552 551
553 return rtc::scoped_ptr<rtcp::Fir>(fir); 552 return rtc::scoped_ptr<rtcp::Fir>(fir);
554 } 553 }
555 554
(...skipping 502 matching lines...) Expand 10 before | Expand all | Expand 10 after
1058 Transport* const transport_; 1057 Transport* const transport_;
1059 bool send_failure_; 1058 bool send_failure_;
1060 } sender(transport_); 1059 } sender(transport_);
1061 1060
1062 uint8_t buffer[IP_PACKET_SIZE]; 1061 uint8_t buffer[IP_PACKET_SIZE];
1063 return packet.BuildExternalBuffer(buffer, IP_PACKET_SIZE, &sender) && 1062 return packet.BuildExternalBuffer(buffer, IP_PACKET_SIZE, &sender) &&
1064 !sender.send_failure_; 1063 !sender.send_failure_;
1065 } 1064 }
1066 1065
1067 } // namespace webrtc 1066 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtcp_receiver_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698