Show framerate in chromoting debug log

remoting/client/chromoting_stats.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/client/chromoting_stats.h"
 
 namespace {
 
 // The default window of bandwidth in seconds. Bandwidth is reported as
-// number of numbers received in this time frame.
-static const int kBandwidthWindow = 3;
+// number of bytes received in this time frame.
+static const base::TimeDelta kBandwidthWindow =
+    base::TimeDelta::FromSeconds(3);

[Wez, 2011/10/21 18:01:43]

I think these still need to be const int, and be converted to TimeDelta only
where necessary, otherwise you're adding static initializers.

+
+// Framerate is reported as number of frames received over the following
+// time window in seconds.
+static const base::TimeDelta kFramerateWindow =
+    base::TimeDelta::FromSeconds(3);

[Wez, 2011/10/21 18:01:43]

Why do we need separate frame-rate and bandwidth windows?  Seems logical to use
the same one.

 
 // We take the last 10 latency numbers and report the average.
 static const int kLatencyWindow = 10;
 
 }  // namespace
 
 namespace remoting {
 
 ChromotingStats::ChromotingStats()
-    : video_bandwidth_(base::TimeDelta::FromSeconds(kBandwidthWindow)),
+    : video_bandwidth_(kBandwidthWindow),
+      video_frame_rate_(kFramerateWindow),
       video_capture_ms_(kLatencyWindow),
       video_encode_ms_(kLatencyWindow),
       video_decode_ms_(kLatencyWindow),
       video_paint_ms_(kLatencyWindow),
       round_trip_ms_(kLatencyWindow) {
 }
 
 ChromotingStats::~ChromotingStats() {
 }
 
 }  // namespace remoting