Set allow_new_privs when launching CRD host controller from the NM process.

Set allow_new_privs when launching CRD host controller from the NM process.

Previously no_new_prevs mode was used when starting CRD host from NM. This
prevents sudo from working properly. r273785 fixed similar problem for the
code that launched NM binaries within chrome.

BUG=378012
R=lambroslambrou@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273983

[Sergey Ulanov, 2014-05-31 00:38:45]

jln@, FYI. This change doesn't affect Chrome of Chrome OS.

[Lambros, 2014-05-31 00:42:20]

LGTM
Any other usages of base::LaunchProcess we need to check?

https://codereview.chromium.org/302083007/diff/1/remoting/host/setup/daemon_c...
File remoting/host/setup/daemon_controller_delegate_linux.cc (right):

https://codereview.chromium.org/302083007/diff/1/remoting/host/setup/daemon_c...
remoting/host/setup/daemon_controller_delegate_linux.cc:106: #if
defined(OS_LINUX) && !defined(OS_CHROMEOS)
Maybe remove OS_LINUX, since this file is .._linux.cc ?

[Sergey Ulanov, 2014-05-31 00:45:32]

> Any other usages of base::LaunchProcess we need to check?

No, this is the only place where we use it on Lunux.

https://codereview.chromium.org/302083007/diff/1/remoting/host/setup/daemon_c...
File remoting/host/setup/daemon_controller_delegate_linux.cc (right):

https://codereview.chromium.org/302083007/diff/1/remoting/host/setup/daemon_c...
remoting/host/setup/daemon_controller_delegate_linux.cc:106: #if
defined(OS_LINUX) && !defined(OS_CHROMEOS)
On 2014/05/31 00:42:20, Lambros wrote:
> Maybe remove OS_LINUX, since this file is .._linux.cc ?
> 

Done.

[Sergey Ulanov, 2014-05-31 00:45:43]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-05-31 00:54:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/302083007/20001

[Sergey Ulanov, 2014-05-31 01:39:08]

Committed patchset #2 manually as r273983 (presubmit successful).