generic-library/webrtc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add back EXPECT_TRUEs.

Browse Source 
These shouldn't fail, but EXPECT_TRUE gives nicer error messages that
work in Release. These changes got through unreviewed in r7726.

R=stefan@webrtc.org
BUG=

Review URL: https://webrtc-codereview.appspot.com/26249004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@7745 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
pbos@webrtc.org 2014-11-25 11:13:28 +00:00
parent ba253473da
commit b951eb12c9
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
webrtc/video/rampup_tests.cc
View File

@ -116,9 +116,7 @@ void StreamObserver::OnReceiveBitrateChanged(
bool StreamObserver::SendRtp(const uint8_t* packet, size_t length) {
CriticalSectionScoped lock(crit_.get());
RTPHeader header;
bool parse_succeeded = rtp_parser_->Parse(packet, length, &header);
RTC_UNUSED(parse_succeeded);
assert(parse_succeeded);
EXPECT_TRUE(rtp_parser_->Parse(packet, length, &header));
receive_stats_->IncomingPacket(header, length, false);
payload_registry_->SetIncomingPayloadType(header);
remote_bitrate_estimator_->IncomingPacket(
@ -268,9 +266,7 @@ PacketReceiver::DeliveryStatus LowRateStreamObserver::DeliverPacket(
const uint8_t* packet, size_t length) {
CriticalSectionScoped lock(crit_.get());
RTPHeader header;
bool parse_succeeded = rtp_parser_->Parse(packet, length, &header);
RTC_UNUSED(parse_succeeded);
assert(parse_succeeded);
EXPECT_TRUE(rtp_parser_->Parse(packet, length, &header));
receive_stats_->IncomingPacket(header, length, false);
remote_bitrate_estimator_->IncomingPacket(
clock_->TimeInMilliseconds(), length - 12, header);