Switch RenderViewContextMenu to use RequestOpenURL

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 945c96ec763622ccb5d52ba036cd723e8867deec..e2b3f784b556d17253b6d2fded4caacfbf201347 100644
--- a/content/public/browser/page_navigator.h
+++ b/content/public/browser/page_navigator.h
@@ -14,6 +14,7 @@
 #include "base/memory/ref_counted.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/global_request_id.h"
+#include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/common/referrer.h"
 #include "content/public/common/resource_request_body.h"
@@ -71,6 +72,11 @@ struct CONTENT_EXPORT OpenURLParams {
   // The browser-global FrameTreeNode ID or -1 to indicate the main frame.
   int frame_tree_node_id;
 
+  // Routing id of the source RenderFrameHost.
+  int frame_id;

[nasko, 2017/02/14 00:10:23]

nit: render_frame_id

[Patrick Noland, 2017/02/14 01:16:16]

Done.

+  // Process id of the source RenderFrameHost.

[nasko, 2017/02/14 00:10:23]

nit: Leave an extra empty line between to match how the rest of the file is
formatted.

[Patrick Noland, 2017/02/14 01:16:16]

Done.

+  int process_id;

[nasko, 2017/02/14 00:10:23]

nit: render_process_id

[Patrick Noland, 2017/02/14 01:16:16]

Done.

+
   // The disposition requested by the navigation source.
   WindowOpenDisposition disposition;