summaryrefslogtreecommitdiffstats
path: root/chrome/browser
diff options
context:
space:
mode:
authormbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-12 20:03:37 +0000
committermbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-12 20:03:37 +0000
commit408e012e33960841aeae4dd147a5ae1828f00371 (patch)
tree75f5a56a4e6c77f2783735664e57415849d3dfbc /chrome/browser
parent10dac378df974d34fdcb7eff99e3baedb4d98a10 (diff)
downloadchromium_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.cc5
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.