Mojo: Remove Channel::AttachMessagePipeEndpoint().

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"
@@ skipping 92 matching lines @@
 //         yet.]
 //       - Note that even if the local side is closed, it may still receive a
 //         "remove" message from the other side (if the other side is closed
 //         simultaneously, and both sides send "remove" messages). In that
 //         case, it must still remain alive until it receives the "remove
 //         ack" (and it must ack the "remove" message that it received).
 class MOJO_SYSTEM_IMPL_EXPORT ChannelEndpoint
     : 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();
@@ skipping 47 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_