Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(635)

Unified Diff: content/common/site_isolation_policy.h

Issue 1377933004: Modify --isolate-extensions to not isolate hosted apps. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@no_isolate_apps3
Patch Set: One more comment fix. Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/site_isolation_policy.h
diff --git a/content/common/site_isolation_policy.h b/content/common/site_isolation_policy.h
index 86020c0ef5606d38c1a0799b5f266dfddad467af..68d1e442e74de284db738580080efa8c9220d4be 100644
--- a/content/common/site_isolation_policy.h
+++ b/content/common/site_isolation_policy.h
@@ -34,15 +34,8 @@ class CONTENT_EXPORT SiteIsolationPolicy {
// particular site merits protection.
static bool AreCrossProcessFramesPossible();
- // Returns true if pages loaded from |effective_url| ought to be handled only
- // by a renderer process isolated from other sites. If --site-per-process is
- // on the command line, this is true for all sites. In other site isolation
- // modes, only a subset of sites will require dedicated processes.
- //
- // |effective_url| must be an effective URL -- practically speaking, that
- // means that this function should only be called on the UI thread in the
- // browser process.
- static bool DoesSiteRequireDedicatedProcess(const GURL& effective_url);
+ // Returns true if every site should be placed in a dedicated process.
+ static bool UseDedicatedProcessesForAllSites();
// Returns true if navigation and history code should maintain per-frame
// navigation entries. This is an in-progress feature related to site
@@ -61,13 +54,6 @@ class CONTENT_EXPORT SiteIsolationPolicy {
// should be removed and its callers cleaned up.
static bool IsSwappedOutStateForbidden();
- // Overrides the default site isolation mode so that all sites are
- // isolated. Returns true if successful. Can fail if SiteIsolationPolicy
- // has already been consulted at runtime prior to the override call --
- // if so, try calling this earlier in the test, maybe before creating
- // any renderer processes.
- static bool IsolateAllSitesForTesting();
-
private:
SiteIsolationPolicy(); // Not instantiable.

Powered by Google App Engine
This is Rietveld 408576698