Site Isolation: test srcdoc navigation after a cross-site navigation.

content/browser/site_per_process_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/site_per_process_browsertest.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 8640 matching lines @@
   // Start a URLRequest to the same URL. This should succeed. This would have
   // hit the 20 seconds delay before https://crbug.com/657195 was fixed.
   EXPECT_TRUE(NavigateToURL(shell(), page_url));
   EXPECT_EQ(page_url, shell()->web_contents()->GetLastCommittedURL());
 
   // Note: even if the test fails and for some reason, the test has not timed
   // out by this point, the test teardown code will still hit a DCHECK when it
   // calls AssertNoURLRequests() in the shell's URLRequestContext destructor.
 }
 
+// Ensure that an iframe that navigates cross-site doesn't use the same process
+// as its parent. Then when it navigates to its "srcdoc" attribute, it must

[Charlie Reis, 2016/11/23 08:02:25]

nit: Then when its parent navigates it via the "srcdoc" attribute, it must

[arthursonzogni, 2016/11/23 09:00:48]

Done.

+// reuse its parent's process.
+IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
+                       IframeSrcdocAfterCrossSiteNavigation) {
+  GURL parent_url(embedded_test_server()->GetURL(
+      "a.com", "/cross_site_iframe_factory.html?a(b)"));
+  GURL child_url(embedded_test_server()->GetURL(
+      "b.com", "/cross_site_iframe_factory.html?b()"));
+  GURL blank_url(url::kAboutBlankURL);

[Charlie Reis, 2016/11/23 08:02:25]

Sanity check: This will change to about:srcdoc after your other CL lands, right?

[arthursonzogni, 2016/11/23 09:00:48]

Yes that's right. I confirm.

+
+  // #1 Navigate to a page with a cross-site iframe.
+  EXPECT_TRUE(NavigateToURL(shell(), parent_url));
+
+  // Ensure that the iframe uses its own process.
+  FrameTreeNode* root = web_contents()->GetFrameTree()->root();
+  ASSERT_EQ(1u, root->child_count());
+  FrameTreeNode* child = root->child_at(0);
+  EXPECT_EQ(parent_url, root->current_url());
+  EXPECT_EQ(child_url, child->current_url());
+  EXPECT_NE(root->current_frame_host()->GetSiteInstance(),
+            child->current_frame_host()->GetSiteInstance());
+  EXPECT_NE(root->current_frame_host()->GetProcess(),
+            child->current_frame_host()->GetProcess());
+
+  // #2 Navigate the iframe to its srcdoc attribute.
+  TestNavigationObserver load_observer(shell()->web_contents());
+  EXPECT_TRUE(ExecuteScript(
+      root,
+      R"(document.getElementById("child-0").srcdoc = "srcdoc content";)"));

[Charlie Reis, 2016/11/23 08:02:25]

Style nit: We don't use R (raw string literals) much here.  I think it might be
more consistent and easier to read as a normal string with single quotes inside.

[arthursonzogni, 2016/11/23 09:00:48]

Done.

+  load_observer.Wait();
+
+  // Ensure that the iframe reuses its parent's process.
+  EXPECT_EQ(blank_url, child->current_url());
+  EXPECT_EQ(root->current_frame_host()->GetSiteInstance(),
+            child->current_frame_host()->GetSiteInstance());
+  EXPECT_EQ(root->current_frame_host()->GetProcess(),
+            child->current_frame_host()->GetProcess());
+}
+
 }  // namespace content