Show max latency on client's status bar

remoting/webapp/base/js/connection_stats.js

 // 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.
 
 /**
  * @fileoverview
  * Module to support debug overlay window with connection stats.
  */
 
 'use strict';
@@ skipping 109 matching lines @@
     }
   }
 
   /**
    * @param {number} value
    * @param {string} units
    * @return {string} Formatted number.
    */
   function formatStatNumber(value, units) {
     if (value != undefined) {
       return value.toFixed(2) + ' ' + units;

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

Please change this to 1 instead of 2. All average values are calculated from 10
integer samples, so the second digit is always 0, so we don't need to display
it. Also the Max value will always be integer, so it can be formatted as
integer. Maybe pass number of digits after period as a parameter to this
function?

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

Acknowledged.

     } else {
       return "n/a";
     }
   }
 
   var statistics = document.getElementById('statistics');
   this.statsElement_.innerText = (
       'Bandwidth: ' + formatStatNumber(videoBandwidth, units) +
       ', Frame Rate: ' + formatStatNumber(stats.videoFrameRate, 'fps') +
       ', Capture: ' + formatStatNumber(stats.captureLatency, 'ms') +
       ', Encode: ' + formatStatNumber(stats.encodeLatency, 'ms') +
       ', Decode: ' + formatStatNumber(stats.decodeLatency, 'ms') +
       ', Render: ' + formatStatNumber(stats.renderLatency, 'ms') +
-      ', Latency: ' + formatStatNumber(stats.roundtripLatency, 'ms'));
+      ', Avg. Latency: ' + formatStatNumber(stats.roundtripLatency, 'ms') +
+      ', Max latency: ' + formatStatNumber(stats.maxRoundtripLatency, 'ms'));
 };