OOPIF: Data URLs are now rendered in the renderer that initiated the navigation.

content/browser/frame_host/render_frame_host_manager_browsertest.cc

 // Copyright 2013 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 <set>
 
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 139 matching lines @@
       new_shell->web_contents()->GetSiteInstance());
   EXPECT_NE(orig_site_instance, new_site_instance);
 
   // We should no longer have script access to the opened window's location.
   success = false;
   EXPECT_TRUE(ExecuteScriptAndExtractBool(
       shell()->web_contents(),
       "window.domAutomationController.send(testScriptAccessToWindow());",
       &success));
   EXPECT_FALSE(success);
+
+  // We now navigate to a blank window.

[Charlie Reis, 2014/12/11 00:43:39]

nit: We now navigate the window to an about:blank page.

("Blank" is ambiguous because the other tests are referring to target=_blank,
not about:blank.)

[lfg, 2014/12/11 23:55:24]

Done.

+  success = false;
+  EXPECT_TRUE(ExecuteScriptAndExtractBool(
+      shell()->web_contents(),
+      "window.domAutomationController.send(clickBlankTargetedLink());",
+      &success));
+  EXPECT_TRUE(success);
+
+  // Wait for the navigation in the new window to finish.
+  WaitForLoadStop(new_shell->web_contents());
+  EXPECT_EQ("blank",

[Charlie Reis, 2014/12/11 00:43:39]

Let's just compare GetLastCommittedURL() to GURL(kAboutBlankURL), rather than
comparing paths.

[lfg, 2014/12/11 23:55:24]

Done.

+            new_shell->web_contents()->GetLastCommittedURL().path());
+  scoped_refptr<SiteInstance> blank_site_instance2(

[Charlie Reis, 2014/12/11 00:43:39]

This can then be blank_site_instance.

[lfg, 2014/12/11 23:55:24]

Removed, as we don't need to save the reference in this case.

+      new_shell->web_contents()->GetSiteInstance());
+  EXPECT_EQ(orig_site_instance, blank_site_instance2);
+
+  // We should have access to the opened window's location.
+  success = false;
+  EXPECT_TRUE(ExecuteScriptAndExtractBool(
+      shell()->web_contents(),
+      "window.domAutomationController.send(testScriptAccessToWindow());",
+      &success));
+  EXPECT_TRUE(success);
 }
 
 // Test for crbug.com/24447.  Following a cross-site link with rel=noreferrer
 // and target=_blank should create a new SiteInstance.
 IN_PROC_BROWSER_TEST_F(RenderFrameHostManagerTest,
                        SwapProcessWithRelNoreferrerAndTargetBlank) {
   StartServer();
 
   // Load a page with links that open in a new window.
   std::string replacement_path;
@@ skipping 1420 matching lines @@
   EXPECT_TRUE(ChildProcessSecurityPolicyImpl::GetInstance()->CanReadFile(
       shell()->web_contents()->GetRenderProcessHost()->GetID(), file_path));
 
   // Navigate to a same site page to trigger a PageState update and ensure the
   // renderer is not killed.
   EXPECT_TRUE(
       NavigateToURL(shell(), test_server()->GetURL("files/title2.html")));
 }
 
 }  // namespace content