Remove CHECKs used for bug investigation.

content/renderer/pepper_plugin_delegate_impl.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 "content/renderer/pepper_plugin_delegate_impl.h"
 
 #include <cmath>
 #include <queue>
 
 #include "base/bind.h"
@@ skipping 512 matching lines @@
 
   bool Init(base::ProcessHandle plugin_process_handle,
             const IPC::ChannelHandle& channel_handle,
             PP_Module pp_module,
             ppapi::proxy::Dispatcher::GetInterfaceFunc local_get_interface,
             const ppapi::Preferences& preferences) {
     if (channel_handle.name.empty())
       return false;
 
 #if defined(OS_POSIX)
-    // Check the validity of fd for bug investigation. Remove after fixed.
-    // See for details: crbug.com/103957.
-    CHECK_NE(-1, channel_handle.socket.fd);

[piman, 2012/02/03 18:52:22]

Should we leave this as a DCHECK to catch regressions? The bug suggests the root
cause isn't entirely known even though it's fixed.

     if (channel_handle.socket.fd == -1)
       return false;
 #endif
 
     dispatcher_delegate_.reset(new DispatcherDelegate);
     dispatcher_.reset(new ppapi::proxy::HostDispatcher(
         plugin_process_handle, pp_module, local_get_interface));
 
     if (!dispatcher_->InitHostWithChannel(dispatcher_delegate_.get(),
                                           channel_handle,
@@ skipping 1562 matching lines @@
 void PepperPluginDelegateImpl::UnSetAndDeleteLockTargetAdapter(
     webkit::ppapi::PluginInstance* instance) {
   LockTargetMap::iterator it = mouse_lock_instances_.find(instance);
   if (it != mouse_lock_instances_.end()) {
     MouseLockDispatcher::LockTarget* target = it->second;
     render_view_->mouse_lock_dispatcher()->OnLockTargetDestroyed(target);
     delete target;
     mouse_lock_instances_.erase(it);
   }
 }