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

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: Created 4 years, 10 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..f8273781a2f20ac5f999a63d4955c1647d09c80c 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) << "First decodable video frame received, complete = " <<
+ (frame->Complete() ? "true" : "false");
pthatcher1 2016/03/05 01:17:05 And here :) "Received first ...
skvlad 2016/03/07 19:36:53 Done.
+ }
+
const int32_t ret = Decode(*frame);
_receiver.ReleaseFrame(frame);
return ret;

Powered by Google App Engine
This is Rietveld 408576698