Add a ChannelEndpointClient abstraction.

mojo/edk/system/channel.h

Index: mojo/edk/system/channel.h
diff --git a/mojo/edk/system/channel.h b/mojo/edk/system/channel.h
index 3460a304e3989cd50e015f1bfddc742242aa9554..010c8c6ea36bd302a16b3d057a60608bf806e000 100644
--- a/mojo/edk/system/channel.h
+++ b/mojo/edk/system/channel.h
@@ -202,9 +202,9 @@ class MOJO_SYSTEM_IMPL_EXPORT Channel
   // about |MessagePipe|. However, we can't just hang on to |ChannelEndpoint|s
   // (even if they have a reference to the |MessagePipe|) since their lifetimes
   // are tied to the "remote" side. When |ChannelEndpoint::OnDisconnect()|

[yzshen1, 2014/11/18 00:28:17]

I think there isn't OnDisconnect anymore

[viettrungluu, 2014/11/18 00:47:29]

Done.

-  // (eventually) results in |ChannelEndpoint::DetachFromMessagePipe()| being
-  // called. We really need to hang on to the "local" side of the message pipe,
-  // to which dispatchers will be "attached".
+  // (eventually) results in |ChannelEndpoint::DetachFromClient()| being called.
+  // We really need to hang on to the "local" side of the message pipe, to which
+  // dispatchers will be "attached".
   IdToMessagePipeMap incoming_message_pipes_;
   // TODO(vtl): We need to keep track of remote IDs (so that we don't collide
   // if/when we wrap).