Simplify VideoReader and VideoWriter

remoting/protocol/host_video_dispatcher.h

+// Copyright 2014 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_HOST_VIDEO_DISPATCHER_H_
+#define REMOTING_PROTOCOL_HOST_VIDEO_DISPATCHER_H_
+
+#include <string>
+
+#include "base/compiler_specific.h"
+#include "remoting/protocol/buffered_socket_writer.h"
+#include "remoting/protocol/channel_dispatcher_base.h"
+#include "remoting/protocol/video_stub.h"
+
+namespace remoting {
+namespace protocol {
+
+class HostVideoDispatcher : public ChannelDispatcherBase, public VideoStub {
+ public:
+  HostVideoDispatcher();
+  virtual ~HostVideoDispatcher();
+
+  // VideoStub interface.
+  virtual void ProcessVideoPacket(scoped_ptr<VideoPacket> packet,
+                                  const base::Closure& done) OVERRIDE;
+
+ private:
+  void OnInitialized();

[Wez, 2014/09/18 00:23:55]

// ChannelDispatcherBase overrides.
virtual void OnInitialized() OVERRIDE;

[Wez, 2014/09/18 00:23:55]

Should this be protected, as in the client impl, rather than private?

[Sergey Ulanov, 2014/09/18 22:35:00]

Done.

[Sergey Ulanov, 2014/09/18 22:35:00]

Done.

+
+

[Wez, 2014/09/18 00:23:55]

nit: Errant blank line!

[Sergey Ulanov, 2014/09/18 22:35:00]

Done.

+  BufferedSocketWriter buffered_writer_;

[Wez, 2014/09/18 00:23:55]

nit: In the ClientVideoDispatcher, the ProtobufVideoReader is just called
reader_, so consider calling this writer_?

[Sergey Ulanov, 2014/09/18 22:35:00]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(HostVideoDispatcher);
+};
+
+}  // namespace protocol
+}  // namespace remoting
+
+#endif  // REMOTING_PROTOCOL_HOST_VIDEO_DISPATCHER_H_