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..da710ac28e85589d7b49ea4664f263f399efed5d 100644
--- a/content/public/browser/page_navigator.h
+++ b/content/public/browser/page_navigator.h
@@ -71,6 +71,12 @@ 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 render_frame_id;

[nasko, 2017/02/15 00:58:07]

I'm sorry I missed this on the previous iterations, but we should be
initializing these to MSG_ROUTING_NONE when constructing the struct. Otherwise
we can get uninitialized memory with garbage content in it and lead to incorrect
values. 

[Patrick Noland, 2017/02/15 22:30:42]

Done.

+
+  // Process id of the source RenderFrameHost.
+  int render_process_id;
+
   // The disposition requested by the navigation source.
   WindowOpenDisposition disposition;