Chromium Code Reviews| Index: remoting/client/chromoting_stats.cc |
| diff --git a/remoting/client/chromoting_stats.cc b/remoting/client/chromoting_stats.cc |
| index 9acaedc7664512730aa6d9854f062ac6a834bda8..32d6eba9b0d377cb5f6eb0153b18d4bf965dfd07 100644 |
| --- a/remoting/client/chromoting_stats.cc |
| +++ b/remoting/client/chromoting_stats.cc |
| @@ -7,9 +7,13 @@ |
| namespace { |
| // The default window of bandwidth in seconds. Bandwidth is reported as |
| -// number of numbers received in this time frame. |
| +// number of bytes received in this time frame. |
| static const int kBandwidthWindow = 3; |
| +// Framerate is reported as number of frames received over the following |
| +// time window. |
|
Wez
2011/10/15 21:21:39
nit: Specify the units.
Jamie
2011/10/15 22:38:27
Or use a TimeDelta object. Not sure if that's allo
|
| +static const int kFramerateWindow = 3; |
| + |
| // We take the last 10 latency numbers and report the average. |
| static const int kLatencyWindow = 10; |
| @@ -19,6 +23,7 @@ namespace remoting { |
| ChromotingStats::ChromotingStats() |
| : video_bandwidth_(base::TimeDelta::FromSeconds(kBandwidthWindow)), |
| + video_frame_rate_(base::TimeDelta::FromSeconds(kFramerateWindow)), |
| video_capture_ms_(kLatencyWindow), |
| video_encode_ms_(kLatencyWindow), |
| video_decode_ms_(kLatencyWindow), |