Fix llvm_symbolizer target on Win-Clang bots

Fix llvm_symbolizer target on Win-Clang bots

This target was introduced in
https://crrev.com/8207d3548f95ab41fec7734fcc4439c223102bc6, and was
made a dependency of the chromium_builder_asan target in
https://crrev.com/91f1267dbeddfdfdb5ac87de26932219e0c2dcf2. Because
the Win-Clang builders build all targets, it would now fail because
llvm-symbolizer is really called llvm-symbolizer.exe.

BUG=82385
NOTRY=true

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

[hans, 2014-11-21 18:12:06]

hans@chromium.org changed reviewers:
+ glider@chromium.org, thakis@chromium.org

[hans, 2014-11-21 18:12:17]

Please take a look.

[Nico, 2014-11-21 18:14:08]

https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers.gyp
File build/sanitizers/sanitizers.gyp (right):

https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers....
build/sanitizers/sanitizers.gyp:69:
'../../third_party/llvm-build/Release+Asserts/bin/llvm-symbolizer',
gyp has a built-in variable for this. Say
'.......llvm-symbolizer<(EXECUTABLE_SUFFIX)'

[Nico, 2014-11-21 18:14:45]

https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers.gyp
File build/sanitizers/sanitizers.gyp (right):

https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers....
build/sanitizers/sanitizers.gyp:69:
'../../third_party/llvm-build/Release+Asserts/bin/llvm-symbolizer',
On 2014/11/21 18:14:08, Nico wrote:
> gyp has a built-in variable for this. Say
> '.......llvm-symbolizer<(EXECUTABLE_SUFFIX)'

(allows you to get rid of the conditional too)

[hans, 2014-11-21 18:24:27]

On 2014/11/21 18:14:45, Nico wrote:
>
https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers.gyp
> File build/sanitizers/sanitizers.gyp (right):
> 
>
https://codereview.chromium.org/751583004/diff/1/build/sanitizers/sanitizers....
> build/sanitizers/sanitizers.gyp:69:
> '../../third_party/llvm-build/Release+Asserts/bin/llvm-symbolizer',
> On 2014/11/21 18:14:08, Nico wrote:
> > gyp has a built-in variable for this. Say
> > '.......llvm-symbolizer<(EXECUTABLE_SUFFIX)'
> 
> (allows you to get rid of the conditional too)

Thanks! Done.

[Nico, 2014-11-21 18:32:57]

lgtm

[hans, 2014-11-21 18:37:17]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2014-11-21 18:38:14]

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

[hans, 2014-11-21 19:25:45]

The CQ bit was unchecked by hans@chromium.org

[hans, 2014-11-21 19:25:56]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2014-11-21 19:26:09]

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

[commit-bot: I haz the power, 2014-11-21 19:27:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-21 19:29:14]

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