PlzNavigate: Add a static CommitNavigation method in FrameTreeNode

content/browser/frame_host/frame_tree_node.cc

Index: content/browser/frame_host/frame_tree_node.cc
diff --git a/content/browser/frame_host/frame_tree_node.cc b/content/browser/frame_host/frame_tree_node.cc
index ebc542e56e31bb2ac71100892bb80ec38d72d15f..19e4d5d52dc1ebc5b8176c6aae1414d3570b8f90 100644
--- a/content/browser/frame_host/frame_tree_node.cc
+++ b/content/browser/frame_host/frame_tree_node.cc
@@ -6,16 +6,44 @@
 
 #include <queue>
 
+#include "base/containers/hash_tables.h"
+#include "base/lazy_instance.h"
 #include "base/stl_util.h"
 #include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/navigator.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
+#include "content/public/browser/browser_thread.h"
 
 namespace content {
 
+namespace {
+typedef base::hash_map<int64, FrameTreeNode*> FrameTreeNodeIDMap;
+
+base::LazyInstance<FrameTreeNodeIDMap> g_frame_tree_node_id_map =
+    LAZY_INSTANCE_INITIALIZER;
+
+FrameTreeNode* FrameTreeNodeFromId(int64 frame_tree_node_id) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  FrameTreeNodeIDMap* nodes = g_frame_tree_node_id_map.Pointer();
+  FrameTreeNodeIDMap::iterator it = nodes->find(frame_tree_node_id);
+  return it == nodes->end() ? NULL : it->second;
+}
+
+}  // namespace
+
 int64 FrameTreeNode::next_frame_tree_node_id_ = 1;
 
+// static
+void FrameTreeNode::CommitNavigationInNode(int64 frame_tree_node_id,

[nasko, 2014/07/30 08:36:16]

nit: the "InNode" part of the method is redundant, since this is already invoked
on a FrameTreeNode. It would make sense if it was a method on FrameTree.

This brings up another question. Does this belong to FrameTree or the FTN? I
don't have a strong preference, but it feels a bit awkward to call a method on
FTN without knowing which FTN it is about.

[clamy, 2014/08/12 13:41:47]

Done.

+                                           const NavigationCommitInfo& info) {
+  FrameTreeNode* node = FrameTreeNodeFromId(frame_tree_node_id);
+  if (!node)
+    return;
+  DCHECK(node->frame_tree_node_id() == frame_tree_node_id);
+  node->render_manager()->CommitNavigation(info);
+}
+
 FrameTreeNode::FrameTreeNode(FrameTree* frame_tree,
                              Navigator* navigator,
                              RenderFrameHostDelegate* render_frame_delegate,
@@ -32,9 +60,13 @@ FrameTreeNode::FrameTreeNode(FrameTree* frame_tree,
                       manager_delegate),
       frame_tree_node_id_(next_frame_tree_node_id_++),
       frame_name_(name),
-      parent_(NULL) {}
+      parent_(NULL) {
+  g_frame_tree_node_id_map.Get().insert(
+      std::make_pair(frame_tree_node_id_, this));
+}
 
 FrameTreeNode::~FrameTreeNode() {
+  g_frame_tree_node_id_map.Get().erase(frame_tree_node_id_);
 }
 
 bool FrameTreeNode::IsMainFrame() const {