Remove event_channel() and control_channel() from Session interface.

remoting/protocol/channel_dispatcher_base.h

+// Copyright (c) 2011 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_CHANNEL_DISPATCHER_BASE_H_
+#define REMOTING_PROTOCOL_CHANNEL_DISPATCHER_BASE_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace net {
+class StreamSocket;
+}  // namespace net
+
+namespace remoting {
+namespace protocol {
+
+class Session;
+
+class ChannelDispatcherBase {

[Wez, 2011/11/17 22:39:41]

What is this class used for?

[Sergey Ulanov, 2011/11/17 23:10:38]

Done.

+ public:
+  // The callback is called when initialization is finished. The
+  // parameter is set to true on success.
+  typedef base::Callback<void(bool)> InitializedCallback;
+
+  explicit ChannelDispatcherBase(const char* name);

[Wez, 2011/11/17 22:39:41]

What does |name| refer to?

[Sergey Ulanov, 2011/11/17 23:10:38]

renamed channel_name_

+  virtual ~ChannelDispatcherBase();
+
+  // Initializes the channel the |session|.

[Wez, 2011/11/17 22:39:41]

typo?

[Sergey Ulanov, 2011/11/17 23:10:38]

Done.

+  void Init(Session* session, const InitializedCallback& callback);
+
+ protected:
+  net::StreamSocket* channel() { return channel_.get(); }
+
+  // Following method should be overriden in the child classes.
+  virtual void OnInitialized() = 0;
+
+ private:
+  void OnChannelReady(net::StreamSocket* socket);
+
+  std::string name_;
+  Session* session_;
+  InitializedCallback initialized_callback_;
+  scoped_ptr<net::StreamSocket> channel_;
+
+  DISALLOW_COPY_AND_ASSIGN(ChannelDispatcherBase);
+};
+
+}  // namespace protocol
+}  // namespace remoting
+
+#endif  // REMOTING_PROTOCOL_CHANNEL_DISPATCHER_BASE_H_