[App Shell] Port app_shell to windows aura.(not use)

[App Shell] Port app_shell to windows aura.

Allow windows developsers to run app_shell.

BUG=None
TEST=Build app_shell on windows with use_aura=1, it launches normally.

[Haojian Wu, 2014-01-29 09:47:33]

James@, how about considering this patch? Any plan to port app_shell to windows?
It may allow windows developers to debug app_shell in visual stdio.

[James Cook, 2014-01-29 17:30:27]

Let me talk to yoz@ and miket@. Our original plan for app_shell was Chrome OS
only -- I just ported it to Linux Aura for my own convenience.  I don't know
much about Windows programming or how hard it would be to support this target.

[James Cook, 2014-01-29 20:41:59]

I talked with Yoyo. Basically, getting app_shell running on Windows is not a
priority for us, but we're willing to take patches for Windows support.

Here's what we'd like to do:
* Windows is not an official target. If it breaks, someone who cares about
Windows support will have to fix it.
* app_shell isn't currently built by the buildbots. When we start building it
there we will only build the linux/chromeos versions.  I don't want to slow down
the already-slow Windows buildbots by adding another target.

If you're OK with that I'm happy to approve this patch.

https://codereview.chromium.org/149463002/diff/1/apps/DEPS
File apps/DEPS (right):

https://codereview.chromium.org/149463002/diff/1/apps/DEPS#newcode12
apps/DEPS:12: "+sandbox",
Consider limiting this to just the file or directory you need.

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc
File apps/shell/shell_main.cc (right):

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc#new...
apps/shell/shell_main.cc:7: #include "sandbox/win/src/sandbox_types.h"
should this go inside the ifdef below?

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc#new...
apps/shell/shell_main.cc:29: #endif
optional nit: consider // defined(OS_WIN) since this is a large-ish block

[Haojian Wu, 2014-01-30 06:28:46]

I'm ok with that. Thanks.

https://codereview.chromium.org/149463002/diff/1/apps/DEPS
File apps/DEPS (right):

https://codereview.chromium.org/149463002/diff/1/apps/DEPS#newcode12
apps/DEPS:12: "+sandbox",
On 2014/01/29 20:42:00, James Cook wrote:
> Consider limiting this to just the file or directory you need.

Done. move it to /apps/shell/DEPS file.

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc
File apps/shell/shell_main.cc (right):

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc#new...
apps/shell/shell_main.cc:7: #include "sandbox/win/src/sandbox_types.h"
On 2014/01/29 20:42:00, James Cook wrote:
> should this go inside the ifdef below?

Yes. Done.

https://codereview.chromium.org/149463002/diff/1/apps/shell/shell_main.cc#new...
apps/shell/shell_main.cc:29: #endif
On 2014/01/29 20:42:00, James Cook wrote:
> optional nit: consider // defined(OS_WIN) since this is a large-ish block

Done.

[James Cook, 2014-01-30 17:32:21]

lgtm

[Haojian Wu, 2014-01-31 15:07:44]

The CQ bit was unchecked by hokein.wu@gmail.com

[Haojian Wu, 2014-01-31 15:07:45]

The CQ bit was checked by hokein.wu@gmail.com

[Haojian Wu, 2014-02-01 12:00:48]

On 2014/01/31 15:07:45, Haojian Wu wrote:
> The CQ bit was checked by mailto:hokein.wu@gmail.com

The same as readme CL, redirect to https://codereview.chromium.org/136093011/

[commit-bot: I haz the power, 2014-02-02 00:32:40]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-02 00:32:42]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-02 00:32:45]

CQ bit was unchecked on CL. Ignoring.