Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(289)

Side by Side Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 989703002: Add support for backgrounding processes on the Mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nits. Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Represents the browser side of the browser <--> renderer communication 5 // Represents the browser side of the browser <--> renderer communication
6 // channel. There will be one RenderProcessHost per renderer process. 6 // channel. There will be one RenderProcessHost per renderer process.
7 7
8 #include "content/browser/renderer_host/render_process_host_impl.h" 8 #include "content/browser/renderer_host/render_process_host_impl.h"
9 9
10 #include <algorithm> 10 #include <algorithm>
11 #include <limits> 11 #include <limits>
(...skipping 2233 matching lines...) Expand 10 before | Expand all | Expand 10 after
2245 #endif // OS_WIN 2245 #endif // OS_WIN
2246 2246
2247 #if defined(OS_WIN) 2247 #if defined(OS_WIN)
2248 // Same as below, but bound to an experiment (http://crbug.com/458594) 2248 // Same as below, but bound to an experiment (http://crbug.com/458594)
2249 // initially on Windows. Enabled by default in the asbence of field trials to 2249 // initially on Windows. Enabled by default in the asbence of field trials to
2250 // get coverage on the perf waterfall. 2250 // get coverage on the perf waterfall.
2251 base::FieldTrial* trial = 2251 base::FieldTrial* trial =
2252 base::FieldTrialList::Find("BackgroundRendererProcesses"); 2252 base::FieldTrialList::Find("BackgroundRendererProcesses");
2253 if (!trial || trial->group_name() != "Disallow") 2253 if (!trial || trial->group_name() != "Disallow")
2254 child_process_launcher_->SetProcessBackgrounded(backgrounded); 2254 child_process_launcher_->SetProcessBackgrounded(backgrounded);
2255 #elif defined(OS_MACOSX)
2256 const std::string trial_group_name =
2257 base::FieldTrialList::FindFullName("BackgroundRendererProcesses");
2258 if (trial_group_name.length() > 0 && trial_group_name != "Disallow") {
2259 child_process_launcher_->SetProcessBackgrounded(backgrounded);
2260 }
2255 #else 2261 #else
2256 // Control the background state from the browser process, otherwise the task 2262 // Control the background state from the browser process, otherwise the task
2257 // telling the renderer to "unbackground" itself may be preempted by other 2263 // telling the renderer to "unbackground" itself may be preempted by other
2258 // tasks executing at lowered priority ahead of it or simply by not being 2264 // tasks executing at lowered priority ahead of it or simply by not being
2259 // swiftly scheduled by the OS per the low process priority 2265 // swiftly scheduled by the OS per the low process priority
2260 // (http://crbug.com/398103). 2266 // (http://crbug.com/398103).
2261 child_process_launcher_->SetProcessBackgrounded(backgrounded); 2267 child_process_launcher_->SetProcessBackgrounded(backgrounded);
2262 #endif // OS_WIN 2268 #endif // OS_WIN
2263 2269
2264 // Notify the child process of background state. 2270 // Notify the child process of background state.
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
2480 if (worker_ref_count_ == 0) 2486 if (worker_ref_count_ == 0)
2481 Cleanup(); 2487 Cleanup();
2482 } 2488 }
2483 2489
2484 void RenderProcessHostImpl::GetAudioOutputControllers( 2490 void RenderProcessHostImpl::GetAudioOutputControllers(
2485 const GetAudioOutputControllersCallback& callback) const { 2491 const GetAudioOutputControllersCallback& callback) const {
2486 audio_renderer_host()->GetOutputControllers(callback); 2492 audio_renderer_host()->GetOutputControllers(callback);
2487 } 2493 }
2488 2494
2489 } // namespace content 2495 } // namespace content
OLDNEW
« content/browser/child_process_launcher.cc ('K') | « content/browser/child_process_launcher.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698