Add a ResourceScheduler to ResourceDispatcherHost.

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 a9d26ff50c011def86cdde1d22d276b9f61781cc..d480ed644ef80b57d2c21866d5408bc4b4d7cb80 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -56,7 +56,10 @@
 #include "content/browser/hyphenator/hyphenator_message_filter.h"
 #include "content/browser/in_process_webkit/indexed_db_context_impl.h"
 #include "content/browser/in_process_webkit/indexed_db_dispatcher_host.h"
+#include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/loader/resource_message_filter.h"
+#include "content/browser/loader/resource_scheduler.h"
+#include "content/browser/loader/resource_scheduler_filter.h"
 #include "content/browser/mime_registry_message_filter.h"
 #include "content/browser/plugin_service_impl.h"
 #include "content/browser/profiler_message_filter.h"
@@ -479,6 +482,7 @@ bool RenderProcessHostImpl::Init() {
 
 void RenderProcessHostImpl::CreateMessageFilters() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  channel_->AddFilter(new ResourceSchedulerFilter(GetID()));
   MediaObserver* media_observer =
       GetContentClient()->browser()->GetMediaObserver();
   // Add BrowserPluginMessageFilter to ensure it gets the first stab at messages
@@ -1455,6 +1459,10 @@ void RenderProcessHostImpl::ProcessDied(bool already_dead) {
         ViewHostMsg_RenderViewGone(iter.GetCurrentKey(),
                                    static_cast<int>(status),
                                    exit_code));
+    if (ResourceDispatcherHostImpl::Get()) {
+      ResourceDispatcherHostImpl::Get()->scheduler()->OnDestroy(

[darin (slow to review), 2012/12/15 05:17:24]

This looks like it is the main thread, and I don't think
ResourceScheduler::OnDestroy is safe to call off of the IO thread.  You should
probably make ResourceScheduler inherit from NonThreadSafe.  It might be nice to
find some other way to communicate the destruction of a RenderView to the
scheduler.

Also, this is only trapping process destruction.  RenderViews can be closed well
before process destruction.  OnCreate seems to be called when a RenderView is
created, so I assumed OnDestroy was intended to be called when a RenderView is
destroyed.  Seems not to be how it works though.

[James Simonsen, 2012/12/17 21:31:38]

Done. Let me know if I should separate out or rename
RenderWidgetHelper::OnCancelResourceRequests.

+          GetID(), iter.GetCurrentKey());
+    }
     iter.Advance();
   }