zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Adds abs-send-time and size field outputs to event log parser.

Browse Source 
Bug: webrtc:10170
Change-Id: I7340bc2afa4fa29139a4358db73e002825153fe0
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/153888
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29287}
This commit is contained in:
Sebastian Jansson
2019-09-20 17:59:45 +02:00
committed by Commit Bot
parent 74344d2aa6
commit c30bc169ea
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
logging/rtc_event_log/logged_events.cc
View File

@ -18,6 +18,10 @@ LoggedPacketInfo::LoggedPacketInfo(const LoggedRtpPacket& rtp,
: ssrc(rtp.header.ssrc),
stream_seq_no(rtp.header.sequenceNumber),
size(static_cast<uint16_t>(rtp.total_length)),
payload_size(static_cast<uint16_t>(rtp.total_length -
rtp.header.paddingLength -
rtp.header.headerLength)),
padding_size(static_cast<uint16_t>(rtp.header.paddingLength)),
payload_type(rtp.header.payloadType),
media_type(media_type),
rtx(rtx),
@ -27,7 +31,10 @@ LoggedPacketInfo::LoggedPacketInfo(const LoggedRtpPacket& rtp,
has_transport_seq_no ? rtp.header.extension.transportSequenceNumber
: 0)),
capture_time(capture_time),
log_packet_time(Timestamp::us(rtp.log_time_us())) {}
log_packet_time(Timestamp::us(rtp.log_time_us())),
reported_send_time(rtp.header.extension.hasAbsoluteSendTime
? rtp.header.extension.GetAbsoluteSendTimestamp()
: Timestamp::MinusInfinity()) {}
LoggedPacketInfo::LoggedPacketInfo(const LoggedPacketInfo&) = default;

5
logging/rtc_event_log/logged_events.h
View File

@ -541,6 +541,8 @@ struct LoggedPacketInfo {
uint32_t ssrc;
uint16_t stream_seq_no;
uint16_t size;
uint16_t payload_size;
uint16_t padding_size;
uint16_t overhead = 0;
uint8_t payload_type;
LoggedMediaType media_type = LoggedMediaType::kUnknown;
@ -555,6 +557,9 @@ struct LoggedPacketInfo {
// The time the packet was logged. This is the receive time for incoming
// packets and send time for outgoing.
Timestamp log_packet_time;
// Send time as reported by abs-send-time extension, For outgoing packets this
// corresponds to log_packet_time, but might be measured using another clock.
Timestamp reported_send_time;
// The receive time that was reported in feedback. For incoming packets this
// corresponds to log_packet_time, but might be measured using another clock.
// PlusInfinity indicates that the packet was lost.