Report video and network stats averaged over 1s, and create corresponding UMA metrics.

remoting/client/chromoting_stats.h

Index: remoting/client/chromoting_stats.h
diff --git a/remoting/client/chromoting_stats.h b/remoting/client/chromoting_stats.h
index da7601b7f9222d3026fe768713db067a8a4c0b78..326a0ede7bfe0eb1844393e1e7f9a50ccd793c39 100644
--- a/remoting/client/chromoting_stats.h
+++ b/remoting/client/chromoting_stats.h
@@ -20,6 +20,11 @@ class ChromotingStats {
 
   RateCounter* video_bandwidth() { return &video_bandwidth_; }

[Wez, 2015/06/29 14:56:13]

While you're here, can we add aunits suffix to this member, please?

[anandc, 2015/06/30 20:17:39]

I've used Bps, to indicate Bytes/sec.

   RateCounter* video_frame_rate() { return &video_frame_rate_; }
+  RateCounter* video_fps()  { return &video_fps_; }

[Wez, 2015/06/29 14:56:13]

What is the difference between this and video_frame_rate(), above...?

[anandc, 2015/06/30 20:17:38]

Video frame-rate has a time-window of 3s. This one has a time-window of 1s.

I've added a suffix to indicate the way these metrics are used.

+  RateCounter* video_bytes_per_s() { return &video_bytes_per_s_; }

[Wez, 2015/06/29 14:56:13]

Similarly, consider clarifying how this differs from "bandwidth", above?

[anandc, 2015/06/30 20:17:38]

Done.

+  RateCounter* video_fps_with_empty_frames() {

[Wez, 2015/06/29 14:56:13]

There is a 1:1 correspondence between video frames and video packets (aka
VideoPacket in our protocol) - it may be clearer to use video_packets_per_s()
here.

You may wish to add comments to each of these fields to explain their
purpose/limitations, etc (e.g. the fact that video_fps() will be zero if there
are no actual on-screen changes to deliver.

[anandc, 2015/06/30 20:17:38]

Done.

+    return &video_fps_with_empty_frames_;
+  }
   RunningAverage* video_capture_ms() { return &video_capture_ms_; }
   RunningAverage* video_encode_ms() { return &video_encode_ms_; }
   RunningAverage* video_decode_ms() { return &video_decode_ms_; }
@@ -29,6 +34,9 @@ class ChromotingStats {
  private:
   RateCounter video_bandwidth_;
   RateCounter video_frame_rate_;
+  RateCounter video_fps_;
+  RateCounter video_bytes_per_s_;
+  RateCounter video_fps_with_empty_frames_;
   RunningAverage video_capture_ms_;
   RunningAverage video_encode_ms_;
   RunningAverage video_decode_ms_;