Use the suid helper for oom_score_adj on Chrome OS.

Use the suid helper for oom_score_adj on Chrome OS.

Reverts https://codereview.chromium.org/1560033003/ for Chrome OS.
Chrome OS has a kernel patch which restricts the permissions of
oom_score_adj.

BUG=576409
Committed: https://crrev.com/64068c39982ad795f5041302500d85ce5fef54aa
Cr-Commit-Position: refs/heads/master@{#373959}

[rickyz (no longer on Chrome), 2016-02-05 21:14:17]

Created Revert of Do not use suid helper when using the namespace sandbox.

[rickyz (no longer on Chrome), 2016-02-05 21:14:40]

Description was changed from

==========
Revert of Do not use suid helper when using the namespace sandbox. (patchset #1
id:1 of https://codereview.chromium.org/1560033003/ )

Reason for revert:
Per issue 576409, an Android kernel patch prevents non-root users from writing
oom_score_adj. Reverting this while we investigate alternatives.

Original issue's description:
> Do not use suid helper when using the namespace sandbox.
> 
> After https://codereview.chromium.org/1519753002, sandboxed processes
> are no longer marked non-dumpable under the namespace sandbox. This
> means that we no longer need to use a suid binary to change oom scores
> for those processes.
> 
> BUG=312380
> 
> Committed: https://crrev.com/baf4d84696ebd6088363888239c3aef5f5a735f7
> Cr-Commit-Position: refs/heads/master@{#367671}

TBR=mdempsky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=312380
==========

to

==========
Revert of Do not use suid helper when using the namespace sandbox. (patchset #1
id:1 of https://codereview.chromium.org/1560033003/ )

Reason for revert:
Per issue 576409, an Android kernel patch prevents non-root users from writing
oom_score_adj. Reverting this while we investigate alternatives.

Original issue's description:
> Do not use suid helper when using the namespace sandbox.
> 
> After https://codereview.chromium.org/1519753002, sandboxed processes
> are no longer marked non-dumpable under the namespace sandbox. This
> means that we no longer need to use a suid binary to change oom scores
> for those processes.
> 
> BUG=312380
> 
> Committed: https://crrev.com/baf4d84696ebd6088363888239c3aef5f5a735f7
> Cr-Commit-Position: refs/heads/master@{#367671}

TBR=mdempsky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=576409
==========

[commit-bot: I haz the power, 2016-02-05 21:16:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1671233002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1671233002/1

[commit-bot: I haz the power, 2016-02-05 21:19:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-05 21:19:40]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rickyz (no longer on Chrome), 2016-02-05 21:38:54]

Description was changed from

==========
Revert of Do not use suid helper when using the namespace sandbox. (patchset #1
id:1 of https://codereview.chromium.org/1560033003/ )

Reason for revert:
Per issue 576409, an Android kernel patch prevents non-root users from writing
oom_score_adj. Reverting this while we investigate alternatives.

Original issue's description:
> Do not use suid helper when using the namespace sandbox.
> 
> After https://codereview.chromium.org/1519753002, sandboxed processes
> are no longer marked non-dumpable under the namespace sandbox. This
> means that we no longer need to use a suid binary to change oom scores
> for those processes.
> 
> BUG=312380
> 
> Committed: https://crrev.com/baf4d84696ebd6088363888239c3aef5f5a735f7
> Cr-Commit-Position: refs/heads/master@{#367671}

TBR=mdempsky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=576409
==========

to

==========
Use the suid helper for oom_score_adj on Chrome OS.

Reverts https://codereview.chromium.org/1560033003/ for Chrome OS.
Chrome OS has a kernel patch which restricts the permissions of
oom_score_adj.

BUG=576409
==========

[rickyz (no longer on Chrome), 2016-02-05 21:41:16]

I've un-TBRed this since the revert didn't apply cleanly, and scoped the revert
to just Chrome OS now.

[mdempsky, 2016-02-05 22:38:36]

lgtm

[rickyz (no longer on Chrome), 2016-02-05 23:56:13]

The CQ bit was checked by rickyz@chromium.org

[commit-bot: I haz the power, 2016-02-05 23:58:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1671233002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1671233002/40001

[commit-bot: I haz the power, 2016-02-06 00:11:02]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-02-06 00:12:22]

Description was changed from

==========
Use the suid helper for oom_score_adj on Chrome OS.

Reverts https://codereview.chromium.org/1560033003/ for Chrome OS.
Chrome OS has a kernel patch which restricts the permissions of
oom_score_adj.

BUG=576409
==========

to

==========
Use the suid helper for oom_score_adj on Chrome OS.

Reverts https://codereview.chromium.org/1560033003/ for Chrome OS.
Chrome OS has a kernel patch which restricts the permissions of
oom_score_adj.

BUG=576409

Committed: https://crrev.com/64068c39982ad795f5041302500d85ce5fef54aa
Cr-Commit-Position: refs/heads/master@{#373959}
==========

[commit-bot: I haz the power, 2016-02-06 00:12:23]

Patchset 2 (id:??) landed as
https://crrev.com/64068c39982ad795f5041302500d85ce5fef54aa
Cr-Commit-Position: refs/heads/master@{#373959}