Hosted apps VS --site-per-process.

chrome/browser/ui/extensions/hosted_app_browsertest.cc

 // Copyright 2015 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 <string>
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/scoped_feature_list.h"
@@ skipping 266 matching lines @@
     return content::FrameMatchingPredicate(
         web_contents, base::Bind(&content::FrameMatchesName, name));
   };
   content::RenderFrameHost* app = web_contents->GetMainFrame();
   content::RenderFrameHost* same_dir = find_frame("SameOrigin-SamePath");
   content::RenderFrameHost* diff_dir = find_frame("SameOrigin-DifferentPath");
   content::RenderFrameHost* same_site = find_frame("OtherSubdomain-SameSite");
   content::RenderFrameHost* cross_site = find_frame("CrossSite");
 
   //
-  // Sanity-check sites of all relevant frames to verify test setup.
-  //
-
-  GURL app_site = content::SiteInstance::GetSiteForURL(
-      app_browser_->profile(), app->GetLastCommittedURL());
-  EXPECT_EQ(extensions::kExtensionScheme, app_site.scheme());
-
-  GURL same_dir_site = content::SiteInstance::GetSiteForURL(
-      app_browser_->profile(), same_dir->GetLastCommittedURL());
-  EXPECT_EQ(extensions::kExtensionScheme, same_dir_site.scheme());
-  EXPECT_EQ(same_dir_site, app_site);
-
-  GURL diff_dir_site = content::SiteInstance::GetSiteForURL(
-      app_browser_->profile(), diff_dir->GetLastCommittedURL());
-  EXPECT_NE(extensions::kExtensionScheme, diff_dir_site.scheme());
-  EXPECT_NE(diff_dir_site, app_site);
-
-  GURL same_site_site = content::SiteInstance::GetSiteForURL(
-      app_browser_->profile(), same_site->GetLastCommittedURL());
-  EXPECT_NE(extensions::kExtensionScheme, same_site_site.scheme());
-  EXPECT_NE(same_site_site, app_site);
-  EXPECT_EQ(same_site_site, diff_dir_site);
-
-  GURL cross_site_site = content::SiteInstance::GetSiteForURL(
-      app_browser_->profile(), cross_site->GetLastCommittedURL());
-  EXPECT_NE(cross_site_site, app_site);
-  EXPECT_NE(cross_site_site, same_site_site);
-
-  //
   // Verify that |same_dir| and |diff_dir| can script each other.
   // (they should - they have the same origin).
   //
 
   std::string same_dir_origin;
   EXPECT_TRUE(content::ExecuteScriptAndExtractString(
       same_dir, "domAutomationController.send(window.origin)",
       &same_dir_origin));
 
   std::string diff_dir_origin;
@@ skipping 11 matching lines @@
       &inner_text_from_other_frame));
   EXPECT_EQ("Simple test page.", inner_text_from_other_frame);
 
   //
   // Verify there are no additional processes for same-site frames.
   //
 
   EXPECT_EQ(same_dir->GetProcess(), app->GetProcess());
   EXPECT_EQ(diff_dir->GetProcess(), app->GetProcess());
   EXPECT_EQ(same_site->GetProcess(), app->GetProcess());
-  // For now it is okay for |cross_site| to be in any process - see
-  // https://crbug.com/679011#c2.
+  if (content::AreAllSitesIsolatedForTesting()) {
+    EXPECT_NE(same_site->GetProcess(), cross_site->GetProcess());
+  }
 }