Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(445)

Unified Diff: remoting/protocol/fake_video_renderer.h

Issue 2113523007: More cleanups in FrameStats (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/software_video_renderer.cc ('k') | remoting/protocol/fake_video_renderer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/fake_video_renderer.h
diff --git a/remoting/protocol/fake_video_renderer.h b/remoting/protocol/fake_video_renderer.h
index 0b8115acc9ecf70e45f227ee34ea2ffabd134908..ced8b6f78754b4de4a119e61a9aecd1b0d728ae4 100644
--- a/remoting/protocol/fake_video_renderer.h
+++ b/remoting/protocol/fake_video_renderer.h
@@ -11,6 +11,7 @@
#include "base/callback.h"
#include "base/threading/thread_checker.h"
#include "remoting/protocol/frame_consumer.h"
+#include "remoting/protocol/frame_stats.h"
#include "remoting/protocol/video_renderer.h"
#include "remoting/protocol/video_stub.h"
@@ -64,6 +65,25 @@ class FakeFrameConsumer : public FrameConsumer {
base::Closure on_frame_callback_;
};
+class FakeFrameStatsConsumer : public FrameStatsConsumer {
+ public:
+ FakeFrameStatsConsumer();
+ ~FakeFrameStatsConsumer() override;
+
+ const std::list<FrameStats>& received_stats() { return received_stats_; }
+
+ void set_on_stats_callback(base::Closure on_stats_callback);
+
+ // FrameStatsConsumer interface.
+ void OnVideoFrameStats(const FrameStats& stats) override;
+
+ private:
+ base::ThreadChecker thread_checker_;
+
+ std::list<FrameStats> received_stats_;
+ base::Closure on_stats_callback_;
+};
+
class FakeVideoRenderer : public VideoRenderer {
public:
FakeVideoRenderer();
@@ -71,16 +91,18 @@ class FakeVideoRenderer : public VideoRenderer {
// VideoRenderer interface.
bool Initialize(const ClientContext& client_context,
- protocol::PerformanceTracker* perf_tracker) override;
+ protocol::FrameStatsConsumer* stats_consumer) override;
void OnSessionConfig(const SessionConfig& config) override;
FakeVideoStub* GetVideoStub() override;
FakeFrameConsumer* GetFrameConsumer() override;
+ FakeFrameStatsConsumer* GetFrameStatsConsumer() override;
private:
base::ThreadChecker thread_checker_;
FakeVideoStub video_stub_;
FakeFrameConsumer frame_consumer_;
+ FakeFrameStatsConsumer frame_stats_consumer_;
};
} // namespace protocol
« no previous file with comments | « remoting/client/software_video_renderer.cc ('k') | remoting/protocol/fake_video_renderer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698