Add frame_tree_node_id to navigation parameters.

content/public/browser/page_navigator.h

Index: content/public/browser/page_navigator.h
diff --git a/content/public/browser/page_navigator.h b/content/public/browser/page_navigator.h
index b097936fcf8e6db7d1b5b97ff583b99cc142a892..d9a88c50080eb921aa600e22b72b5c4399a02ef3 100644
--- a/content/public/browser/page_navigator.h
+++ b/content/public/browser/page_navigator.h
@@ -33,6 +33,7 @@ struct CONTENT_EXPORT OpenURLParams {
   OpenURLParams(const GURL& url,
                 const Referrer& referrer,
                 int64 source_frame_id,

[nasko, 2013/11/20 18:01:30]

nit: TODO to remove this?

[Charlie Reis, 2013/11/20 21:43:06]

Wow.  I expected source_frame_id to be needed for something like the Web
Navigation API (etc), but it appears to be dead code.  We pass it in from
WebContents but forget it in browser_navigator.cc.

Before removing it, I want to do some digging to see how much can be removed and
why it was there in the first place, so I've added a TODO.

+                int64 frame_tree_node_id,
                 WindowOpenDisposition disposition,
                 PageTransition transition,
                 bool is_renderer_initiated);
@@ -59,9 +60,12 @@ struct CONTENT_EXPORT OpenURLParams {
   // is terminated by \r\n.  May be empty if no extra headers are needed.
   std::string extra_headers;
 
-  // The source frame id or -1 to indicate the main frame.
+  // The renderer-specific source frame id or -1 to indicate the main frame.
   int64 source_frame_id;
 
+  // The browser-global FrameTreeNode ID or -1 to indicate the main frame.
+  int64 frame_tree_node_id;
+
   // The disposition requested by the navigation source.
   WindowOpenDisposition disposition;