Fix ShowGlobals to not report on enums

One of our compiler changes means that the Chrome symbols now contains
references to enums as symbols in a way that makes them show up as
duplicated variables in ShowGlobals.exe reports, despite occupying no
space. This change filters out globals that aren't statics, which avoids
this problem.

This change also updates the .vcxproj file to VC++ 2017 and the 17134.0
SDK.

This tool is discussed in the Windows Binary Sizes page:

https://www.chromium.org/developers/windows-binary-sizes

Change-Id: I73a2a7924107b5027bf68a928954e4480e5b5af5
Reviewed-on: https://chromium-review.googlesource.com/1114291
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570256}

tools/win/ShowGlobals/ShowGlobals.cc

@@ -105,6 +105,17 @@ bool DumpInterestingGlobals(IDiaSymbol* global, const wchar_t* filename) {
   for (ULONG celt = 0;
        SUCCEEDED(enum_symbols->Next(1, &symbol, &celt)) && (celt == 1);
        symbol.Release()) {
+    DWORD location_type = 0;
+    // If we can't get the location type then we assume the variable is not of
+    // interest.
+    if (FAILED(symbol->get_locationType(&location_type))) {
+      continue;
+    }
+    // Ignore location types that don't actually correspond to statics and
+    // globals.
+    if (location_type != LocIsStatic)
+      continue;
+
     // If we call get_length on symbol it works for functions but not for
     // data. For some reason for data we have to call get_type() to get
     // another IDiaSymbol object which we can query for length.