admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Expanded error message for unexpected packet

Browse Source 
in the flaky test EndToEndTest.DecodesRetransmittedFrame*

BUG=webrtc:5540
R=pbos@webrtc.org, pbos

Review URL: https://codereview.webrtc.org/2100793002 .

Cr-Commit-Position: refs/heads/master@{#13323}
This commit is contained in:
Danil Chapovalov 2016-06-29 15:29:34 +02:00
parent 9c0c75bd6e
commit 18c65a448f
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
webrtc/video/end_to_end_tests.cc
View File

@ -790,13 +790,19 @@ void EndToEndTest::DecodesRetransmittedFrame(bool enable_rtx, bool enable_red) {
} }
EXPECT_EQ(kVideoSendSsrcs[0], header.ssrc) EXPECT_EQ(kVideoSendSsrcs[0], header.ssrc)
<< "Payload type " << static_cast<int>(header.payloadType) << "Unexpected packet length " << length
<< " not expected."; << ", header_length " << header.headerLength
<< ", padding_length " << header.paddingLength
<< ", timestamp " << header.timestamp
<< ", expected timestamp " << retransmitted_timestamp_
<< ", payload type " << static_cast<int>(header.payloadType);
EXPECT_EQ(payload_type_, header.payloadType); EXPECT_EQ(payload_type_, header.payloadType);
// Found the final packet of the frame to inflict loss to, drop this and // Found the final packet of the frame to inflict loss to, drop this and
// expect a retransmission. // expect a retransmission.
if (header.markerBit && ++marker_bits_observed_ == kDroppedFrameNumber) { if (header.markerBit && ++marker_bits_observed_ == kDroppedFrameNumber) {
// This should be the only dropped packet.
EXPECT_EQ(0u, retransmitted_timestamp_);
retransmitted_timestamp_ = header.timestamp; retransmitted_timestamp_ = header.timestamp;
return DROP_PACKET; return DROP_PACKET;
} }