Fix split in base::ProcessIsProcessBackgrounded()

Fix split in base::ProcessIsProcessBackgrounded()

Most Unix files end in newline, so by having the contents of
/proc/<pid>/cgroup by newline with SPLIT_WANT_ALL, the last entry was
always blank, triggering a NOTREACHED() and polluting the log. This
change uses SPLIT_WANT_NONEMPTY instead to get rid of empty lines.

BUG=548818
TEST=base_unittests --gtest_filter=ProcessTest.TestIsProcessBackgroundedCGroup

Committed: https://crrev.com/99fbcf8043ece4e72b5d15fee9be6d29b562aa06
Cr-Commit-Position: refs/heads/master@{#361383}

[Luis Héctor Chávez, 2015-11-23 22:21:57]

lhchavez@chromium.org changed reviewers:
+ afakhry@chromium.org, thestig@chromium.org

[Luis Héctor Chávez, 2015-11-23 22:21:57]

PTAL

[afakhry, 2015-11-23 22:24:54]

lgtm

[Lei Zhang, 2015-11-23 22:29:20]

Does the unit test actually cover this scenario? If not, can you add a new test
case for this?

[lhc(google), 2015-11-24 02:22:34]

lhchavez@google.com changed reviewers:
+ lhchavez@google.com

[lhc(google), 2015-11-24 02:22:34]

Changed the tests to cover this.

[Lei Zhang, 2015-11-24 02:27:50]

lgtm, thanks

[lhc(google), 2015-11-24 02:28:27]

The CQ bit was checked by lhchavez@google.com

[lhc(google), 2015-11-24 02:28:27]

The patchset sent to the CQ was uploaded after l-g-t-m from afakhry@chromium.org
Link to the patchset: https://codereview.chromium.org/1474483002/#ps40001
(title: "Added tests")

[commit-bot: I haz the power, 2015-11-24 02:29:03]

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

[commit-bot: I haz the power, 2015-11-24 04:14:54]

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

[commit-bot: I haz the power, 2015-11-24 04:14:54]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Luis Héctor Chávez, 2015-11-24 17:03:27]

The CQ bit was checked by lhchavez@chromium.org

[commit-bot: I haz the power, 2015-11-24 17:06:19]

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

[commit-bot: I haz the power, 2015-11-24 17:54:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-24 17:55:44]

Patchset 3 (id:??) landed as
https://crrev.com/99fbcf8043ece4e72b5d15fee9be6d29b562aa06
Cr-Commit-Position: refs/heads/master@{#361383}