Content API changes to improve DOM stitching in ThreatDetails code.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index c6c1e34d157bc3ba9c5771e9fbe6d5a2a0faa12f..3d5253f69d2f1e66fca68ab643c3315283e7773c 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -1062,6 +1062,11 @@ void RenderFrame::ForEach(RenderFrameVisitor* visitor) {
   }
 }
 
+// static
+int RenderFrame::GetRoutingIdForWebFrame(blink::WebFrame* web_frame) {
+  return GetRoutingIdForFrameOrProxy(web_frame);

[Charlie Reis, 2017/05/15 20:23:14]

Did you decide not to go with Dominic's suggestion to replace
GetRoutingIdForFrameOrProxy with RenderFrame::GetRoutingIdForWebFrame, or did
that just fall between the cracks?  Seems like a reasonable change to me.

[lpz, 2017/05/16 15:19:57]

Ahh, I missed that comment. Thanks, done.

Btw, the only other function in web_frame_utils
(GetWebFrameFromRoutingIdForFrameOrProxy) seems to be unused, so that whole file
can probably be deleted. But I'd prefer to leave that for another CL.

[Charlie Reis, 2017/05/16 16:34:08]

Ah, nice!  Yes, we can remove that after this CL lands.

+}
+
 // static
 RenderFrameImpl* RenderFrameImpl::FromWebFrame(blink::WebFrame* web_frame) {
   FrameMap::iterator iter = g_frame_map.Get().find(web_frame);