Introduce worker_devtools_agent_route_id for EmbeddedWorker.

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 12b37ceeb2edc0e3c3a03fab49f7704ba37b4492..e4dc2c1952694583aa8c0f39e3d56b13e3747867 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -762,7 +762,8 @@ void RenderProcessHostImpl::CreateMessageFilters() {
   scoped_refptr<ServiceWorkerDispatcherHost> service_worker_filter =
       new ServiceWorkerDispatcherHost(GetID(), message_port_message_filter_);
   service_worker_filter->Init(
-      storage_partition_impl_->GetServiceWorkerContext());
+      storage_partition_impl_->GetServiceWorkerContext(),
+      base::Bind(&RenderWidgetHelper::GetNextRoutingID, widget_helper_));

[michaeln, 2014/04/25 19:29:50]

ah... this is nice

[michaeln, 2014/04/25 20:10:34]

Here's a thought? Define a new static method and nix the abstraction layer by
way of bind closure and map book keeping. Less and more understandable code.
Wdyt?

// static
int RenderProcessHostImpl::GetNextRouteIDForProcess(process_id)  {
  RenderWidgetHelper* helper =
RenderWidgetHelper::FromProcessHostID(process_id);
  if (!helper)
    return -1;  // Maybe null in unittest.
  return helper->GetNextRoutingID();
}

   AddFilter(service_worker_filter);
 
   // If "--enable-embedded-shared-worker" is set, we use