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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtp_receiver_impl.h

Issue 1765443002: Added log messages for important call setup events: first packet sent/received (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rephrased the log lines to begin with "sent"/"received" Created 4 years, 9 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) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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 11 matching lines...) Expand all
22 22
23 class RtpReceiverImpl : public RtpReceiver { 23 class RtpReceiverImpl : public RtpReceiver {
24 public: 24 public:
25 // Callbacks passed in here may not be NULL (use Null Object callbacks if you 25 // Callbacks passed in here may not be NULL (use Null Object callbacks if you
26 // want callbacks to do nothing). This class takes ownership of the media 26 // want callbacks to do nothing). This class takes ownership of the media
27 // receiver but nothing else. 27 // receiver but nothing else.
28 RtpReceiverImpl(Clock* clock, 28 RtpReceiverImpl(Clock* clock,
29 RtpAudioFeedback* incoming_audio_messages_callback, 29 RtpAudioFeedback* incoming_audio_messages_callback,
30 RtpFeedback* incoming_messages_callback, 30 RtpFeedback* incoming_messages_callback,
31 RTPPayloadRegistry* rtp_payload_registry, 31 RTPPayloadRegistry* rtp_payload_registry,
32 RTPReceiverStrategy* rtp_media_receiver); 32 RTPReceiverStrategy* rtp_media_receiver,
33 bool is_audio);
33 34
34 virtual ~RtpReceiverImpl(); 35 virtual ~RtpReceiverImpl();
35 36
36 int32_t RegisterReceivePayload(const char payload_name[RTP_PAYLOAD_NAME_SIZE], 37 int32_t RegisterReceivePayload(const char payload_name[RTP_PAYLOAD_NAME_SIZE],
37 const int8_t payload_type, 38 const int8_t payload_type,
38 const uint32_t frequency, 39 const uint32_t frequency,
39 const size_t channels, 40 const size_t channels,
40 const uint32_t rate) override; 41 const uint32_t rate) override;
41 42
42 int32_t DeRegisterReceivePayload(const int8_t payload_type) override; 43 int32_t DeRegisterReceivePayload(const int8_t payload_type) override;
(...skipping 27 matching lines...) Expand all
70 void CheckSSRCChanged(const RTPHeader& rtp_header); 71 void CheckSSRCChanged(const RTPHeader& rtp_header);
71 void CheckCSRC(const WebRtcRTPHeader& rtp_header); 72 void CheckCSRC(const WebRtcRTPHeader& rtp_header);
72 int32_t CheckPayloadChanged(const RTPHeader& rtp_header, 73 int32_t CheckPayloadChanged(const RTPHeader& rtp_header,
73 const int8_t first_payload_byte, 74 const int8_t first_payload_byte,
74 bool* is_red, 75 bool* is_red,
75 PayloadUnion* payload); 76 PayloadUnion* payload);
76 77
77 Clock* clock_; 78 Clock* clock_;
78 RTPPayloadRegistry* rtp_payload_registry_; 79 RTPPayloadRegistry* rtp_payload_registry_;
79 rtc::scoped_ptr<RTPReceiverStrategy> rtp_media_receiver_; 80 rtc::scoped_ptr<RTPReceiverStrategy> rtp_media_receiver_;
81 bool is_audio_;
80 82
81 RtpFeedback* cb_rtp_feedback_; 83 RtpFeedback* cb_rtp_feedback_;
82 84
83 rtc::scoped_ptr<CriticalSectionWrapper> critical_section_rtp_receiver_; 85 rtc::scoped_ptr<CriticalSectionWrapper> critical_section_rtp_receiver_;
84 int64_t last_receive_time_; 86 int64_t last_receive_time_;
85 size_t last_received_payload_length_; 87 size_t last_received_payload_length_;
86 88
87 // SSRCs. 89 // SSRCs.
88 uint32_t ssrc_; 90 uint32_t ssrc_;
89 uint8_t num_csrcs_; 91 uint8_t num_csrcs_;
90 uint32_t current_remote_csrc_[kRtpCsrcSize]; 92 uint32_t current_remote_csrc_[kRtpCsrcSize];
91 93
92 uint32_t last_received_timestamp_; 94 uint32_t last_received_timestamp_;
93 int64_t last_received_frame_time_ms_; 95 int64_t last_received_frame_time_ms_;
94 uint16_t last_received_sequence_number_; 96 uint16_t last_received_sequence_number_;
95 97
96 NACKMethod nack_method_; 98 NACKMethod nack_method_;
97 }; 99 };
98 } // namespace webrtc 100 } // namespace webrtc
99 #endif // WEBRTC_MODULES_RTP_RTCP_SOURCE_RTP_RECEIVER_IMPL_H_ 101 #endif // WEBRTC_MODULES_RTP_RTCP_SOURCE_RTP_RECEIVER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698