MacTerminateOnHeapCorruption shouldn't leak the OOM killer into the shared test context.

MacTerminateOnHeapCorruption shouldn't leak the OOM killer into the shared test context.

BUG=196349
TEST=SecurityTest.NewOverflow and CallocOverflow should not fail as part of a long test run


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

[Robert Sesek, 2013-04-05 18:13:22]

lgtm

https://codereview.chromium.org/13529020/diff/1/base/process_util_unittest.cc
File base/process_util_unittest.cc (right):

https://codereview.chromium.org/13529020/diff/1/base/process_util_unittest.cc...
base/process_util_unittest.cc:556: buf =
malloc(std::numeric_limits<size_t>::max() - (2 * PAGE_SIZE) - 1);},
nit: I'd balance the } on a new line indented 4 spaces.

[Avi (use Gerrit), 2013-04-05 18:23:59]

nicer?

[Robert Sesek, 2013-04-05 18:24:16]

Yes. LGTM++

[commit-bot: I haz the power, 2013-04-05 18:25:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/13529020/4001

[commit-bot: I haz the power, 2013-04-05 18:25:34]

Presubmit check for 13529020-4001 failed and returned exit status 1.

INFO:root:Found 1 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/base/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    base/process_util_unittest.cc

[Avi (use Gerrit), 2013-04-05 18:26:24]

Mark, you're an owner here.

[Mark Mentovai, 2013-04-05 18:47:06]

Definitely LGTM

[commit-bot: I haz the power, 2013-04-05 18:56:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/13529020/4001

[commit-bot: I haz the power, 2013-04-06 00:11:15]

Change committed as 192660