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

content/browser/loader/global_routing_id.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_LOADER_GLOBAL_ROUTING_ID_H_
 #define CONTENT_BROWSER_LOADER_GLOBAL_ROUTING_ID_H_
 
 #include <tuple>
 
 namespace content {
@@ skipping 20 matching lines @@
   }
   bool operator==(const GlobalRoutingID& other) const {
     return child_id == other.child_id &&
         route_id == other.route_id;
   }
   bool operator!=(const GlobalRoutingID& other) const {
     return !(*this == other);
   }
 };
 
+// Same as GlobalRoutingID except the route_id must be a render frame routing
+// id.
+struct GlobalFrameRoutingId {
+  GlobalFrameRoutingId() : child_id(-1), route_id(-1) {

[nasko, 2016/01/16 00:15:58]

route_id should be MSG_ROUTING_NONE

[Charlie Harrison, 2016/01/21 18:52:55]

Done.

+  }
+
+  GlobalFrameRoutingId(int child_id, int route_id)
+      : child_id(child_id),
+        route_id(route_id) {
+  }
+
+  // The unique ID of the child process (different from OS's PID).
+  int child_id;
+
+  // The route ID (unique for each URLRequest source).
+  int route_id;

[nasko, 2016/01/16 00:15:58]

Let's be consistent since this is new code and call it "frame_routing_id", so it
is clear that it is the RenderFrameHost one.

[Charlie Harrison, 2016/01/21 18:52:55]

Done.

+
+  bool operator<(const GlobalFrameRoutingId& other) const {
+    return std::tie(child_id, route_id) <
+           std::tie(other.child_id, other.route_id);
+  }
+  bool operator==(const GlobalFrameRoutingId& other) const {
+    return child_id == other.child_id &&
+        route_id == other.route_id;
+  }
+  bool operator!=(const GlobalFrameRoutingId& other) const {
+    return !(*this == other);
+  }
+};
+
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_GLOBAL_ROUTING_ID_H_