[NaCl SDK] Update Windows build to use vs2013 from depot_tools.

[NaCl SDK] Update Windows build to use vs2013 from depot_tools.

I'm not sure why this was working on the bots, but the sdk build was no longer
working on my Windows machine. After some research, I found that Chromium must
be built with vs2013, which was not supported by build_sdk, or by the Native
Client build (see https://code.google.com/p/nativeclient/issues/detail?id=4022)

This change modifies build_sdk.py (and build_artifacts.py) to use VS2013 from
depot_tools if it is available, as well as switching to using gyp_chromium
instead of gyp_nacl for building Native Client host binaries.

BUG=none
R=bradnelson@chromium.org

Committed: https://crrev.com/dcdde1cd950b05f33520c40b63de24410d7deab4
Cr-Commit-Position: refs/heads/master@{#313319}

[bradn, 2014-12-15 23:10:05]

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

[bradn, 2014-12-15 23:10:06]

lgtm

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
File native_client_sdk/src/build_tools/build_artifacts.py (right):

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
native_client_sdk/src/build_tools/build_artifacts.py:369: # gyp_nacl =
join(NACL_DIR, 'build', 'gyp_nacl')
leave this in?

[Sam Clegg, 2014-12-15 23:17:41]

sbc@chromium.org changed reviewers:
+ sbc@chromium.org

[Sam Clegg, 2014-12-15 23:17:41]

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
File native_client_sdk/src/build_tools/build_artifacts.py (right):

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
native_client_sdk/src/build_tools/build_artifacts.py:370: gyp_nacl =
gyp_chromium
Does this slow down the build at all?

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
native_client_sdk/src/build_tools/build_sdk.py:579:
buildbot_common.RemoveDir(os.path.join(OUT_DIR, GYPBUILD_DIR) + '-pnacl-arm')
Why?

[binji, 2014-12-16 01:08:33]

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
File native_client_sdk/src/build_tools/build_artifacts.py (right):

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
native_client_sdk/src/build_tools/build_artifacts.py:370: gyp_nacl =
gyp_chromium
On 2014/12/15 23:17:41, Sam Clegg wrote:
> Does this slow down the build at all?

Doesn't seem to.

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/811533002/diff/1/native_client_sdk/src/build_...
native_client_sdk/src/build_tools/build_sdk.py:579:
buildbot_common.RemoveDir(os.path.join(OUT_DIR, GYPBUILD_DIR) + '-pnacl-arm')
On 2014/12/15 23:17:41, Sam Clegg wrote:
> Why?

When I first rebuilt on Windows, the previous pdb files were from VS2010, but
the new ones were for VS2013, so the build failed.

We should have been cleaning these directories anyway on the buildbots (although
it does mean that the build will be slower). Perhaps this should be on a flag?

[binji, 2014-12-17 00:57:22]

moved the gypbuild directory removal to its own CL

[binji, 2014-12-18 23:47:10]

On 2014/12/17 00:57:22, binji wrote:
> moved the gypbuild directory removal to its own CL

sbc: any other comments before I commit?

[Sam Clegg, 2014-12-19 02:13:31]

lgtm.

[binji, 2014-12-22 23:07:10]

updated to use clang for ARM, PTAL

[Sam Clegg, 2014-12-24 12:58:12]

lgtm % trybots

[binji, 2015-01-27 18:29:52]

The CQ bit was checked by binji@chromium.org

[commit-bot: I haz the power, 2015-01-27 18:31:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/811533002/120001

[commit-bot: I haz the power, 2015-01-27 18:59:28]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-01-27 19:02:21]

Patchset 7 (id:??) landed as
https://crrev.com/dcdde1cd950b05f33520c40b63de24410d7deab4
Cr-Commit-Position: refs/heads/master@{#313319}