Index: chrome/browser/site_details.cc |
diff --git a/chrome/browser/site_details.cc b/chrome/browser/site_details.cc |
index 8ad341db3e9ade6cf764673e9a3cb7a43d58e09d..1f3ff9170335e712b6c3acd326aeb52b3383e4bc 100644 |
--- a/chrome/browser/site_details.cc |
+++ b/chrome/browser/site_details.cc |
@@ -40,19 +40,6 @@ bool ShouldIsolate(BrowserContext* browser_context, |
// the New Tab Page gets counted as two processes under this policy, and |
// extensions are isolated as well. |
return !site.SchemeIs(url::kHttpScheme); |
- case ISOLATE_EXTENSIONS: { |
ncarter (slow)
2017/05/01 20:15:19
I don't think it's necessary to remove this? If a
nasko
2017/05/01 21:25:37
The way I read this code is "What should our numbe
ncarter (slow)
2017/05/01 21:37:43
The value of computing the simulated ISOLATE_EXTEN
|
-#if !BUILDFLAG(ENABLE_EXTENSIONS) |
- return false; |
-#else |
- if (!site.SchemeIs(extensions::kExtensionScheme)) |
- return false; |
- extensions::ExtensionRegistry* registry = |
- extensions::ExtensionRegistry::Get(browser_context); |
- const extensions::Extension* extension = |
- registry->enabled_extensions().GetExtensionOrAppByURL(site); |
- return extension && !extension->is_hosted_app(); |
-#endif |
- } |
} |
NOTREACHED(); |
return true; |
@@ -277,17 +264,4 @@ void SiteDetails::UpdateHistograms( |
UMA_HISTOGRAM_COUNTS_100( |
"SiteIsolation.IsolateHttpsSitesTotalProcessCountEstimate", |
process_count_estimate[ISOLATE_HTTPS_SITES] + non_renderer_process_count); |
- |
- // ISOLATE_EXTENSIONS metrics. |
- UMA_HISTOGRAM_COUNTS_100("SiteIsolation.IsolateExtensionsProcessCountNoLimit", |
- num_isolated_site_instances[ISOLATE_EXTENSIONS]); |
- UMA_HISTOGRAM_COUNTS_100( |
- "SiteIsolation.IsolateExtensionsProcessCountLowerBound", |
- process_count_lower_bound[ISOLATE_EXTENSIONS]); |
- UMA_HISTOGRAM_COUNTS_100( |
- "SiteIsolation.IsolateExtensionsProcessCountEstimate", |
- process_count_estimate[ISOLATE_EXTENSIONS]); |
- UMA_HISTOGRAM_COUNTS_100( |
- "SiteIsolation.IsolateExtensionsTotalProcessCountEstimate", |
- process_count_estimate[ISOLATE_EXTENSIONS] + non_renderer_process_count); |
} |