Add more CHECK in GPU and NPAPI process on invalid file descriptors

content/common/gpu/gpu_channel_manager.cc

Index: content/common/gpu/gpu_channel_manager.cc
diff --git a/content/common/gpu/gpu_channel_manager.cc b/content/common/gpu/gpu_channel_manager.cc
index cacb7eb61b815cf4b903f543d8af5134e34af5d2..f31de5ba51f7613ed844affdee7ddb6f786f16be 100644
--- a/content/common/gpu/gpu_channel_manager.cc
+++ b/content/common/gpu/gpu_channel_manager.cc
@@ -77,10 +77,12 @@ void GpuChannelManager::OnEstablishChannel(int renderer_id) {
   content::GPUInfo gpu_info;
 
   GpuChannelMap::const_iterator iter = gpu_channels_.find(renderer_id);
-  if (iter == gpu_channels_.end())
+  if (iter == gpu_channels_.end()) {
     channel = new GpuChannel(this, watchdog_, renderer_id, false);
-  else
+  } else {
+    CHECK(false);

[ddorwin, 2011/12/08 19:17:27]

If this is temporary, there should be a TODO to clean this up. Otherwise, we
should just CHECK line 80 and remove new lines 82-85.

[xhwang, 2011/12/08 19:37:49]

This should be temporary as I am not confident that this CHECK is totally safe
in reality (as per piman's comment in
http://code.google.com/p/chromium/issues/detail?id=95732#c41).  It may be
improved later but is out of the scope of this investigation.

     channel = iter->second;
+  }
 
   DCHECK(channel != NULL);