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

Unified Diff: chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc

Issue 468033002: Remove worker process related codes from ContentBrowserClient. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 4 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: chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc
diff --git a/chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc b/chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc
index 9e6b95c4025c6c88ca955f15bfdf2505288cd250..e9470328fd3cb56173f944b8e689f196e9be3474 100644
--- a/chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc
+++ b/chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc
@@ -335,17 +335,6 @@ bool ChromeContentBrowserClientExtensionsPart::ShouldSwapProcessesForRedirect(
}
// static
-std::string ChromeContentBrowserClientExtensionsPart::GetWorkerProcessTitle(
- const GURL& url, content::ResourceContext* context) {
- // Check if it's an extension-created worker, in which case we want to use
- // the name of the extension.
- ProfileIOData* io_data = ProfileIOData::FromResourceContext(context);
- const Extension* extension =
- io_data->GetExtensionInfoMap()->extensions().GetByID(url.host());
- return extension ? extension->name() : std::string();
-}
-
-// static
bool ChromeContentBrowserClientExtensionsPart::ShouldAllowOpenURL(
content::SiteInstance* site_instance,
const GURL& from_url,
@@ -466,33 +455,6 @@ void ChromeContentBrowserClientExtensionsPart::SiteInstanceDeleting(
site_instance->GetId()));
}
-void ChromeContentBrowserClientExtensionsPart::WorkerProcessCreated(
- SiteInstance* site_instance,
- int worker_process_id) {
- ExtensionRegistry* extension_registry =
- ExtensionRegistry::Get(site_instance->GetBrowserContext());
- if (!extension_registry)
- return;
- const Extension* extension =
- extension_registry->enabled_extensions().GetExtensionOrAppByURL(
- site_instance->GetSiteURL());
- if (!extension)
- return;
- ExtensionSystem* extension_system =
- ExtensionSystem::Get(site_instance->GetBrowserContext());
- extension_system->info_map()->RegisterExtensionWorkerProcess(
- extension->id(), worker_process_id, site_instance->GetId());
-}
-
-void ChromeContentBrowserClientExtensionsPart::WorkerProcessTerminated(
- SiteInstance* site_instance,
- int worker_process_id) {
- ExtensionSystem* extension_system =
- ExtensionSystem::Get(site_instance->GetBrowserContext());
- extension_system->info_map()->UnregisterExtensionWorkerProcess(
- worker_process_id);
-}
-
void ChromeContentBrowserClientExtensionsPart::OverrideWebkitPrefs(
RenderViewHost* rvh,
const GURL& url,

Powered by Google App Engine
This is Rietveld 408576698