diff options
author | wfh@chromium.org <wfh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-26 20:08:09 +0000 |
---|---|---|
committer | wfh@chromium.org <wfh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-26 20:08:09 +0000 |
commit | 853703c50540791a83ff3c151fe0b06d0282d7af (patch) | |
tree | ef293dd587146087f82c6acb988a77b1c889bb87 | |
parent | f524522e69f7d7d38d7e9f455be8a22e8bb2963e (diff) | |
download | chromium_src-853703c50540791a83ff3c151fe0b06d0282d7af.zip chromium_src-853703c50540791a83ff3c151fe0b06d0282d7af.tar.gz chromium_src-853703c50540791a83ff3c151fe0b06d0282d7af.tar.bz2 |
Do not set low priority on background tabs.
BUG=381820
Review URL: https://codereview.chromium.org/420603014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285802 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | content/browser/child_process_launcher.cc | 12 | ||||
-rw-r--r-- | content/browser/renderer_host/render_process_host_impl.cc | 4 |
2 files changed, 10 insertions, 6 deletions
diff --git a/content/browser/child_process_launcher.cc b/content/browser/child_process_launcher.cc index b49a982..a0dc5b7 100644 --- a/content/browser/child_process_launcher.cc +++ b/content/browser/child_process_launcher.cc @@ -510,11 +510,13 @@ base::TerminationStatus ChildProcessLauncher::GetChildTerminationStatus( } void ChildProcessLauncher::SetProcessBackgrounded(bool background) { - BrowserThread::PostTask( - BrowserThread::PROCESS_LAUNCHER, FROM_HERE, - base::Bind( - &ChildProcessLauncher::Context::SetProcessBackgrounded, - GetHandle(), background)); + // TODO(wfh): re-enable this after testing if dropping the priority of + // backgrounded tabs is the root cause of http://crbug.com/381820. + // BrowserThread::PostTask( + // BrowserThread::PROCESS_LAUNCHER, FROM_HERE, + // base::Bind( + // &ChildProcessLauncher::Context::SetProcessBackgrounded, + // GetHandle(), background)); } void ChildProcessLauncher::SetTerminateChildOnShutdown( diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc index be63ff2..5c3fbc3 100644 --- a/content/browser/renderer_host/render_process_host_impl.cc +++ b/content/browser/renderer_host/render_process_host_impl.cc @@ -2033,7 +2033,9 @@ void RenderProcessHostImpl::SetBackgrounded(bool backgrounded) { #endif // OS_WIN // Notify the child process of background state. - Send(new ChildProcessMsg_SetProcessBackgrounded(backgrounded)); + // TODO(wfh): re-enable this after testing if dropping the priority of + // backgrounded tabs is the root cause of http://crbug.com/381820. + // Send(new ChildProcessMsg_SetProcessBackgrounded(backgrounded)); #if !defined(OS_WIN) // Backgrounding may require elevated privileges not available to renderer |