Swap the main frame in --site-per-process mode too.

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 <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 2294 matching lines @@
       " Site A\n"
       "   |--Site A\n"
       "   +--Site A\n"
       "        |--Site A\n"
       "        +--Site A\n"
       "             +--Site A\n"
       "Where A = http://127.0.0.1/",
       DepictFrameTree(root));
 }
 
+IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
+                       OpenCrossSiteWindowWithFrames) {
+  ASSERT_EQ(1u, Shell::windows().size());
+  Shell* main_window = Shell::windows()[0];
+
+  // Navigate the main window.
+  GURL main_url(embedded_test_server()->GetURL("/site_per_process_main.html"));
+  NavigateToURL(main_window, main_url);
+  EXPECT_EQ(main_window->web_contents()->GetLastCommittedURL(), main_url);
+
+  // Load a cross-site page into a new window.
+  GURL cross_url =
+      embedded_test_server()->GetURL("foo.com", "/site_per_process_main.html");
+  std::string script = "window.open('" + cross_url.spec() + "')";
+  EXPECT_TRUE(ExecuteScript(main_window->web_contents(), script));
+  ASSERT_EQ(2u, Shell::windows().size());
+  Shell* cross_window = Shell::windows()[1];
+  WaitForLoadStop(cross_window->web_contents());
+  EXPECT_EQ(cross_window->web_contents()->GetLastCommittedURL(), cross_url);
+
+  // Make sure the main window is still live. To create a synchronization point
+  // and ensure that the renderer has processed all the messages for updating
+  // the frame tree, execute a simple bit of JS first.

[dcheng, 2015/05/05 21:58:08]

It seems like there should be a better way to do this... but meh.

+  EXPECT_TRUE(ExecuteScript(main_window->web_contents(), "true"));
+  FrameTreeNode* root =
+      static_cast<WebContentsImpl*>(main_window->web_contents())
+          ->GetFrameTree()
+          ->root();
+  EXPECT_TRUE(root->current_frame_host()->IsRenderFrameLive());

[dcheng, 2015/05/05 21:58:08]

Should I do this for the cross-site window too?

[nasko, 2015/05/06 21:55:24]

I don't think it will hurt. Also, why not use WebConetntsImpl::GetMainFrame()?
It is equivalent to doing frame_tree.root()->current_frame_host().

[dcheng, 2015/05/06 22:52:27]

Done.

No real reason other than I didn't know about it =)

+}
+
 }  // namespace content