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 1019 matching lines @@
 
 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());

[Sergey Ulanov, 2016/03/17 19:12:37]

I think ideally we want to have  max values for all other *Latency values as
well.

[Yuwei, 2016/03/17 20:24:44]

so we will have *Latency for average latency and max*Latency for max latency,
right?

[Yuwei, 2016/03/17 22:00:38]

We can have max values for all latencies but we probably don't have enough space
(UI) to show all of them on the status bar...

   data->SetDouble("roundtripLatency", perf_tracker_.round_trip_ms());
+  data->SetDouble("maxRoundtripLatency", perf_tracker_.max_round_trip_ms());
   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