Add frameId to chrome.tabs.executeScript/insertCSS

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 <utility>
 
 #include "base/bind.h"
 #include "base/containers/hash_tables.h"
@@ skipping 300 matching lines @@
   return process_;
 }
 
 RenderFrameHost* RenderFrameHostImpl::GetParent() {
   FrameTreeNode* parent_node = frame_tree_node_->parent();
   if (!parent_node)
     return NULL;
   return parent_node->current_frame_host();
 }
 
+void RenderFrameHostImpl::ForEachChildFrame(
+    const base::Callback<void(RenderFrameHost*)>& on_frame) {
+  if (this != frame_tree_node_->current_frame_host()) {

[jam, 2016/01/26 23:42:34]

why is this needed?

[robwu, 2016/01/26 23:57:59]

Just in case someone calls ForEachChildFrame on a RenderFrameHost that was
swapped. This shouldn't happen, but code will still compile, so it could happen,
and I don't want to report child frames which are not childs of the given RFH.

+    NOTREACHED();
+    return;
+  }
+  for (size_t i = 0; i < frame_tree_node_->child_count(); ++i) {
+    FrameTreeNode* child = frame_tree_node_->child_at(i);
+    on_frame.Run(child->current_frame_host());
+  }
+}
+
 int RenderFrameHostImpl::GetFrameTreeNodeId() {
   return frame_tree_node_->frame_tree_node_id();
 }
 
 const std::string& RenderFrameHostImpl::GetFrameName() {
   return frame_tree_node_->frame_name();
 }
 
 bool RenderFrameHostImpl::IsCrossProcessSubframe() {
   FrameTreeNode* parent_node = frame_tree_node_->parent();
@@ skipping 2221 matching lines @@
   *dst = src;
 
   if (src.routing_id != -1)
     dst->tree_id = RoutingIDToAXTreeID(src.routing_id);
 
   if (src.parent_routing_id != -1)
     dst->parent_tree_id = RoutingIDToAXTreeID(src.parent_routing_id);
 }
 
 }  // namespace content