Pass origin information for remote frame creation.

content/browser/frame_host/frame_tree_browsertest.cc

Index: content/browser/frame_host/frame_tree_browsertest.cc
diff --git a/content/browser/frame_host/frame_tree_browsertest.cc b/content/browser/frame_host/frame_tree_browsertest.cc
index 9f16066cd38752be333d5af39867ef7d4618d346..a9b2f5558ac458d2ea1f4c1d28e63d282e336844 100644
--- a/content/browser/frame_host/frame_tree_browsertest.cc
+++ b/content/browser/frame_host/frame_tree_browsertest.cc
@@ -199,6 +199,42 @@ IN_PROC_BROWSER_TEST_F(FrameTreeBrowserTest, IsRenderFrameLive) {
   EXPECT_TRUE(root->child_at(0)->current_frame_host()->IsRenderFrameLive());
 }
 
+// Ensure that origins are correctly set on navigations.
+IN_PROC_BROWSER_TEST_F(FrameTreeBrowserTest, OriginSetOnNavigation) {

[Charlie Reis, 2014/11/13 18:00:57]

Perhaps we can also add a --site-per-process test on the renderer side (e.g.,
render_view_browsertest.cc), which could verify that the RenderFrameProxy ends
up with the right state?

[alexmos, 2014/11/18 18:25:31]

Done -- please see RenderViewImplTest.OriginReplicationForSwapOut.  I could also
add something similar for the other two remote frame creation paths
(ViewMsg_New, NewFrameProxy), if I can figure out how to get those to work as
render view tests.

[Charlie Reis, 2014/11/19 00:46:17]

Thanks!

+  host_resolver()->AddRule("*", "127.0.0.1");
+  ASSERT_TRUE(test_server()->Start());
+  GURL main_url(test_server()->GetURL("files/site_per_process_main.html"));

[Charlie Reis, 2014/11/13 18:00:57]

nit: This isn't a site per process test, so it's a little unusual to use
site_per_process_main.html.  Would something like files/frame_tree/top.html work
here?

[alexmos, 2014/11/18 18:25:31]

Done.  It doesn't matter to the test as long as there's at least one frame in
the file.

+  NavigateToURL(shell(), main_url);
+
+  // It is safe to obtain the root frame tree node here, as it doesn't change.
+  FrameTreeNode* root = static_cast<WebContentsImpl*>(shell()->web_contents())
+                            ->GetFrameTree()->root();
+
+  EXPECT_EQ(root->current_replication_state().origin.string(),
+            main_url.GetOrigin().spec());
+  EXPECT_FALSE(root->current_replication_state().is_unique_origin);
+
+  GURL frame_url(test_server()->GetURL("files/title1.html"));
+  NavigateFrameToURL(root->child_at(0), frame_url);
+
+  EXPECT_EQ(root->child_at(0)->current_replication_state().origin.string(),
+            frame_url.GetOrigin().spec());
+  EXPECT_FALSE(root->child_at(0)->current_replication_state().is_unique_origin);
+
+  // Navigating to a data URL should set a unique origin.
+  GURL data_url("data:text/html,foo");
+  NavigateToURL(shell(), data_url);
+
+  EXPECT_TRUE(root->current_replication_state().is_unique_origin);
+
+  // Navigating to a normal URL should update origin and clear unique origin
+  // flag.
+  NavigateToURL(shell(), main_url);
+  EXPECT_EQ(root->current_replication_state().origin.string(),
+            main_url.GetOrigin().spec());
+  EXPECT_FALSE(root->current_replication_state().is_unique_origin);
+}
+
 class CrossProcessFrameTreeBrowserTest : public ContentBrowserTest {
  public:
   CrossProcessFrameTreeBrowserTest() {}