Added the RenderView routing_id to the DidCommitProvisionalLoad message.

content/browser/loader/resource_scheduler_filter.cc

Index: content/browser/loader/resource_scheduler_filter.cc
diff --git a/content/browser/loader/resource_scheduler_filter.cc b/content/browser/loader/resource_scheduler_filter.cc
index 4c825dab987fa728e2f23f21366bc39e6dead45b..fb9c46e410d2de648d87cbc236e3e1e7fb991f84 100644
--- a/content/browser/loader/resource_scheduler_filter.cc
+++ b/content/browser/loader/resource_scheduler_filter.cc
@@ -45,7 +45,13 @@ bool ResourceSchedulerFilter::OnMessageReceived(const IPC::Message& message) {
       }
       if (PageTransitionIsMainFrame(params.transition) &&
           !params.was_within_same_page) {
-        scheduler->OnNavigate(child_id_, message.routing_id());
+        // We need to track the RenderViewHost routing_id because of downstream
+        // dependencies on being able to look up the view based on the ID stored
+        // in the resource requests (without it a lot of things break).  Once
+        // those dependencies are unwound or moved to RenderViewHost we can move

[nasko, 2014/07/21 13:11:23]

Same as other comment.

+        // the client to be based on the routing_id of the RenderViewHost
+        // instead (see crbug.com/392171).
+        scheduler->OnNavigate(child_id_, params.render_view_routing_id);
       }
       break;
     }