Content API changes to improve DOM stitching in ThreatDetails code.

content/public/renderer/render_frame.h

 // Copyright 2013 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_PUBLIC_RENDERER_RENDER_FRAME_H_
 #define CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 84 matching lines @@
 
   // Returns the RenderFrame given a WebFrame.
   static RenderFrame* FromWebFrame(blink::WebFrame* web_frame);
 
   // Returns the RenderFrame given a routing id.
   static RenderFrame* FromRoutingID(int routing_id);
 
   // Visit all live RenderFrames.
   static void ForEach(RenderFrameVisitor* visitor);
 
+  static int GetRoutingIdForWebFrame(blink::WebFrame* web_frame);

[Charlie Reis, 2017/05/05 21:03:08]

Please add a comment.  Note that it's easier to talk about remote frames in the
renderer process than the browser process.  Something like:

Returns the routing ID for |web_frame|, whether it is a WebLocalFrame in this
process or a WebRemoteFrame placeholder for a frame in a different process.

[lpz, 2017/05/10 14:21:10]

Done.

+
   // Returns the RenderView associated with this frame.
   virtual RenderView* GetRenderView() = 0;
 
   // Return the RenderAccessibility associated with this frame.
   virtual RenderAccessibility* GetRenderAccessibility() = 0;
 
   // Get the routing ID of the frame.
   virtual int GetRoutingID() = 0;
 
   // Returns the associated WebFrame.
@@ skipping 152 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameImpl;
   RenderFrame() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_