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

content/browser/renderer_host/render_process_host_impl.cc

Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index 717877410e36859f4c56328a7b64c1d67c30e94b..a6ad6a94de48b5ba49edb54f36792d08b5f29895 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -921,7 +921,7 @@ ServiceRegistry* RenderProcessHostImpl::GetServiceRegistry() {
 void RenderProcessHostImpl::AddRoute(
     int32 routing_id,
     IPC::Listener* listener) {
-  DCHECK(widget_helper_->IsRoutingIDProbablyValid(routing_id))
+  CHECK(widget_helper_->IsRoutingIDProbablyValid(routing_id))
       << "Found Routing ID conflicts: " << routing_id;

[nasko, 2014/10/14 18:49:33]

Should we add the previous CHECK we had too? I know the probability of this
failing without actually adding a duplicate is very low, but with the number of
users running the code you never know.

Also, let's log some more data (the RPH id, OS process id), so we have more to
look at in the crash reports.

[Hajime Morrita, 2014/10/14 19:36:55]

I worried to get as many crashes as before but true, we need it.
Added.
Add PID. Do we have log meessage in our crash report?
I wasn't aware of that. Could you point me where I can see it?

If we can see the dying message, we can do more. Not in this CL
but I'm thinking about passing base::Location from the caller
to AddRoute(). 

[nasko, 2014/10/14 20:54:16]

That's fine, at least we will know the relative amount of one type of crash vs
the other type.
 
Unfortunately we don't. We can use debug::Alias to ensure values we want to know
are on the stack and we can examine them.
It will be nice to add base::Location, but AddRoute is a public interface method
and changing it doesn't seem right for a temp change to collect debug stats.

   listeners_.AddWithID(listener, routing_id);
 }