Mojo: Remove MessagePipe::OnRemove(); instead just use MessagePipe::Close().

mojo/system/message_pipe.h

 // Copyright 2013 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_MESSAGE_PIPE_H_
 #define MOJO_SYSTEM_MESSAGE_PIPE_H_
 
 #include <stdint.h>
 
 #include <vector>
@@ skipping 77 matching lines @@
   // This is called by the dispatcher to convert a local endpoint to a proxy
   // endpoint.
   scoped_refptr<ChannelEndpoint> ConvertLocalToProxy(unsigned port);
 
   // This is used by |Channel| to enqueue messages (typically to a
   // |LocalMessagePipeEndpoint|). Unlike |WriteMessage()|, |port| is the
   // *destination* port.
   MojoResult EnqueueMessage(unsigned port,
                             scoped_ptr<MessageInTransit> message);
 
-  // This is used by |Channel|. TODO(vtl): Rename it (and have the
-  // |ChannelEndpoint| call it instead).
-  void OnRemove(unsigned port);
-
  private:
   MessagePipe();
 
   friend class base::RefCountedThreadSafe<MessagePipe>;
   virtual ~MessagePipe();
 
   // This is used internally by |WriteMessage()| and by |EnqueueMessage()|.
   // |transports| may be non-null only if it's nonempty and |message| has no
   // dispatchers attached.
   MojoResult EnqueueMessageInternal(
@@ skipping 15 matching lines @@
   base::Lock lock_;  // Protects the following members.
   scoped_ptr<MessagePipeEndpoint> endpoints_[2];
 
   DISALLOW_COPY_AND_ASSIGN(MessagePipe);
 };
 
 }  // namespace system
 }  // namespace mojo
 
 #endif  // MOJO_SYSTEM_MESSAGE_PIPE_H_