More cleanups in FrameStats

remoting/protocol/performance_tracker.h

 // Copyright 2015 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.
 
 #ifndef REMOTING_PROTOCOL_PERFORMANCE_TRACKER_H_
 #define REMOTING_PROTOCOL_PERFORMANCE_TRACKER_H_
 
 #include <stdint.h>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/timer/timer.h"
 #include "remoting/base/rate_counter.h"
 #include "remoting/base/running_samples.h"
+#include "remoting/protocol/frame_stats.h"
 
 namespace remoting {
 namespace protocol {
 
-struct FrameStats;
-
 // PerformanceTracker defines a bundle of performance counters and statistics
 // for chromoting.
-class PerformanceTracker {
+class PerformanceTracker : public FrameStatsConsumer {
  public:
   // Callback that updates UMA custom counts or custom times histograms.
   typedef base::Callback<void(const std::string& histogram_name,
                               int64_t value,
                               int histogram_min,
                               int histogram_max,
                               int histogram_buckets)>
       UpdateUmaCustomHistogramCallback;
 
   // Callback that updates UMA enumeration histograms.
   typedef base::Callback<
       void(const std::string& histogram_name, int64_t value, int histogram_max)>
       UpdateUmaEnumHistogramCallback;
 
   PerformanceTracker();
-  virtual ~PerformanceTracker();
+  ~PerformanceTracker() override;
 
   // Constant used to calculate the average for rate metrics and used by the
   // plugin for the frequency at which stats should be updated.
   static const int kStatsUpdatePeriodSeconds = 1;
 
   // Return rates and running-averages for different metrics.
   double video_bandwidth() { return video_bandwidth_.Rate(); }
   double video_frame_rate() { return video_frame_rate_.Rate(); }
   double video_packet_rate() { return video_packet_rate_.Rate(); }
   const RunningSamples& video_capture_ms() { return video_capture_ms_; }
   const RunningSamples& video_encode_ms() { return video_encode_ms_; }
   const RunningSamples& video_decode_ms() { return video_decode_ms_; }
   const RunningSamples& video_paint_ms() { return video_paint_ms_; }
   const RunningSamples& round_trip_ms() { return round_trip_ms_; }
 
-  // Record stats for a video-packet.
-  void RecordVideoFrameStats(const FrameStats& stats);
+  // FrameStatsConsumer interface.
+  void OnVideoFrameStats(const FrameStats& stats) override;
 
   // Sets callbacks in ChromotingInstance to update a UMA custom counts, custom
   // times or enum histogram.
   void SetUpdateUmaCallbacks(
       UpdateUmaCustomHistogramCallback update_uma_custom_counts_callback,
       UpdateUmaCustomHistogramCallback update_uma_custom_times_callback,
       UpdateUmaEnumHistogramCallback update_uma_enum_histogram_callback);
 
   void OnPauseStateChanged(bool paused);
 
@@ skipping 33 matching lines @@
 
   base::RepeatingTimer upload_uma_stats_timer_;
 
   DISALLOW_COPY_AND_ASSIGN(PerformanceTracker);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_PERFORMANCE_TRACKER_H_