Content API changes to improve DOM stitching in ThreatDetails code.

content/browser/frame_host/render_frame_host_impl.cc

 // 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.
 
 #include "content/browser/frame_host/render_frame_host_impl.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 3363 matching lines @@
          frame_tree_node_->current_url().SchemeIs(kChromeDevToolsScheme) ||
          ChildProcessSecurityPolicyImpl::GetInstance()->HasWebUIBindings(
              GetProcess()->GetID()) ||
          // It's possible to load about:blank in a Web UI renderer.
          // See http://crbug.com/42547
          (frame_tree_node_->current_url().spec() == url::kAboutBlankURL) ||
          // InterstitialPageImpl should be the only case matching this.
          (delegate_->GetAsWebContents() == nullptr);
 }
 
+// static
+int RenderFrameHost::LookupOtherFrameTreeNodeId(int process_id,
+                                                int routing_id) {
+  // Note: this code is copied/modified from RoutingIDToAXTreeID (below).

[dmazzoni, 2017/05/01 20:10:07]

Let's try to share this and come up with a cleaner name.

[lpz, 2017/05/02 19:36:24]

Done.

+  RenderFrameProxyHost* rfph =
+      RenderFrameProxyHost::FromID(process_id, routing_id);
+  if (rfph) {
+    return rfph->frame_tree_node()->frame_tree_node_id();
+  } else {
+    RenderFrameHost* rfh = RenderFrameHost::FromID(process_id, routing_id);
+    if (rfh)
+      return rfh->GetFrameTreeNodeId();
+  }
+  return -1;

[dmazzoni, 2017/05/01 20:10:07]

Is there a constant somewhere like kNoFrameTreeNodeId?

[lpz, 2017/05/02 19:36:24]

Couldn't find one so added it.

+}
+
 ui::AXTreeIDRegistry::AXTreeID RenderFrameHostImpl::RoutingIDToAXTreeID(
     int routing_id) {
   RenderFrameHostImpl* rfh = nullptr;
   RenderFrameProxyHost* rfph = RenderFrameProxyHost::FromID(
       GetProcess()->GetID(), routing_id);
   if (rfph) {
     FrameTree* frame_tree = rfph->frame_tree_node()->frame_tree();
     FrameTreeNode* frame_tree_node = frame_tree->FindByRoutingID(
         GetProcess()->GetID(), routing_id);
     rfh = frame_tree_node->render_manager()->current_frame_host();
@@ skipping 287 matching lines @@
 }
 
 void RenderFrameHostImpl::ForwardGetInterfaceToRenderFrame(
     const std::string& interface_name,
     mojo::ScopedMessagePipeHandle pipe) {
   GetRemoteInterfaces()->GetInterface(interface_name, std::move(pipe));
 }
 #endif
 
 }  // namespace content