Commit ffa9760e authored by creis@chromium.org's avatar creis@chromium.org

Don't allow the task manager to share a process with anything else.

BUG=103605
TEST=Open more tabs than process limit, then preferences, then task manager.  Killing preferences shouldn't kill task manager.

Review URL: http://codereview.chromium.org/8520019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110520 0039d316-1c4b-4281-b951-d872f2087c98
parent 69e361eb
......@@ -401,6 +401,14 @@ bool ChromeContentBrowserClient::IsSuitableHost(
ExtensionService* service = profile->GetExtensionService();
extensions::ProcessMap* process_map = service->process_map();
// Don't allow the Task Manager to share a process with anything else.
// Otherwise it can affect the renderers it is observing.
// Note: we could create another RenderProcessHostPrivilege bucket for
// this to allow multiple chrome://tasks instances to share, but that's
// a very unlikely case without serious consequences.
if (site_url.GetOrigin() == GURL(chrome::kChromeUITaskManagerURL).GetOrigin())
return false;
// These may be NULL during tests. In that case, just assume any site can
// share any host.
if (!service || !process_map)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment