Show max latency on client's status bar

remoting/base/running_samples.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.

[Sergey Ulanov, 2016/03/18 19:42:06]

Please update year for moved or renamed files. Also please remove (c).
https://www.chromium.org/developers/coding-style#TOC-File-headers

[Yuwei, 2016/03/18 20:05:59]

Acknowledged.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef REMOTING_BASE_RUNNING_AVERAGE_H_
-#define REMOTING_BASE_RUNNING_AVERAGE_H_
+#ifndef REMOTING_BASE_RUNNING_SAMPLES_H_
+#define REMOTING_BASE_RUNNING_SAMPLES_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <deque>
 
 #include "base/macros.h"
 #include "base/threading/thread_checker.h"
 
 namespace remoting {
 
 // Calculates the average of the most recent N recorded samples.

[Sergey Ulanov, 2016/03/18 19:42:06]

update this comment

[Yuwei, 2016/03/18 20:05:59]

Acknowledged.

 // This is typically used to smooth out random variation in point samples
 // over bandwidth, frame rate, etc.
-class RunningAverage {
+class RunningSamples {
  public:
   // Constructs a helper to average over the |window_size| most recent samples.
-  explicit RunningAverage(int window_size);
-  virtual ~RunningAverage();
+  explicit RunningSamples(int window_size);
+  virtual ~RunningSamples();
 
   // Records a point sample.
   void Record(int64_t value);
 
   // Returns the average over up to |window_size| of the most recent samples.
   double Average();

[Sergey Ulanov, 2016/03/18 19:42:06]

mark this const.

[Yuwei, 2016/03/18 20:05:59]

Acknowledged.

 
+  // Returns the max over up to |window_size| of the most recent samples.
+  // 0 if no sample available
+  int64_t Max();

[Sergey Ulanov, 2016/03/18 19:42:06]

const

[Yuwei, 2016/03/18 20:05:59]

Acknowledged.

+
  private:
   // Stores the desired window size, as size_t to avoid casting when comparing
   // with the size of |data_points_|.
   const size_t window_size_;
 
   // Stores the |window_size| most recently recorded samples.
   std::deque<int64_t> data_points_;
 
   // Holds the sum of the samples in |data_points_|.
   int64_t sum_;
 
   base::ThreadChecker thread_checker_;
 
-  DISALLOW_COPY_AND_ASSIGN(RunningAverage);
+  DISALLOW_COPY_AND_ASSIGN(RunningSamples);
 };
 
 }  // namespace remoting
 
-#endif  // REMOTING_BASE_RUNNING_AVERAGE_H_
+#endif  // REMOTING_BASE_RUNNING_SAMPLES_H_