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

Unified Diff: remoting/protocol/performance_tracker.h

Issue 1811833002: Show max latency on client's status bar (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: JS compiler fix and lld fix Created 4 years, 9 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 | « remoting/protocol/capture_scheduler.h ('k') | remoting/remoting_srcs.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/performance_tracker.h
diff --git a/remoting/protocol/performance_tracker.h b/remoting/protocol/performance_tracker.h
index 3e5bf2c5f6b6eedafb65e64a3d292ffb0128c39b..c31ca91eb6774d750737130d62dc53674287d3ce 100644
--- a/remoting/protocol/performance_tracker.h
+++ b/remoting/protocol/performance_tracker.h
@@ -13,7 +13,7 @@
#include "base/macros.h"
#include "base/timer/timer.h"
#include "remoting/base/rate_counter.h"
-#include "remoting/base/running_average.h"
+#include "remoting/base/running_samples.h"
namespace remoting {
@@ -49,11 +49,11 @@ class PerformanceTracker {
double video_bandwidth() { return video_bandwidth_.Rate(); }
double video_frame_rate() { return video_frame_rate_.Rate(); }
double video_packet_rate() { return video_packet_rate_.Rate(); }
- double video_capture_ms() { return video_capture_ms_.Average(); }
- double video_encode_ms() { return video_encode_ms_.Average(); }
- double video_decode_ms() { return video_decode_ms_.Average(); }
- double video_paint_ms() { return video_paint_ms_.Average(); }
- double round_trip_ms() { return round_trip_ms_.Average(); }
+ const RunningSamples& video_capture_ms() { return video_capture_ms_; }
+ const RunningSamples& video_encode_ms() { return video_encode_ms_; }
+ const RunningSamples& video_decode_ms() { return video_decode_ms_; }
+ const RunningSamples& video_paint_ms() { return video_paint_ms_; }
+ const RunningSamples& round_trip_ms() { return round_trip_ms_; }
// Record stats for a video-packet.
void RecordVideoPacketStats(const VideoPacket& packet);
@@ -113,11 +113,11 @@ class PerformanceTracker {
// The following running-averages are uploaded to UMA per video packet and
// also used for display to users, averaged over the N most recent samples.
// N = kLatencySampleSize.
- RunningAverage video_capture_ms_;
- RunningAverage video_encode_ms_;
- RunningAverage video_decode_ms_;
- RunningAverage video_paint_ms_;
- RunningAverage round_trip_ms_;
+ RunningSamples video_capture_ms_;
+ RunningSamples video_encode_ms_;
+ RunningSamples video_decode_ms_;
+ RunningSamples video_paint_ms_;
+ RunningSamples round_trip_ms_;
// Used to update UMA stats, if set.
UpdateUmaCustomHistogramCallback uma_custom_counts_updater_;
« no previous file with comments | « remoting/protocol/capture_scheduler.h ('k') | remoting/remoting_srcs.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698