Simplify VideoReader and VideoWriter

remoting/protocol/client_video_dispatcher.cc

+// 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.
+
+#include "remoting/protocol/client_video_dispatcher.h"
+
+#include "base/bind.h"
+#include "net/socket/stream_socket.h"
+#include "remoting/base/constants.h"
+#include "remoting/proto/video.pb.h"
+#include "remoting/protocol/video_stub.h"
+
+namespace remoting {
+namespace protocol {
+
+ClientVideoDispatcher::ClientVideoDispatcher()
+    : ChannelDispatcherBase(kVideoChannelName),
+      video_stub_(NULL) {
+}
+
+ClientVideoDispatcher::~ClientVideoDispatcher() {
+}
+
+void ClientVideoDispatcher::OnInitialized() {
+  reader_.Init(channel(), base::Bind(&ClientVideoDispatcher::OnVideoPacket,
+                                     base::Unretained(this)));
+}
+
+void ClientVideoDispatcher::OnVideoPacket(scoped_ptr<VideoPacket> packet,
+                                          const base::Closure& done_task) {
+  video_stub_->ProcessVideoPacket(packet.Pass(), done_task);

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

nit to consider for a future CL: Since ProtobufVideoReader is also part of
remoting/protocol, perhaps we should just be Init()ing it with the VideoStub
directly, and cut out this middle-man?

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

Done.

+}
+
+}  // namespace protocol
+}  // namespace remoting