Fix Self-Referencing OOPIF Infinite Loop

content/browser/site_per_process_browsertest.cc

Index: content/browser/site_per_process_browsertest.cc
diff --git a/content/browser/site_per_process_browsertest.cc b/content/browser/site_per_process_browsertest.cc
index 71b1dcbbd9740ba1a18474acc817beba9c2d4e32..4d67b814e32e274232d9e0a5c61b6db75f4a831d 100644
--- a/content/browser/site_per_process_browsertest.cc
+++ b/content/browser/site_per_process_browsertest.cc
@@ -8979,4 +8979,34 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
   EXPECT_TRUE(rvh->is_active());
 }
 
+// Ensure that loading a page with a cross-site coreferencing iframe
+// does not cause an infinite number of nested iframes to be created.
+// TODO(davidsac): This test should be broken until issue 650332 is resolved.
+// Also, a decision of how to handle the issue will determine what FrameTree
+// string to expect
+IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, CoReferencingFrames) {

[alexmos, 2016/12/28 00:25:59]

This test looks identical to the one above in RenderFrameHostManagerTest, which
has coverage both without and with --site-per-process (the latter on our FYI
bots and try bot).  Can it be removed, or was there a reason for keeping it?

[davidsac (gone - try alexmos), 2017/01/06 00:44:58]

It has been removed. :)  I think when I first wrote both tests, I didn't realize
that RenderFrameHostManagerTest was run in both modes.

+  // Load a page with a cross-site coreferencing iframe.
+  GURL url_1(
+      embedded_test_server()->GetURL("a.com", "/coreferencingframe_1.html"));
+  EXPECT_TRUE(NavigateToURL(shell(), url_1));
+
+  FrameTreeNode* root = web_contents()->GetFrameTree()->root();
+
+  // The Frame Tree contains two successful instances of each site plus an
+  // unsuccessfully-navigated third instance of B with a blank URL.
+  EXPECT_EQ(
+      " Site A ------------ proxies for B\n"
+      "   +--Site B ------- proxies for A\n"
+      "        +--Site A -- proxies for B\n"
+      "             +--Site B -- proxies for A\n"
+      "                  +--Site B -- proxies for A\n"
+      "Where A = http://a.com/\n"
+      "      B = http://b.com/",
+      DepictFrameTree(root));
+
+  EXPECT_EQ(
+      root->child_at(0)->child_at(0)->child_at(0)->child_at(0)->current_url(),
+      "");
+}
+
 }  // namespace content