Remove the default setting for clcompile_cmd in msvs-ninja

The 'msvs_external_builder_clcompile_cmd' logic is preserved, this commit
simply removes the default setting.  It can be restored by individual
developers based on personal preference.

BUG=352876
R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/246613003

git-svn-id: http://gyp.googlecode.com/svn/trunk@1901 78cadc50-ecff-11dd-a971-7dbc132099af
win: disable two flaky tests on gyp-win32

After much investigation (linked bug and earlier ps), I don't
really know much other than that these tests are failing when
the toolchain emits:

"LINK : unable to load ILK file; performing full link"

I haven't been able to figure out why that's happening though, so
I'm going to disable these two tests on x86 hosts so that the bots
aren't red.

I'm also going to add the diagnostic flag to the chromium build to
see if we can diagnose any unexpected incremental link failures.

R=thakis@chromium.org
BUG=365833 and https://groups.google.com/d/topic/gyp-developer/gKRi0DB-DWw/discussion

Review URL: https://codereview.chromium.org/247713002

git-svn-id: http://gyp.googlecode.com/svn/trunk@1902 78cadc50-ecff-11dd-a971-7dbc132099af
Show ASM output file instead of input file, just like CC and CXX.
R=scottmg@chromium.org

Patch by wtc@chromium.org.

Review URL: https://codereview.chromium.org/256593008

git-svn-id: http://gyp.googlecode.com/svn/trunk@1903 78cadc50-ecff-11dd-a971-7dbc132099af
gyp: avoid copying items that will be deleted. 

We spent a few CPU seconds copying items we deleted a few lines further 
down. Seems a bit wasteful. This saves around 0.5 real world seconds for 
me. 

BUG=362075
R=mark@chromium.org

Review URL: https://codereview.chromium.org/233503002/
Patch from Daniel Bratell <bratell@opera.com>


git-svn-id: http://gyp.googlecode.com/svn/trunk@1904 78cadc50-ecff-11dd-a971-7dbc132099af
gyp: isinstance -> type for another 1-5% save.

isinstance is crazily slow. Save another 3 CPU seconds (<1 real second
with 8 cores) by using type() instead. This works well since we know
exactly what types we are using.

There are still some isinstance around, but only in cold paths where
performance doesn't matter.

BUG=362075

Review URL: https://codereview.chromium.org/232023006/
Patch from Daniel Bratell <bratell@opera.com>!


git-svn-id: http://gyp.googlecode.com/svn/trunk@1905 78cadc50-ecff-11dd-a971-7dbc132099af
gyp performance: Another fixed O(n^2) in dependency calcs.

Save some time by avoiding "x in list" operations in the dependency
calculations. In a project the size of chromium this code uses about
0.4 seconds before the patch and around 0.23 seconds after the patch.

The patch is the same as in some other changes, to combine the list with
a set() for the "x in list" lookups.

BUG=362075

Review URL: https://codereview.chromium.org/240013003/
Patch from Daniel Bratell <bratell@opera.com>!


git-svn-id: http://gyp.googlecode.com/svn/trunk@1906 78cadc50-ecff-11dd-a971-7dbc132099af