Adding the base ChromotingInstance implementation and unittests. This class will be used by the ap…

remoting/test/video_renderer.h

Index: remoting/test/video_renderer.h
diff --git a/remoting/test/video_renderer.h b/remoting/test/video_renderer.h
new file mode 100644
index 0000000000000000000000000000000000000000..dd31621928cce83342694d408aaf8982d0ac954e
--- /dev/null
+++ b/remoting/test/video_renderer.h
@@ -0,0 +1,44 @@
+// 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_TEST_VIDEO_RENDERER_H_
+#define REMOTING_TEST_VIDEO_RENDERER_H_
+
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+#include "remoting/client/video_renderer.h"
+#include "remoting/protocol/video_stub.h"
+
+namespace remoting {
+namespace test {
+
+// Processes video packets as they are received from the remote host.
+class VideoRenderer :

[Sergey Ulanov, 2015/03/09 06:44:41]

call it TestVideoRenderer?

[joedow, 2015/03/09 21:09:26]

Done.

+    public remoting::VideoRenderer,
+    public protocol::VideoStub {
+ public:
+  VideoRenderer();
+  ~VideoRenderer() override;
+
+ private:
+  // remoting::VideoRenderer interface.

[Sergey Ulanov, 2015/03/09 06:44:41]

This should be in the public part of this class. We put interface overrides in
the private section only when the fact that the class implements that interface
is an implementation detail. In this case VideoStub can be in private section
because it could be implemented in a separate class and it wouldn't change how
clients of VideoRenderer would use it. In the same time this class must
implement VideoRenderer interface, so implementation of that interface is
public.

[joedow, 2015/03/09 21:09:26]

Done.

+  void OnSessionConfig(const protocol::SessionConfig& config) override;
+  ChromotingStats* GetStats() override;
+  protocol::VideoStub* GetVideoStub() override;
+
+  // protocol::VideoStub interface.
+  void ProcessVideoPacket(
+      scoped_ptr<remoting::VideoPacket> video_packet,
+      const base::Closure& done) override;
+
+  // Track the number of populated video frames which have been received.
+  int video_frames_processed_;
+
+  DISALLOW_COPY_AND_ASSIGN(VideoRenderer);
+};
+
+}  // namespace test
+}  // namespace remoting
+
+#endif  // REMOTING_TEST_VIDEO_RENDERER_H_