Move gyp-win-tool to the GN Windows toolchain.

Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_tool.py

The copy step of the old tool is currently kept pending a fix to NaCl that references it.

I changed internal references to itself to tool_wrapper.py, and split the internal strings containing the word "Copyright" (which it uses to filter tool output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer copied to what will be the current directory when linking, I believe this will be broken. But the GN build does not use the manifest linking command. To keep changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl roll).

BUG=642014
R=scottmg@chromium.org

Committed: https://crrev.com/3826c51de883b9beedc92de5c17e9946a92dfe6d
Cr-Commit-Position: refs/heads/master@{#415034}

[brettw, 2016-08-29 17:13:03]

fix

[brettw, 2016-08-29 17:13:04]

The CQ bit was checked by brettw@chromium.org to run a CQ dry run

[brettw, 2016-08-29 17:14:02]

Description was changed from

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes.
==========

to

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

TBR=scottmg@chromium.org
==========

[brettw, 2016-08-29 17:14:03]

brettw@chromium.org changed reviewers:
+ scottmg@chromium.org

[brettw, 2016-08-29 17:14:18]

Description was changed from

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

TBR=scottmg@chromium.org
==========

to

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

BUG=642014
TBR=scottmg@chromium.org
==========

[brettw, 2016-08-29 17:14:36]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2016-08-29 17:14:39]

The CQ bit was checked by brettw@chromium.org

[scottmg, 2016-08-29 17:17:17]

lgtm

[commit-bot: I haz the power, 2016-08-29 17:17:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[brettw, 2016-08-29 17:23:02]

Description was changed from

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

BUG=642014
TBR=scottmg@chromium.org
==========

to

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

BUG=642014
R=scottmg@chromium.org
==========

[brettw, 2016-08-29 19:58:41]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2016-08-29 19:58:45]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-08-29 20:21:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-29 22:09:21]

Description was changed from

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

BUG=642014
R=scottmg@chromium.org
==========

to

==========
Move gyp-win-tool to the GN Windows toolchain.

Renames it tool_wrapper.py but otherwise keeps it unchanged from the original
https://chromium.googlesource.com/external/gyp.git/+/master/pylib/gyp/win_too...

The copy step of the old tool is currently kept pending a fix to NaCl that
references it.

I changed internal references to itself to tool_wrapper.py, and split the
internal strings containing the word "Copyright" (which it uses to filter tool
output) to avoid triggering the copyright presubmit check.

Changes the build to refer to the file in-place rather than copying it to the
build directory which was confusing.

The tool runs itself in the link_manifest step. Since the file is no longer
copied to what will be the current directory when linking, I believe this will
be broken. But the GN build does not use the manifest linking command. To keep
changes minimal, I kept this code in for now.

Reland of http://crrev.com/2287603003 with fixes (keep the tool pending NaCl
roll).

BUG=642014
R=scottmg@chromium.org

Committed: https://crrev.com/3826c51de883b9beedc92de5c17e9946a92dfe6d
Cr-Commit-Position: refs/heads/master@{#415034}
==========

[commit-bot: I haz the power, 2016-08-29 22:09:22]

Patchset 2 (id:??) landed as
https://crrev.com/3826c51de883b9beedc92de5c17e9946a92dfe6d
Cr-Commit-Position: refs/heads/master@{#415034}