From 715faaf5cedd39ef8947b46842620c852283868c Mon Sep 17 00:00:00 2001 From: "stefan@webrtc.org" Date: Tue, 28 Aug 2012 15:20:39 +0000 Subject: [PATCH] Change so that capture_time_ms == 0 is considered invalid. No transmission offset computed if capture_time_ms is less than or equal to zero. BUG= Review URL: https://webrtc-codereview.appspot.com/761004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@2675 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/modules/rtp_rtcp/source/rtp_sender.cc | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/modules/rtp_rtcp/source/rtp_sender.cc b/src/modules/rtp_rtcp/source/rtp_sender.cc index 757d25c314..5f83fe16c6 100644 --- a/src/modules/rtp_rtcp/source/rtp_sender.cc +++ b/src/modules/rtp_rtcp/source/rtp_sender.cc @@ -914,7 +914,10 @@ RTPSender::SendToNetwork(WebRtc_UWord8* buffer, return 0; } - if (capture_time_ms >= 0) { + // |capture_time_ms| <= 0 is considered invalid. + // TODO(holmer): This should be changed all over Video Engine so that negative + // time is consider invalid, while 0 is considered a valid time. + if (capture_time_ms > 0) { ModuleRTPUtility::RTPHeaderParser rtpParser(buffer, length); WebRtcRTPHeader rtp_header; rtpParser.Parse(rtp_header);