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

remoting/client/plugin/pepper_video_renderer_3d.cc

Index: remoting/client/plugin/pepper_video_renderer_3d.cc
diff --git a/remoting/client/plugin/pepper_video_renderer_3d.cc b/remoting/client/plugin/pepper_video_renderer_3d.cc
index 711c2104b702cf083f78a4b4b337286fb20eb91e..e251f910b048dcd10ba726641ac40a734314113e 100644
--- a/remoting/client/plugin/pepper_video_renderer_3d.cc
+++ b/remoting/client/plugin/pepper_video_renderer_3d.cc
@@ -189,14 +189,18 @@ void PepperVideoRenderer3D::ProcessVideoPacket(scoped_ptr<VideoPacket> packet,
                                                const base::Closure& done) {
   base::ScopedClosureRunner done_runner(done);
 
+  stats_.video_packets_per_s_UMA()->Record(1);
   // Don't need to do anything if the packet is empty. Host sends empty video
   // packets when the screen is not changing.
-  if (!packet->data().size())
+  if (!packet->data().size()) {

[Sergey Ulanov, 2015/07/06 21:23:12]

please don't add {} for consistency with other single-line if statements in this
file.

[anandc, 2015/07/07 01:11:43]

Yes, didn't clean this up from previous patch-set where we had multiple lines
here.

     return;
+  }
 
   // Update statistics.

[Sergey Ulanov, 2015/07/06 21:23:12]

This code is duplicated in SoftwareVideoRenderer. I think ideally it should be
moved to ChromotingStats. Please add TODO here.

[anandc, 2015/07/07 01:11:43]

Done.

-  stats_.video_frame_rate()->Record(1);
-  stats_.video_bandwidth()->Record(packet->data().size());
+  stats_.video_fps_webapp()->Record(1);
+  stats_.video_fps_UMA()->Record(1);
+  stats_.video_Bps_webapp()->Record(packet->data().size());
+  stats_.video_Bps_UMA()->Record(packet->data().size());
   if (packet->has_capture_time_ms())
     stats_.video_capture_ms()->Record(packet->capture_time_ms());
   if (packet->has_encode_time_ms())