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

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: use rtc::TimeMillis 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 ad9beb35e75384801cf03d264f935a7137a9b383..05fd441b8a07e485187e684f1655aa37c6d652f0 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 << ",";
+ ss << "[tot_bps:" << total_bitrate_bps << "],";
pbos-webrtc 2016/07/10 16:58:51 Remove []s unless this format is from somewhere. A
åsapersson 2016/08/01 15:55:22 Done.
+ ss << "[w:" << width << "],";
+ ss << "[h:" << height << "],";
+ ss << "[network_fps:" << network_frame_rate << "],";
+ ss << "[dec_fps:" << decode_frame_rate << "],";
+ ss << "[render_fps:" << render_frame_rate << "],";
+ ss << "[dec_ms:" << decode_ms << "],";
+ ss << "[max_dec_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