Clang on Windows: Remove -Wno-unused-variable.

remoting/remoting_host_win.gypi

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'remoting_host_installer_win_roots': [
       'host/installer/win/',
     ],
     'remoting_host_installer_win_files': [
@@ skipping 67 matching lines @@
       'defines': [
         # Prepend 'Ps' to the MIDL-generated routines. This includes
         # DllGetClassObject, DllCanUnloadNow, DllRegisterServer,
         # DllUnregisterServer, and DllMain.
         'ENTRY_PREFIX=Ps',
         'REGISTER_PROXY_DLL',
       ],
       'dependencies': [
         'remoting_lib_idl',
       ],
+      'variables': {
+        'clang_warning_flags': [
+          # Generated code contains unused variables.
+          '-Wno-unused-variable',
+        ],
+      },
       'sources': [
         '<(SHARED_INTERMEDIATE_DIR)/remoting/host/chromoting_lib.dlldata.c',
         '<(SHARED_INTERMEDIATE_DIR)/remoting/host/chromoting_lib_p.c',
       ],
       'variables': {
         'clang_warning_flags': [
           # MIDL generated code has a habit of omitting optional braces.
           '-Wno-missing-braces',
           # Source files generated by the MIDL compiler trigger warnings with
           # -Wincompatible-pointer-types enabled.
           '-Wno-incompatible-pointer-types',

[Nico, 2015/07/10 03:31:31]

gyp: Key 'variables' repeated at level 3 with key path 'targets.2' while reading
/b/build/slave/android/build/src/remoting/remoting_host_win.gypi while reading
includes of /b/build/slave/android/build/src/remoting/remoting_host.gypi while
reading includes of /b/build/slave/android/build/src/remoting/remoting.gyp


aha, add the flag here instead of duplicating the variables block

         ],
       },
     },  # end of target 'remoting_lib_ps'
 
     # Regenerates 'chromoting_lib.rc' (used to embed 'chromoting_lib.tlb'
     # into remoting_core.dll's resources) every time
     # 'chromoting_lib_idl.templ' changes. Making remoting_core depend on
     # both this and 'remoting_lib_idl' targets ensures that the resorces
     # are rebuilt every time the type library is updated. GYP alone is
     # not smart enough to figure out this dependency on its own.
@@ skipping 522 matching lines @@
 
         {
           'target_name': 'remoting_me2me_host_archive',
           'type': 'none',
         },
       ],  # end of 'targets'
     }],  # 'wix_exists == "True" and sas_dll_exists == "True"'
 
   ],  # end of 'conditions'
 }