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

Unified Diff: webrtc/video/video_receive_stream.cc

Issue 2133073002: Add periodic logging of video stats. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase, address comments Created 4 years, 5 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/video/video_receive_stream.cc
diff --git a/webrtc/video/video_receive_stream.cc b/webrtc/video/video_receive_stream.cc
index 609a789fa2b530a2d65ecdb29c10fe98ee1b66eb..4471697bc37ff59b413b21becd57949bd3a4a625 100644
--- a/webrtc/video/video_receive_stream.cc
+++ b/webrtc/video/video_receive_stream.cc
@@ -109,6 +109,26 @@ std::string VideoReceiveStream::Config::Rtp::ToString() const {
return ss.str();
}
+std::string VideoReceiveStream::Stats::ToString(int64_t time_ms) const {
+ std::stringstream ss;
+ ss << "VideoReceiveStream stats:" << time_ms << ", {ssrc:" << ssrc << ", ";
stefan-webrtc 2016/08/01 11:14:09 I'd prefer having a space after each ":", same in
åsapersson 2016/08/01 15:55:22 Done.
+ ss << "total_bps:" << total_bitrate_bps << ", ";
+ ss << "width:" << width << ", ";
+ ss << "height:" << height << ", ";
+ ss << "network_fps:" << network_frame_rate << ", ";
+ ss << "decode_fps:" << decode_frame_rate << ", ";
+ ss << "render_fps:" << render_frame_rate << ", ";
+ ss << "decode_ms:" << decode_ms << ", ";
+ ss << "max_decode_ms:" << max_decode_ms << ", ";
+ ss << "cur_delay_ms:" << current_delay_ms << ", ";
+ ss << "targ_delay_ms:" << target_delay_ms << ", ";
+ ss << "jb_delay_ms:" << jitter_buffer_ms << ", ";
+ ss << "min_playout_delay_ms:" << min_playout_delay_ms << ", ";
+ ss << "discarded:" << discarded_packets;
+ ss << '}';
+ return ss.str();
+}
+
namespace {
VideoCodec CreateDecoderVideoCodec(const VideoReceiveStream::Decoder& decoder) {
VideoCodec codec;

Powered by Google App Engine
This is Rietveld 408576698