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

Unified Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 289863005: [Mac] Maximise timer slack for background tabs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: A few small fixes Created 6 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index cdaa29c44e2b211fe3522dbd104124bf908dc08d..51cd57aab28c6409d45d9440c421d77c0d57dca7 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -1950,16 +1950,16 @@ void RenderProcessHostImpl::SetBackgrounded(bool backgrounded) {
// is to not invoke the SetPriorityClass API if the dll is loaded.
if (GetModuleHandle(L"cbstext.dll"))
return;
+#endif // OS_WIN
- // Windows Vista+ has a fancy process backgrounding mode that can only be set
- // from within the process. So notify the child process of background state.
+ // Notify the child process of background state.
Send(new ChildProcessMsg_SetProcessBackgrounded(backgrounded));
-#else
+#if !defined(OS_WIN)
// Backgrounding may require elevated privileges not available to renderer
// processes, so control backgrounding from the process host.
Avi (use Gerrit) 2014/05/29 14:24:16 Move the comment about Vista+ requiring doing it f
jeremy 2014/06/02 10:21:28 Done.
child_process_launcher_->SetProcessBackgrounded(backgrounded);
-#endif // OS_WIN
+#endif // !OS_WIN
}
void RenderProcessHostImpl::OnProcessLaunched() {

Powered by Google App Engine
This is Rietveld 408576698