Chromium Code Reviews| 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..946627e86a5c7dca02db4c9cff97ac50fade5d9b 100644 |
| --- a/remoting/client/plugin/pepper_video_renderer_3d.cc |
| +++ b/remoting/client/plugin/pepper_video_renderer_3d.cc |
| @@ -189,14 +189,19 @@ void PepperVideoRenderer3D::ProcessVideoPacket(scoped_ptr<VideoPacket> packet, |
| const base::Closure& done) { |
| base::ScopedClosureRunner done_runner(done); |
| + stats_.video_packet_rate_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()) |
| return; |
| // Update statistics. |
| - stats_.video_frame_rate()->Record(1); |
| - stats_.video_bandwidth()->Record(packet->data().size()); |
| + // TODO(anandc): The following tasks are duplicated in |
| + // software_video_renderer.cc. Move to ChromotingStats. |
|
Wez
2015/07/08 23:42:30
nit: File a bug for that work, and refer to it in
anandc
2015/07/09 19:17:10
Done.
|
| + stats_.video_frame_rate_display()->Record(1); |
| + stats_.video_frame_rate_uma()->Record(1); |
| + stats_.video_bandwidth_display()->Record(packet->data().size()); |
| + stats_.video_bandwidth_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()) |