RenderHostImpl::AddRoute - Turn a DCHECK to CHECK()

content/browser/renderer_host/render_process_host_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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <vector>
 
 #if defined(OS_POSIX)
 #include <utility>  // for pair<>
 #endif
 
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
+#include "base/debug/alias.h"
 #include "base/debug/trace_event.h"
 #include "base/files/file.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/path_service.h"
 #include "base/process/process_handle.h"
 #include "base/rand_util.h"
 #include "base/stl_util.h"
@@ skipping 878 matching lines @@
 
 void RenderProcessHostImpl::NotifyTimezoneChange() {
   Send(new ViewMsg_TimezoneChange());
 }
 
 ServiceRegistry* RenderProcessHostImpl::GetServiceRegistry() {
   DCHECK(mojo_application_host_);
   return mojo_application_host_->service_registry();
 }
 
+namespace {
+struct DebugInfoToCapture {
+ public:
+  DebugInfoToCapture(int32 rid, int pid)
+      : routing_id(rid), process_id(pid) {
+    signature[0] = 'I';
+    signature[1] = 'N';
+    signature[2] = 'F';
+    signature[3] = 'O';
+  }
+
+  char signature[4];
+  int32 routing_id;
+  int process_id;
+};
+}
+
 void RenderProcessHostImpl::AddRoute(
     int32 routing_id,
     IPC::Listener* listener) {
-  DCHECK(widget_helper_->IsRoutingIDProbablyValid(routing_id))
-      << "Found Routing ID conflicts: " << routing_id;
+  DebugInfoToCapture info(routing_id, GetID());
+  base::debug::Alias(&info);
+  CHECK(widget_helper_->IsRoutingIDProbablyValid(routing_id))
+      << "Found Invalid Routing ID: " << routing_id;
+  CHECK(!listeners_.Lookup(routing_id))
+      << "Found Routing ID conflicts: " << routing_id
+      << " with Child PID: " << GetID();
   listeners_.AddWithID(listener, routing_id);
 }
 
 void RenderProcessHostImpl::RemoveRoute(int32 routing_id) {
   DCHECK(listeners_.Lookup(routing_id) != NULL);
   listeners_.Remove(routing_id);
 
 #if defined(OS_WIN)
   // Dump the handle table if handle auditing is enabled.
   const base::CommandLine& browser_command_line =
@@ skipping 1316 matching lines @@
   if (worker_ref_count_ == 0)
     Cleanup();
 }
 
 void RenderProcessHostImpl::EnsureMojoActivated() {
   mojo_activation_required_ = true;
   MaybeActivateMojo();
 }
 
 }  // namespace content