From 0d540c3762a9ec6bf960cea2e5d7d70585bb593c Mon Sep 17 00:00:00 2001 From: "hclam@chromium.org" Date: Tue, 21 May 2013 00:16:01 +0000 Subject: [PATCH] Log timestamp of the frame when it's dropped from the render module R=wu@webrtc.org Review URL: https://webrtc-codereview.appspot.com/1515005 git-svn-id: http://webrtc.googlecode.com/svn/trunk@4075 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/modules/video_render/video_render_frames.cc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/webrtc/modules/video_render/video_render_frames.cc b/webrtc/modules/video_render/video_render_frames.cc index 2c137a3190..2968dfa3d5 100644 --- a/webrtc/modules/video_render/video_render_frames.cc +++ b/webrtc/modules/video_render/video_render_frames.cc @@ -36,12 +36,14 @@ int32_t VideoRenderFrames::AddFrame(I420VideoFrame* new_frame) { if (new_frame->render_time_ms() + KOldRenderTimestampMS < time_now) { WEBRTC_TRACE(kTraceWarning, kTraceVideoRenderer, -1, - "%s: too old frame.", __FUNCTION__); + "%s: too old frame, timestamp=%u.", + __FUNCTION__, new_frame->timestamp()); return -1; } if (new_frame->render_time_ms() > time_now + KFutureRenderTimestampMS) { WEBRTC_TRACE(kTraceWarning, kTraceVideoRenderer, -1, - "%s: frame too long into the future.", __FUNCTION__); + "%s: frame too long into the future, timestamp=%u.", + __FUNCTION__, new_frame->timestamp()); return -1; } @@ -59,8 +61,8 @@ int32_t VideoRenderFrames::AddFrame(I420VideoFrame* new_frame) { KMaxNumberOfFrames) { // Already allocated too many frames. WEBRTC_TRACE(kTraceWarning, kTraceVideoRenderer, - -1, "%s: too many frames, limit: %d", __FUNCTION__, - KMaxNumberOfFrames); + -1, "%s: too many frames, timestamp=%u, limit=%d", + __FUNCTION__, new_frame->timestamp(), KMaxNumberOfFrames); return -1; }