diff options
| author | turaj@webrtc.org <turaj@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | 2014-07-25 17:50:10 +0000 |
|---|---|---|
| committer | turaj@webrtc.org <turaj@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | 2014-07-25 17:50:10 +0000 |
| commit | ca4bc68229671d72f8b68481a4ca3fa1407bc2d9 (patch) | |
| tree | 5029093a7fa5944dda81af89b4eb60b8de8189bb /voice_engine/channel.cc | |
| parent | 648d555669a414b9dbadc931c4f44ac813189ef9 (diff) | |
| download | webrtc-ca4bc68229671d72f8b68481a4ca3fa1407bc2d9.tar.gz | |
Remove timestamp retreival warning/error.
An error reported while retreiving playout timestamp if no RTP packet received, yet. This causes an overflow of errors/warnings in applications where few channel are created but only one is actively engaged in a conversation. Therefore, we don't find such logging informative (there is no check upon correctness of timestamp computaion only if a packet already received).
BUG=3545
TEST=manual with voe_cmd_test,try bots
R=andrew@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/18869004
git-svn-id: http://webrtc.googlecode.com/svn/trunk/webrtc@6781 4adac7df-926f-26a2-2b94-8c16560cd09d
Diffstat (limited to 'voice_engine/channel.cc')
| -rw-r--r-- | voice_engine/channel.cc | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/voice_engine/channel.cc b/voice_engine/channel.cc index af773e77..e811a273 100644 --- a/voice_engine/channel.cc +++ b/voice_engine/channel.cc @@ -4025,12 +4025,8 @@ void Channel::UpdatePlayoutTimestamp(bool rtcp) { uint32_t playout_timestamp = 0; if (audio_coding_->PlayoutTimestamp(&playout_timestamp) == -1) { - WEBRTC_TRACE(kTraceWarning, kTraceVoice, VoEId(_instanceId,_channelId), - "Channel::UpdatePlayoutTimestamp() failed to read playout" - " timestamp from the ACM"); - _engineStatisticsPtr->SetLastError( - VE_CANNOT_RETRIEVE_VALUE, kTraceError, - "UpdatePlayoutTimestamp() failed to retrieve timestamp"); + // This can happen if this channel has not been received any RTP packet. In + // this case, NetEq is not capable of computing playout timestamp. return; } |