Show max latency on client's status bar

remoting/client/plugin/chromoting_instance.cc

 // Copyright (c) 2012 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/plugin/chromoting_instance.h"
 
 #include <nacl_io/nacl_io.h>
 #include <sys/mount.h>
 
 #include <string>
@@ skipping 1016 matching lines @@
   data->SetString("iq", iq);
   PostLegacyJsonMessage("sendOutgoingIq", std::move(data));
 }
 
 void ChromotingInstance::UpdatePerfStatsInUI() {
   // Fetch performance stats from the VideoRenderer and send them to the client
   // for display to users.
   scoped_ptr<base::DictionaryValue> data(new base::DictionaryValue());
   data->SetDouble("videoBandwidth", perf_tracker_.video_bandwidth());
   data->SetDouble("videoFrameRate", perf_tracker_.video_frame_rate());
-  data->SetDouble("captureLatency", perf_tracker_.video_capture_ms());
-  data->SetDouble("encodeLatency", perf_tracker_.video_encode_ms());
-  data->SetDouble("decodeLatency", perf_tracker_.video_decode_ms());
-  data->SetDouble("renderLatency", perf_tracker_.video_paint_ms());
-  data->SetDouble("roundtripLatency", perf_tracker_.round_trip_ms());
+  data->SetDouble("captureLatency", perf_tracker_.video_capture_ms().Average());
+  data->SetDouble("encodeLatency", perf_tracker_.video_encode_ms().Average());
+  data->SetDouble("decodeLatency", perf_tracker_.video_decode_ms().Average());
+  data->SetDouble("renderLatency", perf_tracker_.video_paint_ms().Average());
+  data->SetDouble("roundtripLatency", perf_tracker_.round_trip_ms().Average());
+  data->SetDouble("maxCaptureLatency", perf_tracker_.video_capture_ms().Max());

[Sergey Ulanov, 2016/03/21 19:35:06]

maybe reorder these so max is always after average?

[Yuwei, 2016/03/21 19:59:40]

Sure. Then duplicating lines will be easier

+  data->SetDouble("maxEncodeLatency", perf_tracker_.video_encode_ms().Max());
+  data->SetDouble("maxDecodeLatency", perf_tracker_.video_decode_ms().Max());
+  data->SetDouble("maxRenderLatency", perf_tracker_.video_paint_ms().Max());
+  data->SetDouble("maxRoundtripLatency", perf_tracker_.round_trip_ms().Max());
   PostLegacyJsonMessage("onPerfStats", std::move(data));
 }
 
 // static
 void ChromotingInstance::RegisterLogMessageHandler() {
   base::AutoLock lock(g_logging_lock.Get());
 
   // Set up log message handler.
   // This is not thread-safe so we need it within our lock.
   logging::SetLogMessageHandler(&LogToUI);
@@ skipping 80 matching lines @@
   if (is_custom_counts_histogram) {
     uma.HistogramCustomCounts(histogram_name, value, histogram_min,
                               histogram_max, histogram_buckets);
   } else {
     uma.HistogramCustomTimes(histogram_name, value, histogram_min,
                              histogram_max, histogram_buckets);
   }
 }
 
 }  // namespace remoting