From 54231f06627bf61b9cdab0181b0ebf725003fe06 Mon Sep 17 00:00:00 2001 From: "wu@webrtc.org" Date: Wed, 14 May 2014 23:06:23 +0000 Subject: [PATCH] Ignore the return value of UpdateRtcpTimestamp instead of printing warning. Because UpdateRtcpTimestamp may fail when there's no valid RTCP SR, which can happen in the first couple seconds or when the channel is a send only channel. Either case we don't want the warning log. BUG=crbug/371714 R=stefan@webrtc.org Review URL: https://webrtc-codereview.appspot.com/15509004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@6166 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/video_engine/vie_receiver.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/webrtc/video_engine/vie_receiver.cc b/webrtc/video_engine/vie_receiver.cc index 62f4f7e1db..c8d75084b5 100644 --- a/webrtc/video_engine/vie_receiver.cc +++ b/webrtc/video_engine/vie_receiver.cc @@ -333,9 +333,7 @@ int ViEReceiver::InsertRTCPPacket(const uint8_t* rtcp_packet, return ret; } - if (!ntp_estimator_->UpdateRtcpTimestamp(rtp_receiver_->SSRC(), rtp_rtcp_)) { - LOG(LS_WARNING) << "Failed to retrieve timestamp information from RTCP SR."; - } + ntp_estimator_->UpdateRtcpTimestamp(rtp_receiver_->SSRC(), rtp_rtcp_); return 0; }