Slightly simplify icu.gyp.

icu.gyp

 # Copyright (c) 2012 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.
 
 {
   'includes': [
     'icu.gypi',
   ],
   'variables': {
     'use_system_icu%': 0,
@@ skipping 137 matching lines @@
             'U_I18N_IMPLEMENTATION',
           ],
           'dependencies': [
             'icuuc',
           ],
           'direct_dependent_settings': {
             'include_dirs': [
               'source/i18n',
             ],
           },
+          # Since ICU wants to internally use its own deprecated APIs, don't
+          # complain about it.
+          'cflags': [
+            '-Wno-deprecated-declarations',
+          ],
+          'cflags_cc': [
+            '-frtti',
+          ],
+          'xcode_settings': {
+            'GCC_ENABLE_CPP_RTTI': 'YES',       # -frtti
+          },
+          'msvs_settings': {
+            'VCCLCompilerTool': {
+              'RuntimeTypeInfo': 'true',
+            },
+          },
           'conditions': [
             [ 'use_system_icu==1 and want_separate_host_toolset==1', {
               'toolsets': ['host'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==1', {
               'toolsets': ['host', 'target'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==0', {
               'toolsets': ['target'],
             }],
-            [ 'os_posix == 1 and OS != "mac" and OS != "ios"', {
-              # Since ICU wants to internally use its own deprecated APIs, don't
-              # complain about it.
-              'cflags': [
-                '-Wno-deprecated-declarations',
-              ],
-              'cflags_cc': [
-                '-frtti',
-              ],
-            }],
-            ['OS == "mac" or OS == "ios"', {
-              'xcode_settings': {
-                'GCC_ENABLE_CPP_RTTI': 'YES',       # -frtti
-              },
-            }],
-            ['OS == "win"', {
-              'msvs_settings': {
-                'VCCLCompilerTool': {
-                  'RuntimeTypeInfo': 'true',
-                },
-              }
-            }],
             ['clang==1', {
               'xcode_settings': {
                 'WARNING_CFLAGS': [
                   # ICU uses its own deprecated functions.
                   '-Wno-deprecated-declarations',
                   # ICU prefers `a && b || c` over `(a && b) || c`.
                   '-Wno-logical-op-parentheses',
                   # ICU has some `unsigned < 0` checks.
                   '-Wno-tautological-compare',
                   # uspoof.h has a U_NAMESPACE_USE macro. That's a bug,
@@ skipping 55 matching lines @@
               'source/common',
             ],
             'conditions': [
               [ 'component=="static_library"', {
                 'defines': [
                   'U_STATIC_IMPLEMENTATION',
                 ],
               }],
             ],
           },
+          'cflags': [
+            # Since ICU wants to internally use its own deprecated APIs,
+            # don't complain about it.
+            '-Wno-deprecated-declarations',
+            '-Wno-unused-function',
+          ],
+          'cflags_cc': [
+            '-frtti',
+          ],
+          'xcode_settings': {
+            'GCC_ENABLE_CPP_RTTI': 'YES',       # -frtti
+          },
+          'msvs_settings': {
+            'VCCLCompilerTool': {
+              'RuntimeTypeInfo': 'true',
+            },
+          },
           'conditions': [
             [ 'use_system_icu==1 and want_separate_host_toolset==1', {
               'toolsets': ['host'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==1', {
               'toolsets': ['host', 'target'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==0', {
               'toolsets': ['target'],
             }],
             [ 'OS == "win"', {
               'sources': [
                 'source/stubdata/stubdata.c',
               ],
             }],
-            [ 'os_posix == 1 and OS != "mac" and OS != "ios"', {
-              'cflags': [
-                # Since ICU wants to internally use its own deprecated APIs,
-                # don't complain about it.
-                '-Wno-deprecated-declarations',
-                '-Wno-unused-function',
-              ],
-              'cflags_cc': [
-                '-frtti',
-              ],
-            }],
-            ['OS == "mac" or OS == "ios"', {
-              'xcode_settings': {
-                'GCC_ENABLE_CPP_RTTI': 'YES',       # -frtti
-              },
-            }],
-            ['OS == "win"', {
-              'msvs_settings': {
-                'VCCLCompilerTool': {
-                  'RuntimeTypeInfo': 'true',
-                },
-              },
-            }],
             ['OS == "android" and use_system_stlport == 1', {
               'target_conditions': [
                 ['_toolset == "target"', {
                   # ICU requires RTTI, which is not present in the system's
                   # stlport, so we have to include gabi++.
                   'include_dirs': [
                     '<(android_src)/abi/cpp/include',
                   ],
                   'link_settings': {
                     'libraries': [
@@ skipping 267 matching lines @@
           },
           'includes': [
             '../../build/shim_headers.gypi',
           ],
           'toolsets': ['target'],
         },
       ], # targets
     }],
   ], # conditions
 }