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

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: Rephrased the log lines to begin with "sent"/"received" 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 e7844d0add3aaf2c2f441043278f0264c27c6f76..84f437c8ecb390904f162014d5f0a2ec1ed1d1ca 100644
--- a/webrtc/modules/video_coding/video_receiver.cc
+++ b/webrtc/modules/video_coding/video_receiver.cc
@@ -49,7 +49,8 @@ VideoReceiver::VideoReceiver(Clock* clock, EventFactory* event_factory)
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");
@@ -310,6 +311,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