Gpu: Unregister channel name in the gpu process when a channel closes.

chrome/gpu/gpu_channel.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/gpu/gpu_channel.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
@@ skipping 47 matching lines @@
     OnControlMessageReceived(message);
   } else {
     // Fail silently if the GPU process has destroyed while the IPC message was
     // en-route.
     router_.RouteMessage(message);
   }
 }
 
 void GpuChannel::OnChannelError() {
   static_cast<GpuThread*>(ChildThread::current())->RemoveChannel(renderer_id_);
+#if defined(OS_POSIX)
+  IPC::RemoveAndCloseChannelSocket(GetChannelName());

[apatrick, 2010/11/16 17:48:55]

I think this is the right thing to do but I wonder if calling
RemoveAndCloseChannelCocket should have symmetry with AddChannelSocket, which is
called in Init. Did you try putting it in the GpuChannel destructor? I'm
concerned about the channel being closed for reasons other than an error.

+#endif
 }
 
 bool GpuChannel::Send(IPC::Message* message) {
   if (log_messages_) {
     VLOG(1) << "sending message @" << message << " on channel @" << this
             << " with type " << message->type();
   }
 
   if (!channel_.get()) {
     delete message;
@@ skipping 164 matching lines @@
       channel_name, IPC::Channel::MODE_SERVER, this, NULL,
       ChildProcess::current()->io_message_loop(), false,
       ChildProcess::current()->GetShutDownEvent()));
 
   return true;
 }
 
 std::string GpuChannel::GetChannelName() {
   return StringPrintf("%d.r%d", base::GetCurrentProcId(), renderer_id_);
 }