[RDHI] Refactored blocked_loaders_map_ to key by render frame route id

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 a7d4c3f584fa8a7696838af822ad2a4aac6f96b0..f221bd22d2f4e4269deb4be99c2f8df9ed95e12b 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -1917,8 +1917,8 @@ base::TimeDelta RenderProcessHostImpl::GetChildProcessIdleTime() const {
   return base::TimeTicks::Now() - child_process_activity_time_;
 }
 
-void RenderProcessHostImpl::ResumeRequestsForView(int route_id) {
-  widget_helper_->ResumeRequestsForView(route_id);
+void RenderProcessHostImpl::ResumeRequestsForFrame(int route_id) {

[Randy Smith (Not in Mondays), 2015/12/29 23:11:37]

I hope you were really careful about making sure this transformation propagated
up and down the callstack from here, since the compiler wouldn't have said a
word :-}.

[Charlie Harrison, 2015/12/30 20:51:49]

The approach I did here (and other places) was to change "ForRoute" or "ForView"
to "ForFrame", forcing all callers to change. I hoped that would catch all
cases, but your comment that the "compiler wouldn't have said a word" makes me
worried.

[Randy Smith (Not in Mondays), 2016/01/04 21:24:48]

No, actually, that sounds like a good approach.  Sorry to make you worried;
thanks for sketching out your method.

+  widget_helper_->ResumeRequestsForFrame(route_id);
 }
 
 void RenderProcessHostImpl::FilterURL(bool empty_allowed, GURL* url) {