Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(281)

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 1413543005: Use FrameTreeNode ID as frameId in extension APIs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits in #16, s/FindByFrameTreeNodeID/FindFrameByFrameTreeNodeID/ Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index a938e330d89984cf0a2911855a2209a2f6ea33ba..d4a5b17d11fad7854a7db6b98fd6e3040946fac9 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -764,6 +764,12 @@ RenderFrameHostImpl* WebContentsImpl::GetFocusedFrame() {
return focused_node->current_frame_host();
}
+RenderFrameHostImpl* WebContentsImpl::FindFrameByFrameTreeNodeID(
+ int frame_tree_node_id) {
+ FrameTreeNode* frame = frame_tree_.FindByID(frame_tree_node_id);
+ return frame ? frame->current_frame_host() : nullptr;
+}
+
void WebContentsImpl::ForEachFrame(
const base::Callback<void(RenderFrameHost*)>& on_frame) {
frame_tree_.ForEach(base::Bind(&ForEachFrameInternal, on_frame));

Powered by Google App Engine
This is Rietveld 408576698