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

Unified Diff: content/public/browser/content_browser_client.h

Issue 2399853003: [M54 merge] Lock down creation of blob:chrome-extension URLs from non-extension processes. (Closed)
Patch Set: Rebase Created 4 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/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index d22a4866c6164a40879e6ae05e492906391c19b3..6da43c333166b7691eb7337088db5b0da45b070d 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -191,17 +191,17 @@ class CONTENT_EXPORT ContentBrowserClient {
virtual bool ShouldUseProcessPerSite(BrowserContext* browser_context,
const GURL& effective_url);
- // Returns true if site isolation should be enabled for |effective_url|. This
- // call allows the embedder to supplement the site isolation policy enforced
- // by the content layer.
+ // Returns true if site isolation should be enabled for |effective_site_url|.
+ // This call allows the embedder to supplement the site isolation policy
+ // enforced by the content layer.
//
// Will only be called if both of the following happen:
// 1. The embedder asked to be consulted, by returning true from
// ContentClient::IsSupplementarySiteIsolationModeEnabled().
- // 2. The content layer didn't decide to isolate |effective_url| according
- // to its internal policy (e.g. because of --site-per-process).
+ // 2. The content layer didn't decide to isolate |effective_site_url|
+ // according to its internal policy (e.g. because of --site-per-process).
virtual bool DoesSiteRequireDedicatedProcess(BrowserContext* browser_context,
- const GURL& effective_url);
+ const GURL& effective_site_url);
// Returns true unless the effective URL is part of a site that cannot live in
// a process restricted to just that site. This is only called if site
« no previous file with comments | « content/public/browser/child_process_security_policy.h ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698