Remove DCHECK when handle stuffing fails.

sandbox/win/src/handle_closer_agent.cc

 // Copyright (c) 2012 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 "sandbox/win/src/handle_closer_agent.h"
 
 #include <limits.h>
 #include <stddef.h>
 
 #include "base/logging.h"
@@ skipping 67 matching lines @@
       break;
     if (dup_dummy != closed_handle)
       to_close.push_back(dup_dummy);
   } while (count-- &&
            reinterpret_cast<uintptr_t>(dup_dummy) <
                reinterpret_cast<uintptr_t>(closed_handle));
 
   for (HANDLE h : to_close)
     ::CloseHandle(h);
 
-  // Useful to know when we're not able to stuff handles.
-  DCHECK(dup_dummy == closed_handle);
-
+  // TODO(wfh): Investigate why stuffing handles sometimes fails.
+  // http://crbug.com/649904
   return dup_dummy == closed_handle;
 }
 
 // Reads g_handles_to_close and creates the lookup map.
 void HandleCloserAgent::InitializeHandlesToClose(bool* is_csrss_connected) {
   CHECK(g_handles_to_close != NULL);
 
   // Default to connected state
   *is_csrss_connected = true;
 
@@ skipping 93 matching lines @@
         return false;
       // Attempt to stuff this handle with a new dummy Event.
       AttemptToStuffHandleSlot(handle, result->first);
     }
   }
 
   return true;
 }
 
 }  // namespace sandbox