Changing command line flag from disable appcontainer to enable appcontainer.

Changing command line flag from disable appcontainer to enable appcontainer.
As there are some reports on stable channel which suggest that CreateProcess is randomly failing when used with lowbox token, changing disable appcontainer flag to enable if required.

BUG=467920
R=jschuh@chromium.org,wfh@chromium.org

Committed: https://crrev.com/354d94497b96ffa913a7c5d562d2d88d658b9401
Cr-Commit-Position: refs/heads/master@{#340815}

[Will Harris, 2015-07-28 03:04:30]

I think we should aim to get actionable data here.

I suggest putting this behind a experiment which we can control, then we can do
an field-trial on some of the population and this will allow us to compare
stability numbers across the population, via stability dashboard.

It is my understanding that these crashes don't generate crash reports, but we
should still get reliable stability data from this experiment.

[Will Harris, 2015-07-28 03:22:36]

I looked through the stability data for bug 467920 and can't find any
correlation here that suggest that app container is to blame for these crashes,
but then I can't even see the crashes, the stability of M44 is actually 10%
better than M43 and win81 is the most stable platform. I'm requesting more
information on the bug.

[Shrikant Kelkar, 2015-07-28 03:36:57]

On 2015/07/28 03:22:36, Will Harris wrote:
> I looked through the stability data for bug 467920 and can't find any
> correlation here that suggest that app container is to blame for these
crashes,
> but then I can't even see the crashes, the stability of M44 is actually 10%
> better than M43 and win81 is the most stable platform. I'm requesting more
> information on the bug.

I am outside right now, let's talk tomorrow in-person. Data to look here is UMA.

[Shrikant Kelkar, 2015-07-28 19:57:36]

On 2015/07/28 03:36:57, Shrikant Kelkar wrote:
> On 2015/07/28 03:22:36, Will Harris wrote:
> > I looked through the stability data for bug 467920 and can't find any
> > correlation here that suggest that app container is to blame for these
> crashes,
> > but then I can't even see the crashes, the stability of M44 is actually 10%
> > better than M43 and win81 is the most stable platform. I'm requesting more
> > information on the bug.
> 
> I am outside right now, let's talk tomorrow in-person. Data to look here is
UMA.

ping.. after discussion.

[Will Harris, 2015-07-28 20:21:25]

lgtm for sandbox_win.cc - probably need a content/ owner for the rest.

[Will Harris, 2015-07-28 20:31:24]

wfh@chromium.org changed reviewers:
+ nasko@chromium.org

[Will Harris, 2015-07-28 20:31:31]

+nasko

[nasko, 2015-07-28 22:33:27]

LGTM

[Will Harris, 2015-07-28 22:50:28]

The CQ bit was checked by wfh@chromium.org

[commit-bot: I haz the power, 2015-07-28 22:51:27]

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

[commit-bot: I haz the power, 2015-07-28 23:57:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-28 23:57:58]

Patchset 1 (id:??) landed as
https://crrev.com/354d94497b96ffa913a7c5d562d2d88d658b9401
Cr-Commit-Position: refs/heads/master@{#340815}