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

Unified Diff: webrtc/video/send_statistics_proxy.h

Issue 1720883002: Move RTCP histograms from vie_channel to video channel stats proxies. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase 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
« no previous file with comments | « webrtc/video/receive_statistics_proxy.cc ('k') | webrtc/video/send_statistics_proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/send_statistics_proxy.h
diff --git a/webrtc/video/send_statistics_proxy.h b/webrtc/video/send_statistics_proxy.h
index b978fab10b74cb727a72bc28d8fa8a0bd72a9493..0d0d0e071a8e20f2f62d1e81cb112deb540ecfe6 100644
--- a/webrtc/video/send_statistics_proxy.h
+++ b/webrtc/video/send_statistics_proxy.h
@@ -142,12 +142,16 @@ class SendStatisticsProxy : public CpuOveruseMetricsObserver,
// content type changes between real-time video and screenshare, since these
// will be reported separately.
struct UmaSamplesContainer {
- explicit UmaSamplesContainer(const char* prefix);
+ UmaSamplesContainer(const char* prefix,
+ const VideoSendStream::Stats& start_stats,
+ Clock* clock);
~UmaSamplesContainer();
- void UpdateHistograms();
+ void UpdateHistograms(const VideoSendStream::Config& config,
+ const VideoSendStream::Stats& current_stats);
const std::string uma_prefix_;
+ Clock* const clock_;
int max_sent_width_per_timestamp_;
int max_sent_height_per_timestamp_;
SampleCounter input_width_counter_;
@@ -166,6 +170,7 @@ class SendStatisticsProxy : public CpuOveruseMetricsObserver,
rtc::RateTracker sent_frame_rate_tracker_;
int64_t first_rtcp_stats_time_ms_;
ReportBlockStats report_block_stats_;
+ const VideoSendStream::Stats start_stats_;
};
rtc::scoped_ptr<UmaSamplesContainer> uma_container_ GUARDED_BY(crit_);
« no previous file with comments | « webrtc/video/receive_statistics_proxy.cc ('k') | webrtc/video/send_statistics_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698