Mojo: Remove knowledge of Channel, etc. from ProxyMessagePipeEndpoint.

mojo/system/channel_endpoint.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 MOJO_SYSTEM_CHANNEL_ENDPOINT_H_
 #define MOJO_SYSTEM_CHANNEL_ENDPOINT_H_
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/synchronization/lock.h"
 #include "mojo/system/message_in_transit.h"
 #include "mojo/system/system_impl_export.h"
 
 namespace mojo {
 namespace system {
 
 class Channel;
 class MessagePipe;
 
@@ skipping 89 matching lines @@
     : public base::RefCountedThreadSafe<ChannelEndpoint> {
  public:
   // TODO(vtl): More comments....
   // The caller must keep |channel| alive until the |Channel| has taken
   // ownership of a reference to the created object. (TODO(vtl): Convert
   // |Channel::AttachMessagePipeEndpoint()| to a |Channel::AttachEndpoint()|
   // that takes a |ChannelEndpoint|, and move |ChannelEndpoint| creation out of
   // |Channel|.)
   ChannelEndpoint(MessagePipe* message_pipe, unsigned port);
 
+  // Methods called by |MessagePipe| (via |ProxyMessagePipeEndpoint|):
+
+  // TODO(vtl): This currently only works if we're "running". We'll move the
+  // "paused message queue" here (will this be needed when we have
+  // locally-allocated remote IDs?).
+  bool EnqueueMessage(scoped_ptr<MessageInTransit> message);
+
+  void DetachFromMessagePipe();
+
+  // Methods called by |Channel|:
+
+  // Called by |Channel| when it takes a reference to this object.
   void AttachToChannel(Channel* channel, MessageInTransit::EndpointId local_id);
+
+  // TODO(vtl): Combine this with |AttachToChannel()|.
   void Run(MessageInTransit::EndpointId remote_id);
 
   // Called by |Channel| before it gives up its reference to this object.
   void DetachFromChannel();
 
  private:
   enum State {
     // Attached, possibly running or not.
     STATE_NORMAL,
     // "Zombie" states:
@@ skipping 26 matching lines @@
   MessageInTransit::EndpointId local_id_;
   MessageInTransit::EndpointId remote_id_;
 
   DISALLOW_COPY_AND_ASSIGN(ChannelEndpoint);
 };
 
 }  // namespace system
 }  // namespace mojo
 
 #endif  // MOJO_SYSTEM_CHANNEL_ENDPOINT_H_