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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webrtc/pc/statscollector.cc

Issue 2946413002: Report timing frames info in GetStats. (Closed)
Patch Set: Implement Hbos@ comments Created 3 years, 6 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
« webrtc/modules/video_coding/timing.h ('K') | « webrtc/modules/video_coding/timing.cc ('k') | webrtc/test/fuzzers/rtp_packet_fuzzer.cc » ('j') | webrtc/video/receive_statistics_proxy.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/pc/statscollector.cc
diff --git a/webrtc/pc/statscollector.cc b/webrtc/pc/statscollector.cc
index c9325831598549b50c3d1e63de24cddd54ab8ab3..1a6ae8ea3e9a720fb098a073e8913460e6ce5ed2 100644
--- a/webrtc/pc/statscollector.cc
+++ b/webrtc/pc/statscollector.cc
@@ -249,6 +249,9 @@ void ExtractStats(const cricket::VideoReceiverInfo& info, StatsReport* report) {
for (const auto& i : ints)
report->AddInt(i.name, i.value);
report->AddString(StatsReport::kStatsValueNameMediaType, "video");
+ if (info.timing_frame_info)
+ report->AddString(StatsReport::kStatsValueNameTimingFrameInfo,
+ info.timing_frame_info->ToString());
sprang_webrtc 2017/06/28 15:31:41 nit: curly brackets for multi-line if, plus indent
ilnik 2017/06/29 08:39:56 Done.
}
void ExtractStats(const cricket::VideoSenderInfo& info, StatsReport* report) {
« webrtc/modules/video_coding/timing.h ('K') | « webrtc/modules/video_coding/timing.cc ('k') | webrtc/test/fuzzers/rtp_packet_fuzzer.cc » ('j') | webrtc/video/receive_statistics_proxy.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698