Navigation transitions: Place transition page in same process as destination page.

content/browser/site_instance_impl_unittest.cc

Index: content/browser/site_instance_impl_unittest.cc
diff --git a/content/browser/site_instance_impl_unittest.cc b/content/browser/site_instance_impl_unittest.cc
index 646f4d079577593e5fdc0be8d9a10c8dc1696a19..aca7679073234446317e9624f6737cc103aade09 100644
--- a/content/browser/site_instance_impl_unittest.cc
+++ b/content/browser/site_instance_impl_unittest.cc
@@ -372,6 +372,7 @@ TEST_F(SiteInstanceTest, IsSameWebSite) {
   GURL url_foo_https = GURL("https://foo/a.html");
   GURL url_foo_port = GURL("http://foo:8080/a.html");
   GURL url_javascript = GURL("javascript:alert(1);");
+  GURL url_blank = GURL(url::kAboutBlankURL);
 
   // Same scheme and port -> same site.
   EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_foo, url_foo2));
@@ -388,6 +389,11 @@ TEST_F(SiteInstanceTest, IsSameWebSite) {
   EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_javascript, url_foo_https));
   EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_javascript, url_foo_port));
 
+  // Blank pages should be considered same site for anything.

[Charlie Reis, 2014/07/25 17:31:22]

Great.  Let's update this to show that it's for blank destination URLs only,
though.

[shatch, 2014/07/25 20:57:06]

Done.

+  EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_blank, url_foo));
+  EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_blank, url_foo_https));
+  EXPECT_TRUE(SiteInstance::IsSameWebSite(NULL, url_blank, url_foo_port));
+
   DrainMessageLoops();
 }