More cleanups in FrameStats

remoting/protocol/fake_video_renderer.cc

Index: remoting/protocol/fake_video_renderer.cc
diff --git a/remoting/protocol/fake_video_renderer.cc b/remoting/protocol/fake_video_renderer.cc
index 8dfff378210c9ad4c4ff17448fb91fa5d3ab1d21..bcad0e56ec2667e29d0fc64cfd2a4a8583e17a5f 100644
--- a/remoting/protocol/fake_video_renderer.cc
+++ b/remoting/protocol/fake_video_renderer.cc
@@ -63,11 +63,27 @@ FrameConsumer::PixelFormat FakeFrameConsumer::GetPixelFormat() {
   return FORMAT_BGRA;
 }
 
+FakeFrameStatsConsumer::FakeFrameStatsConsumer() {}
+FakeFrameStatsConsumer::~FakeFrameStatsConsumer() {}
+
+void FakeFrameStatsConsumer::set_on_stats_callback(
+    base::Closure on_stats_callback) {
+  on_stats_callback_ = on_stats_callback;
+}
+
+void FakeFrameStatsConsumer::OnVideoFrameStats(const FrameStats& stats) {
+  CHECK(thread_checker_.CalledOnValidThread());
+  received_stats_.push_back(stats);
+  if (!on_stats_callback_.is_null())
+    on_stats_callback_.Run();
+}
+
 FakeVideoRenderer::FakeVideoRenderer() {}
 FakeVideoRenderer::~FakeVideoRenderer() {}
 
-bool FakeVideoRenderer::Initialize(const ClientContext& client_context,
-                                   protocol::PerformanceTracker* perf_tracker) {
+bool FakeVideoRenderer::Initialize(
+    const ClientContext& client_context,
+    protocol::FrameStatsConsumer* stats_consumer) {
   return true;
 }
 
@@ -83,5 +99,10 @@ FakeFrameConsumer* FakeVideoRenderer::GetFrameConsumer() {
   return &frame_consumer_;
 }
 
+FakeFrameStatsConsumer* FakeVideoRenderer::GetFrameStatsConsumer() {
+  CHECK(thread_checker_.CalledOnValidThread());
+  return &frame_stats_consumer_;
+}
+
 }  // namespace protocol
 }  // namespace remoting