Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Unified Diff: webrtc/modules/video_coding/video_receiver.cc

Issue 1765443002: Added log messages for important call setup events: first packet sent/received (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Moved first packet logging from rtp_receiver_impl.cc to rtp_receiver_audio/video.cc Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/video_receiver.cc
diff --git a/webrtc/modules/video_coding/video_receiver.cc b/webrtc/modules/video_coding/video_receiver.cc
index 5aadcf91e0eb41a1f3ed8b244db7c0de9d186f77..e0b15b4c9c3dc3e7fab9cf9b8cd13f972073cae0 100644
--- a/webrtc/modules/video_coding/video_receiver.cc
+++ b/webrtc/modules/video_coding/video_receiver.cc
@@ -56,7 +56,8 @@ VideoReceiver::VideoReceiver(Clock* clock,
pre_decode_image_callback_(NULL),
_receiveStatsTimer(1000, clock_),
_retransmissionTimer(10, clock_),
- _keyRequestTimer(500, clock_) {
+ _keyRequestTimer(500, clock_),
+ received_first_frame_(false) {
assert(clock_);
#ifdef DEBUG_DECODER_BIT_STREAM
_bitStreamBeforeDecoder = fopen("decoderBitStream.bit", "wb");
@@ -323,6 +324,13 @@ int32_t VideoReceiver::Decode(uint16_t maxWaitTimeMs) {
}
}
#endif
+
+ if (!received_first_frame_) {
+ received_first_frame_ = true;
+ LOG(LS_INFO) << "Received first decodable video frame, complete = "
+ << (frame->Complete() ? "true" : "false");
+ }
+
const int32_t ret = Decode(*frame);
_receiver.ReleaseFrame(frame);
return ret;

Powered by Google App Engine
This is Rietveld 408576698