Commit 15bcab8b authored by Peter Collingbourne's avatar Peter Collingbourne Committed by Commit Bot

service_manager: Only start the service manager thread in the browser process.

The browser process is the only embedder process that needs a
service manager thread.

With this change we avoid spinning up an unnecessary I/O thread (and
presumably, once issue 729596 is fixed, a service manager) in
processes that are neither subprocesses nor the browser process,
such as the cloud print service process.

Bug: 855193
Change-Id: Iffae3405f6dd541ce898ac2033c9c10daefd0031
Reviewed-on: https://chromium-review.googlesource.com/1117473
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Reviewed-by: default avatarKen Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571640}
parent d47e2e3c
......@@ -449,7 +449,7 @@ int Main(const MainParams& params) {
case ProcessType::kEmbedder:
if (delegate->IsEmbedderSubprocess())
CommonSubprocessInit();
else {
if (command_line.GetSwitchValueASCII(switches::kProcessType).empty()) {
// TODO(https://crbug.com/729596): Use this task runner to start
// ServiceManager.
scoped_refptr<base::SingleThreadTaskRunner> task_runner =
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment