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

Side by Side Diff: media/cast/net/rtcp/rtcp_sender.h

Issue 499433002: Minor code redundancy cleanup. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_ 5 #ifndef MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_
6 #define MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_ 6 #define MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <list> 9 #include <list>
10 #include <string> 10 #include <string>
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 // of this class is to only to build a RTCP packet but not to send it. 44 // of this class is to only to build a RTCP packet but not to send it.
45 class RtcpSender { 45 class RtcpSender {
46 public: 46 public:
47 RtcpSender(PacedPacketSender* outgoing_transport, 47 RtcpSender(PacedPacketSender* outgoing_transport,
48 uint32 sending_ssrc); 48 uint32 sending_ssrc);
49 ~RtcpSender(); 49 ~RtcpSender();
50 50
51 // TODO(hclam): This method should be to build a packet instead of 51 // TODO(hclam): This method should be to build a packet instead of
52 // sending it. 52 // sending it.
53 void SendRtcpFromRtpReceiver( 53 void SendRtcpFromRtpReceiver(
54 uint32 packet_type_flags,
55 const RtcpReportBlock* report_block, 54 const RtcpReportBlock* report_block,
56 const RtcpReceiverReferenceTimeReport* rrtr, 55 const RtcpReceiverReferenceTimeReport* rrtr,
57 const RtcpCastMessage* cast_message, 56 const RtcpCastMessage* cast_message,
58 const ReceiverRtcpEventSubscriber::RtcpEventMultiMap* rtcp_events, 57 const ReceiverRtcpEventSubscriber::RtcpEventMultiMap* rtcp_events,
59 base::TimeDelta target_delay); 58 base::TimeDelta target_delay);
60 59
61 // TODO(hclam): This method should be to build a packet instead of 60 // TODO(hclam): This method should be to build a packet instead of
62 // sending it. 61 // sending it.
63 void SendRtcpFromRtpSender(uint32 packet_type_flags, 62 void SendRtcpFromRtpSender(const RtcpSenderInfo* sender_info,
64 const RtcpSenderInfo& sender_info, 63 const RtcpDlrrReportBlock* dlrr);
Alpha Left Google 2014/08/21 20:18:23 DLRR can be removed.
hubbe 2014/08/21 21:34:14 Done.
65 const RtcpDlrrReportBlock& dlrr);
66 64
67 private: 65 private:
68 void BuildRR(const RtcpReportBlock* report_block, 66 void BuildRR(const RtcpReportBlock* report_block,
69 Packet* packet) const; 67 Packet* packet) const;
70 68
71 void AddReportBlocks(const RtcpReportBlock& report_block, 69 void AddReportBlocks(const RtcpReportBlock& report_block,
72 Packet* packet) const; 70 Packet* packet) const;
73 71
74 void BuildRrtr(const RtcpReceiverReferenceTimeReport* rrtr, 72 void BuildRrtr(const RtcpReceiverReferenceTimeReport* rrtr,
75 Packet* packet) const; 73 Packet* packet) const;
(...skipping 25 matching lines...) Expand all
101 99
102 std::deque<RtcpReceiverLogMessage> rtcp_events_history_; 100 std::deque<RtcpReceiverLogMessage> rtcp_events_history_;
103 101
104 DISALLOW_COPY_AND_ASSIGN(RtcpSender); 102 DISALLOW_COPY_AND_ASSIGN(RtcpSender);
105 }; 103 };
106 104
107 } // namespace cast 105 } // namespace cast
108 } // namespace media 106 } // namespace media
109 107
110 #endif // MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_ 108 #endif // MEDIA_CAST_NET_RTCP_RTCP_SENDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698