PlzNavigate: properly set the initiator of the navigation

content/test/test_render_frame_host.cc

Index: content/test/test_render_frame_host.cc
diff --git a/content/test/test_render_frame_host.cc b/content/test/test_render_frame_host.cc
index 115d6069a172c12b9e581da70aacecb2d527281a..42501f59845105d78fe3f579b5341e28336a69e8 100644
--- a/content/test/test_render_frame_host.cc
+++ b/content/test/test_render_frame_host.cc
@@ -363,9 +363,9 @@ void TestRenderFrameHost::SendRendererInitiatedNavigationRequest(
   InitializeRenderFrameIfNeeded();
 
   if (IsBrowserSideNavigationEnabled()) {
-    BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL,
-                                       has_user_gesture, false,
-                                       REQUEST_CONTEXT_TYPE_HYPERLINK);
+    BeginNavigationParams begin_params(
+        std::string(), net::LOAD_NORMAL, has_user_gesture, false,
+        REQUEST_CONTEXT_TYPE_HYPERLINK, url::Origin(url));

[Mike West, 2016/06/27 12:10:08]

Can you change this to a unique origin (`url::Origin()`), and add a comment to
the effect of `// TODO(mkwst): The initiator origin here is incorrect.`? I don't
think any test which relies on this code actually cares about the initiator, but
if any are added, we'll need to be a little more clever with our implementation
here.

[clamy, 2016/06/27 13:36:15]

Done.

     CommonNavigationParams common_params;
     common_params.url = url;
     common_params.referrer = Referrer(GURL(), blink::WebReferrerPolicyDefault);