Prepare for using ninja for win64.

Prepare for using ninja for win64.

BUG=chromium:508921
LOG=n

Committed: https://crrev.com/0a3b77397b59f3146a6eb4780de3326c4248eb33
Cr-Commit-Position: refs/heads/master@{#29605}

[Michael Achenbach, 2015-07-10 14:07:55]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, jochen@chromium.org

[Michael Achenbach, 2015-07-10 20:01:42]

PTAL - this will be accompanied with an infra-side fix on the ninja bot (if
neccessary).

[Jakob Kummerow, 2015-07-13 08:33:50]

LGTM if it works.

https://codereview.chromium.org/1234443003/diff/1/build/standalone.gypi
File build/standalone.gypi (right):

https://codereview.chromium.org/1234443003/diff/1/build/standalone.gypi#newco...
build/standalone.gypi:361: 'conditions': [
nit: indentation (one more space).

[Michael Achenbach, 2015-07-13 10:42:11]

On 2015/07/13 08:33:50, Jakob wrote:
> LGTM if it works.
> 
> https://codereview.chromium.org/1234443003/diff/1/build/standalone.gypi
> File build/standalone.gypi (right):
> 
>
https://codereview.chromium.org/1234443003/diff/1/build/standalone.gypi#newco...
> build/standalone.gypi:361: 'conditions': [
> nit: indentation (one more space).

Done.

Good question whether it works. It already looks as if it worked now:
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_ninja_rel/builds/1

But I want to take the warning
https://code.google.com/p/chromium/issues/detail?id=508921#c6 seriously and add
a patch on the build side first.

The configuration is basically a dummy, as our configurations are
bitness-independent. Whatever target_arch is set to is authoritative. I don't
know what would be compiled with target_arch=ia32 using Release_x64, but we
won't do that.

[Michael Achenbach, 2015-07-13 11:09:55]

PTAL at patch 3. This makes things work after
https://codereview.chromium.org/1231423003/ lands.

[Jakob Kummerow, 2015-07-13 11:12:04]

Patch set 3 LGTM.

[jochen (gone - plz use gerrit), 2015-07-13 11:12:54]

lgtm

[Michael Achenbach, 2015-07-13 11:35:21]

Looks like this works:
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_ninja_rel/builds/2

[Michael Achenbach, 2015-07-13 11:35:25]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-07-13 11:35:34]

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

[commit-bot: I haz the power, 2015-07-13 12:39:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-13 12:39:35]

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