Revert 284349 "Add winrt to the INCLUDE directories of the toolc..."

Revert 284349 "Add winrt to the INCLUDE directories of the toolc..."

Sorry, I screwed up. This breaks Express-only per skia:2781. I'll reland it before I push the next VS update.

> Add winrt to the INCLUDE directories of the toolchain bundler.
> 
> The motivation is that vcvarsall sets it, and that it might make it possible to
> land https://codereview.chromium.org/405723002/ one day.
> 
> BUG=395405
> TBR=jochen
> 
> Review URL: https://codereview.chromium.org/393233004

TBR=thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285924

[scottmg, 2014-07-28 16:42:16]

Committed patchset #1 manually as r285924 (tree was closed).

[Nico, 2015-05-05 16:40:40]

(out of interest, did this ever get relanded?)

[scottmg, 2015-05-05 17:06:26]

What year is it!?!??

It looks like it did not, sorry.

As we don't use the Express toolchain any longer though, it would be safe to
reland now if you like. That is, the script is only used during packaging the
toolchain, not on average developer machines which only unzip the toolchain now.

On the other hand, there probably won't be any further revs to 2013, so I'm not
sure that it'll make much difference either way as there'll probably be a
variety of changes for 2015 (e.g. not having a winrt directory).

[Nico, 2015-05-05 17:35:43]

ok, then let's wait for 2015

(Maybe you could send that to us a while before you check it in, so that we can
make sure clang can handle the new headers. we looked at it some a while back,
but probably good to check shortly before the switch)

[scottmg, 2015-05-05 17:48:54]

On 2015/05/05 17:35:43, Nico wrote:
> ok, then let's wait for 2015
> 
> (Maybe you could send that to us a while before you check it in, so that we
can
> make sure clang can handle the new headers. we looked at it some a while back,
> but probably good to check shortly before the switch)

Will do. There's a linker crash in the current RC that's frequent enough that'll
it block switching. So unless that's fixed before RTM we probably won't be
upgrading that soon. But that would be a good time to look into headers anyway.

[Nico, 2015-05-05 17:51:21]

On 2015/05/05 17:48:54, scottmg wrote:
> On 2015/05/05 17:35:43, Nico wrote:
> > ok, then let's wait for 2015
> > 
> > (Maybe you could send that to us a while before you check it in, so that we
> can
> > make sure clang can handle the new headers. we looked at it some a while
back,
> > but probably good to check shortly before the switch)
> 
> Will do. There's a linker crash in the current RC that's frequent enough
that'll
> it block switching. So unless that's fixed before RTM we probably won't be
> upgrading that soon. But that would be a good time to look into headers
anyway.

We could use 2015 + lld :-P

(not really)