diff options
author | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-12 20:03:37 +0000 |
---|---|---|
committer | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-12 20:03:37 +0000 |
commit | 408e012e33960841aeae4dd147a5ae1828f00371 (patch) | |
tree | 75f5a56a4e6c77f2783735664e57415849d3dfbc /chrome/browser | |
parent | 10dac378df974d34fdcb7eff99e3baedb4d98a10 (diff) | |
download | chromium_src-408e012e33960841aeae4dd147a5ae1828f00371.zip chromium_src-408e012e33960841aeae4dd147a5ae1828f00371.tar.gz chromium_src-408e012e33960841aeae4dd147a5ae1828f00371.tar.bz2 |
If setProcessBackgrounded fails, don't continue through
this function. This is defensive coding being added in
part to see if it doesn't reveal anything interesting
for bug 1231568
Review URL: http://codereview.chromium.org/11003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5286 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
-rw-r--r-- | chrome/browser/render_process_host.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/browser/render_process_host.cc b/chrome/browser/render_process_host.cc index 450fdba..d5bba2b 100644 --- a/chrome/browser/render_process_host.cc +++ b/chrome/browser/render_process_host.cc @@ -751,7 +751,10 @@ RenderProcessHost* RenderProcessHost::GetExistingProcessHost(Profile* profile) { void RenderProcessHost::SetBackgrounded(bool backgrounded) { // If the process_ is NULL, the process hasn't been created yet. if (process_.handle()) { - process_.SetProcessBackgrounded(backgrounded); + bool rv = process_.SetProcessBackgrounded(backgrounded); + if (!rv) { + return; + } // Now tune the memory footprint of the renderer. // If the OS needs to page, we'd rather it page idle renderers. |