Resetting FrameTreeNode must remove child FrameTreeNodes from global map.

content/browser/frame_host/frame_tree.h

Index: content/browser/frame_host/frame_tree.h
diff --git a/content/browser/frame_host/frame_tree.h b/content/browser/frame_host/frame_tree.h
index 5f0c7aad333650f8fe2549f45346bbc710d751f1..5ef3e84dd285061c7617278de367408ec6a94995 100644
--- a/content/browser/frame_host/frame_tree.h
+++ b/content/browser/frame_host/frame_tree.h
@@ -76,6 +76,10 @@ class CONTENT_EXPORT FrameTree {
                                 const std::string& frame_name);
   void RemoveFrame(FrameTreeNode* child);
 
+  // This method only removes the FrameTreeNode from the global map of ids

[Charlie Reis, 2015/02/12 17:12:43]

It feels a bit odd to have a public method for modifying a private global map. 
It would be nice to hide this behind RemoveFrame.

[nasko, 2015/02/12 23:21:08]

Done.

+  // to objects.
+  void RemoveNodeFromMap(int64 frame_tree_node_id);
+
   // This method walks the entire frame tree and creates a RenderFrameProxyHost
   // for the given |site_instance| in each node except the |source| one --
   // the source will have a RenderFrameHost. It assumes that no frame tree