Cleanup channel dispatchers

Now ChannelDispatcherBase handles initialization of reader and writer,
which makes all dispatcher classes simpler. ChannelDispatcherBase now
can handle writer errors. Also renamed
ProtobufMessageReader -> ProtobufMessageParser.

Review URL: https://codereview.chromium.org/841773005

Cr-Commit-Position: refs/heads/master@{#310993}

1 files changed, 7 insertions, 4 deletions

diff --git a/remoting/protocol/connection_to_client.h b/remoting/protocol/connection_to_client.h
index 8f075b7..11f4a34 100644
--- a/remoting/protocol/connection_to_client.h
+++ b/remoting/protocol/connection_to_client.h
@@ -31,7 +31,8 @@ class VideoStub;
 // host. It sets up all protocol channels and connects them to the
 // stubs.
 class ConnectionToClient : public base::NonThreadSafe,
-                           public Session::EventHandler {
+                           public Session::EventHandler,
+                           public ChannelDispatcherBase::EventHandler {
  public:
   class EventHandler {
    public:
@@ -105,10 +106,12 @@ class ConnectionToClient : public base::NonThreadSafe,
   void OnSessionRouteChange(const std::string& channel_name,
                             const TransportRoute& route) override;
 
- private:
-  // Callback for channel initialization.
-  void OnChannelInitialized(bool successful);
+  // ChannelDispatcherBase::EventHandler interface.
+  void OnChannelInitialized(ChannelDispatcherBase* channel_dispatcher) override;
+  void OnChannelError(ChannelDispatcherBase* channel_dispatcher,
+                      ErrorCode error) override;
 
+ private:
   void NotifyIfChannelsReady();
 
   void Close(ErrorCode error);