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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 8827013: Move/replace/rename URL-based extension getters from ExtensionService to/in ExtensionSet. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: extent Created 9 years 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: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 6856cbfdd47ad4dd3021d2bd76f4444aebf8b0fa..b57e0437fce64cf3a6b1c67ebe21f84f124b3f5a 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -178,7 +178,8 @@ RenderProcessHostPrivilege GetPrivilegeRequiredByUrl(
return PRIV_NORMAL;
if (url.SchemeIs(chrome::kExtensionScheme)) {
- const Extension* extension = service->GetExtensionByURL(url);
+ const Extension* extension =
+ service->extensions()->GetByURL(ExtensionURLInfo(url));
if (extension && extension->is_storage_isolated())
return PRIV_ISOLATED;
if (extension && extension->is_hosted_app())
@@ -377,7 +378,7 @@ GURL ChromeContentBrowserClient::GetEffectiveURL(
return url;
const Extension* extension =
- profile->GetExtensionService()->GetExtensionByWebExtent(url);
+ profile->GetExtensionService()->extensions()->GetByWebExtent(url);
if (!extension)
return url;
@@ -403,8 +404,8 @@ bool ChromeContentBrowserClient::ShouldUseProcessPerSite(
if (!profile || !profile->GetExtensionService())
return false;
- const Extension* extension =
- profile->GetExtensionService()->GetExtensionByURL(effective_url);
+ const Extension* extension = profile->GetExtensionService()->extensions()->
+ GetByURL(ExtensionURLInfo(effective_url));
if (!extension)
return false;
@@ -485,9 +486,7 @@ void ChromeContentBrowserClient::SiteInstanceGotProcess(
return;
const Extension* extension =
- service->GetExtensionByURL(site_instance->site());
- if (!extension)
- extension = service->GetExtensionByWebExtent(site_instance->site());
+ service->extensions()->GetByURL(ExtensionURLInfo(site_instance->site()));
if (!extension)
return;
@@ -515,9 +514,7 @@ void ChromeContentBrowserClient::SiteInstanceDeleting(
return;
const Extension* extension =
- service->GetExtensionByURL(site_instance->site());
- if (!extension)
- extension = service->GetExtensionByWebExtent(site_instance->site());
+ service->extensions()->GetByURL(ExtensionURLInfo(site_instance->site()));
if (!extension)
return;

Powered by Google App Engine
This is Rietveld 408576698